[C++/STL] list et parcours avec un itérateur : problème

list et parcours avec un itérateur : problème [C++/STL] - C++ - Programmation

Marsh Posté le 28-04-2004 à 14:54:46    

Bonjour tout le monde,
 
J'ai un petit soucis avec un parcours de list. Ca doit être quelque chose d'assez évident, mais je ne le vois pas. Voici le code qui segfault :
 

Code :
  1. #include <iostream>
  2. #include <list>
  4. using namespace std ;
  6. struct XY {
  7. unsigned x,y ;
  8. } ;
  10. struct pix {
  11. XY coord ;
  12. } ;
  14. struct conteneur {
  15. list<pix>* pixels ;
  16. conteneur()  { pixels = new list<pix> ; }
  17. ~conteneur() { delete pixels ; }
  18. } ;
  20. int main(int argc, char* argv[])
  21. {
  22. conteneur foo ;
  24. pix fake1 = {0,1} ;
  25. pix fake2 = {2,3} ;
  26. pix fake3 = {4,5} ;
  28. foo.pixels->push_back( fake1 ) ;
  29. foo.pixels->push_back( fake2 ) ;
  30. foo.pixels->push_back( fake3 ) ;
  32. list<conteneur> bar(10, foo) ;
  33. list<conteneur>::iterator i = bar.begin() ;
  35. while ( i != bar.end() ) {
  36.  list<pix>::iterator j = (*i).pixels->begin() ;
  38.  while ( j != (*i).pixels->end() ) ++j ;
  40.  ++i ;
  41. }
  42. }


 
Est-ce que ca vous saute aux yeux ?
Merci pour votre aide !
 
@+

Reply

Marsh Posté le 28-04-2004 à 14:54:46   

Reply

Marsh Posté le 28-04-2004 à 14:57:29    

c'est quoi l'erreur exactement ?


---------------
Fleur de métal, entité invulnérable, vêtue tant bien que mal, d'une muraille inébranlable...
Reply

Marsh Posté le 28-04-2004 à 14:58:19    

je pige pas pourquoi tu utilises des itérateur, c'est c'est pour faire while (i != ) et ++i ;
 
utilise foreach

Reply

Marsh Posté le 28-04-2004 à 15:12:30    

myst78 a écrit a écrit :

c'est quoi l'erreur exactement ?




 
segfault [:ddr555]
 

JagStang a écrit a écrit :

je pige pas pourquoi tu utilises des itérateur, c'est c'est pour faire while (i != ) et ++i ;
utilise foreach




 
Comment on fait pour parcourir une liste et effectuer un traitement différent pour certains de ses élements ? Même si ca n'est pas la solution la plus élégante j'aimerais bien comprendre pourquoi ca ne fonctionne pas.


Message édité par Evadream -jbd- le 28-04-2004 à 15:12:42
Reply

Marsh Posté le 28-04-2004 à 15:17:13    

hmm la comme ca je vois pas moi
t'as essaye un mode debug pour voir a quelle iteration ca plante ?


---------------
Fleur de métal, entité invulnérable, vêtue tant bien que mal, d'une muraille inébranlable...
Reply

Marsh Posté le 28-04-2004 à 15:27:36    

Code :
  1. list<conteneur> bar(10, foo) ;
  2. list<conteneur>::iterator i = bar.begin() ;


 
Avec ces 2 lignes, tu crée 10 copies de foo que tu met dans la liste bar. Hors, comme tu n'as pas de constructeur par recopie, chacun de ces foo se partage le pointeur pixel. Il suffit que cette ligne déclenche la construction d'un objet temporaire qu'elle desaloue imédiatement pour que la liste pixel soit desalouée et que toutes tes copies de l'objet foo  dans la liste bar se retrouvent avec un pointeur vers une zone de mémoire manquante.

Reply

Marsh Posté le 28-04-2004 à 15:32:46    

grillaid :(
je dirais donc même plus, c à la destruction que ça plante.
si tu ne veux pas d'erreur, n'utilise pas de pointeurs, ou défini un constructeur par recopie.
ceci devrait fonctionner.

Code :
  1. #include <iostream>
  2. #include <list>
  4. using namespace std ;
  6. struct XY {
  7.    unsigned x,y ;
  8. } ;
  10. struct pix {
  11.    XY coord ;
  12. } ;
  14. struct conteneur {
  15.    list<pix> pixels ;
  16. //   conteneur()  { pixels = new list<pix> ; }
  17. //   ~conteneur() { delete pixels ; }
  18. } ;
  20. int main(int argc, char* argv[])
  21. {
  22.    conteneur foo ;
  23.  
  24.    pix fake1 = {0,1} ;
  25.    pix fake2 = {2,3} ;
  26.    pix fake3 = {4,5} ;
  27.    
  28.    foo.pixels.push_back( fake1 ) ;
  29.    foo.pixels.push_back( fake2 ) ;
  30.    foo.pixels.push_back( fake3 ) ;
  31.  
  32.    list<conteneur> bar(10, foo) ;
  33.    list<conteneur>::iterator i = bar.begin() ;
  34.  
  35.    while ( i != bar.end() ) {
  36.       list<pix>::iterator j = (*i).pixels.begin() ;
  37.    
  38.       while ( j != (*i).pixels.end() ) ++j ;
  39.      
  40.       ++i ;
  41.    }
  43.    cout << "done";
  44.    cin.ignore();
  45.    cout << endl;
  46.    return 0;
  47. }


Reply

Marsh Posté le 28-04-2004 à 15:40:33    

Kristoph > Bien vu ! Merci à toi.

Code :
  1. struct conteneur {
  2.         list<pix>* pixels ;
  3.         conteneur()  { pixels = new list<pix> ; }
  4.         ~conteneur() { delete pixels ; }
  5.         conteneur(const conteneur& c) {
  6.                  pixels = new list<pix>(*(c.pixels)) ;
  7.         }
  8. } ;


 
SoWhatIn22 > Ze veux (j'ai besoin) de mes pointeurs ! ;)
 
Merci à tous pour vos interventions et votre aide !
 
@++


Message édité par Evadream -jbd- le 28-04-2004 à 17:04:48
Reply

Marsh Posté le 28-04-2004 à 16:36:35    

pas du tout une bonne idée, le delete dans ton constructeur par recopie, puisque tu détruis de la mémoire non allouée. A la recherche du nouveau segfault?

Reply

Marsh Posté le 28-04-2004 à 17:04:28    

Copier/coller malheureux, merci de l'avoir fait remarquer.

Reply

Sujets relatifs:

Leave a Replay

Make sure you enter the(*)required information where indicate.HTML code is not allowed