Création d'une structure, erreur de segmentation

Version imprimable

Hello, j'ai un souci avec la création de la structure suivante :
Code:

1 2 3 4 5 6 7 8 9 struct ligne_MTE { int terminal; // terminal associé a la table int ElementListSize; //nombre d'elements associé a la ligne int MultiplexTableEntryNumber; //numero inclus dans l'entree int NumeroLCN; // logical channel associé std::string RepeatCount; // RC (UCF ? chiffre ?) std::string type; // audio / video / both };
En effet, lorsque, dans une fonction j'effecture la commande :

Code:

ligne_MTE resultat;

Le programme n'a aucune erreur de compilation, mais plante sur ce point précis (erreur de segmentation).
S'agit-il d'un problème d'allocation mémoire ? comment le prévenir ?

10/07/2006, 09h51
Médinoc

Avec si peu d'information, je ne vois rien qui pourrait faire planter là-dedans...
L'erreur doit venir d'autre part.
10/07/2006, 09h54
Matthieu Brucher

C'est aussi bon pour moi, ce code n'a pas de souci. Essaie d'élaguer ton code pour trouver le problème.

Effectivement, ce code ne posait aucun problème sous environnement dev-cpp/windows, mais son adaptation sous Solaris est plus problématique.

La partie du main concernée :

Code:

1
2
3
4
5
6
7
8
9
10
11
12
13
//--------------------------------------GENERER LA MTE-------------------------------------//

cout << "GENERATION DE LA MTE ..."<< endl;


MTE table1;int index_balayage=0;int row = 0;
//while(1){
table1.recup_tableau_vals(tableau_valeurs);
int temp = index_balayage;
table1.set_index(index_balayage);
table1.recup_ligne(row);
table1.affichetable_MTE();
std::cout << "FIN AFFICHAGE DE LA TABLE" << std::endl;

Le début du code de "recup_ligne" :

Code:

1
2
3
4
5
6
7
8
9
10
11
12
13
14
15
16
17
18
19
20
21
22
23
24
25
26
27
28
29
30
31
32
33
34
35
36
37
38
39
40
41
42
43
44
45
46
47
48
49
50
51
52
53
54
55
56
57
void MTE::recup_ligne(int &row){

std::cout <<  "fonction recup lignes" << std::endl;
std::cout <<  "creation de l iterateur" << std::endl;
std::vector<struct ligne_valeurs>::iterator iter = tableau_vals.begin() + index +1 ;

int nombre_desc = 0;
ligne_MTE resultat;
std::cout <<  "creation de la ligne MTE resultat" << std::endl;

std::cout << "debut de la boucle while pour balayer le tableau" << std::endl;
while (iter != tableau_vals.end ()) {

int SNrep;int SNreq;
	  if(iter -> correspondance.find("multiplexEntrySend (6)") != -1)
	  {std::cout << "MES trouve" << std::endl;
	   		  
//                                    MES ACQUITTE PAR L'AUTRE TERMINAL ?                         //
    
	  int terminalreq = iter ->num_terminal;
	  std::istringstream iss(((iter+1)->correspondance).substr(0,1));
	  iss>>SNreq;
      //std::vector<struct ligne_valeurs>::iterator iter2 = tableau_vals.begin() + std::distance(tableau_vals.begin(), iter) ;	  
	  std::vector<struct ligne_valeurs>::iterator iter2 = tableau_vals.begin();
	  
	  while (iter2 != tableau_vals.end ()) {
	  		std::cout << "(iter2->correspondance) : " << (iter2->correspondance) << std::endl;
	  if((iter2->correspondance).find("multiplexEntrySendAck")!=-1)
	  {
	   															   
      int terminalrep = iter2 -> num_terminal;
	  //std::cout <<  "Terminal Req : " << terminalreq << " et Terminal Rep : " << terminalrep <<std::endl;
	  if(terminalrep == terminalreq){break;}

	  std::cout <<  "MESAck Trouve" << std::endl; 															   
      std::istringstream iss(((iter2+1)->correspondance).substr(0,1));
	  iss>>SNrep;
	  std::cout <<  "SNReq vaut : " << SNreq << " et SNRep vaut : " << SNrep << std::endl;
	  if(SNrep == SNreq){break;}
	  }
	  iter2++;
	  }
	  std::cout <<  "Fin balayage tableau pour verif d'Ack" << std::endl;
	  int ack =1;
	  //VALEUR A MODIFIER POUR OUTREPASSER L'ACQUITTEMENT MES
	  if(SNrep == SNreq){
	  		     ack =1;
				 std::cout <<  "TRAME ACQUITTEE => MTE VALABLE"<<std::endl;
				 }
	  
	  if(ack==0){
  			 std::cout <<  "TRAME NON ACQUITTEE => MTE NON VALABLE"<<std::endl;
	  			 break;}
	  
	  std:: cout << "Fin ACK" << std::endl;

//                                              MES ACQUITTEE                               //

Le truc, c'est que la méthode plante au début, sur la création de la structure, ce qui est très étonnant, a priori. Comment s'assurer qu'il n'y a pas un problème lié à la mémoire ?

10/07/2006, 10h09
Matthieu Brucher

Je me poserai plus de questions avec l'itérateur qu'avec cette structure qui n'a rien de particulier. Tu as essayé en la supprimant ? au fait, pourquoi passer row en argument référence ?
10/07/2006, 10h19
Thordax

Le passage en référence est effectivement inutile, je le supprime de ce pas.

Pour ce qui concerne l'itérateur, il est nécessaire pour le "balayage" d'un tableau de valeurs. Supprimer la structure ? pour la remplacer par ?

La méthode rajoute des éléments dans une structure créée au départ (ici la structure "resultat"), puis rajoute la structure dans un tableau de structures (vector de variables différentes).

Si je supprime la structure créée au départ, je ne peux plus faire fonctionner la méthode.
10/07/2006, 10h36
Matthieu Brucher

Ben fais-le une fois pour voir si c'est bien la structure qui cause ton erreur.
10/07/2006, 10h43
SnakemaN

Heu je voudrais pas dire de bétises, mais a aucun moment tu fait une allocation memoire pour ta structure ? meme pô un p'tit malloc() ?
:?
10/07/2006, 10h46
Thordax

Le malloc() n'est-il pas propre au C et non au C++ ?
10/07/2006, 10h47
Matthieu Brucher

Si si, mais parfois, certaines personnes l'utilisent encore.

Mais là n'est pas la question, puisque la structure est directement sur la pile ici.

Edit: J'ai un peu réindenté le code, car on n'y comprenait rien:

Code:

1
2
3
4
5
6
7
8
9
10
11
12
13
14
15
16
17
18
19
20
21
22
23
24
25
26
27
28
29
30
31
32
33
34
35
36
37
38
39
40
41
42
43
44
45
46
47
48
49
50
51
52
53
54
55
56
57
58
59
60
61
62
63
64
void MTE::recup_ligne(int &row)
{
    std::cout <<  "fonction recup lignes" << std::endl;
    std::cout <<  "creation de l iterateur" << std::endl;
    std::vector<struct ligne_valeurs>::iterator iter = tableau_vals.begin() + index +1 ;
 
    int nombre_desc = 0;
    ligne_MTE resultat;
    std::cout <<  "creation de la ligne MTE resultat" << std::endl;
 
    std::cout << "debut de la boucle while pour balayer le tableau" << std::endl;
    while (iter != tableau_vals.end ())
        {
 
        int SNrep;int SNreq;
        if(iter -> correspondance.find("multiplexEntrySend (6)") != -1)
        {
            std::cout << "MES trouve" << std::endl;
 
            //                                    MES ACQUITTE PAR L'AUTRE TERMINAL ?                         //
 
            int terminalreq = iter ->num_terminal;
            std::istringstream iss(((iter+1)->correspondance).substr(0,1));
            iss>>SNreq;
            //std::vector<struct ligne_valeurs>::iterator iter2 = tableau_vals.begin() + std::distance(tableau_vals.begin(), iter) ;      
            std::vector<struct ligne_valeurs>::iterator iter2 = tableau_vals.begin();
 
            while (iter2 != tableau_vals.end ())
            {
                std::cout << "(iter2->correspondance) : " << (iter2->correspondance) << std::endl;
                if((iter2->correspondance).find("multiplexEntrySendAck")!=-1)
                {
                    int terminalrep = iter2 -> num_terminal;
                    //std::cout <<  "Terminal Req : " << terminalreq << " et Terminal Rep : " << terminalrep <<std::endl;
                    if(terminalrep == terminalreq)
                        { break; }
 
                    std::cout <<  "MESAck Trouve" << std::endl;                                                                
                    std::istringstream iss(((iter2+1)->correspondance).substr(0,1));
                    iss>>SNrep;
                    std::cout <<  "SNReq vaut : " << SNreq << " et SNRep vaut : " << SNrep << std::endl;
                    if(SNrep == SNreq)
                        { break; }
                }
                iter2++;
            }
            std::cout <<  "Fin balayage tableau pour verif d'Ack" << std::endl;
            int ack =1;
            //VALEUR A MODIFIER POUR OUTREPASSER L'ACQUITTEMENT MES
            if(SNrep == SNreq)
            {
                ack =1;
                std::cout <<  "TRAME ACQUITTEE => MTE VALABLE"<<std::endl;
            }
 
            if(ack==0)
            {
                std::cout <<  "TRAME NON ACQUITTEE => MTE NON VALABLE"<<std::endl;
                break;
            }
 
            std:: cout << "Fin ACK" << std::endl;
 
            //                                              MES ACQUITTEE                               //

10/07/2006, 10h50
SnakemaN

oui en C++ il est sensé allouer la mem en meme temps que l'instanciation mais bon ca coute rien de l'alouer d'abord pouis de la remplir...pour voir
oui oui....:aie: (fiouuu fatigué moua)

mais tu code sous win ou linux car avec un debogueur ce serait plus precis pour trouver l'erreur :)
10/07/2006, 10h56
SnakemaN

Et avec un :

ligne_MTE* resultat;

?
10/07/2006, 10h57
Matthieu Brucher

A quoi ça servirait ?
10/07/2006, 11h01
Médinoc

Au fait, c'est normal que resultat ne soit pas utilisé dans le code posté ?
10/07/2006, 11h09
SnakemaN

Citation:

Envoyé par Miles

A quoi ça servirait ?

Ben comme ca il pourrait utiliser le pointeur vers cette structure, ca serait plus propre question gestion de memoire, ustement parceque la dans ce code il n'est apparement pas utilisé, c'est qu'il doit servir ailleur et peut etre que ailleur il s'en sert justement comme un pointeur vers cette structure...ouf :P

Moi je m'arrange toujours pour utliser des pointeurs, c'est plus chiant mais plus "propre", en fin je dit ca mais ca n'engage que moi :)
10/07/2006, 11h11
Matthieu Brucher

Ah, OK ;)
Je n'utilise des pointeurs que lorsque la ressource doit être partagée entre plusieurs instances de classe. Et alors, j'utilise aussi des pointeurs intelligents - cf ma signature -
10/07/2006, 11h22
SnakemaN

D'ailleur si cette structure est vraiment utilisée j'en ferai un "vrai" objet, puisque tu utilise le C++ autant en utliser les avantages !
grace au new() et au delete() pas de probleme de mémoire ! :D

ps: a oui ca a l'air sympa les pointeur inteligents, j'en avais entendu parler mais pas encore utilisé, merci Miles !
10/07/2006, 11h28
Matthieu Brucher

Citation:

Envoyé par SnakemaN

ps: a oui ca a l'air sympa les pointeur inteligents, j'en avais entendu parler mais pas encore utilisé, merci Miles !

Je t'en prie, on est là pour partager ;)
11/07/2006, 00h40
Luc Hermitte

NB: malloc n'est pas compatible avec des classes. La construction n'est pas réalisée.
11/07/2006, 17h11
Thordax

Merci pour toutes vos réponses, je ne suis pas retournée ici depuis la question.

Le problème ne venait clairement pas d'une allocation mémoire, mais d'un problème de fond bien avant l'appel de cette méthode.