Bug tres etrange avec une itération sur une map

Bonjour

j'ai un bug que je ne comprends pas, voici le code :

Code:

---

1 2 3 4 5

for (std::map<int, SchedTask*>::iterator i = handlers.begin(); i != handlers.end(); ++i) { handlers.erase(i->second->id); delete i->second; }

---

je compile sous linux avec GCC4.1 -> ca plante : l'iterator continue même s'il est en dehors de la map (donc bug sur le delete)
je compile avec VS 2005 -> ca marche.

et il me semble que chez un collègue ca marchait il y a qq mois (sous linux GCC3.X)

Est ce que qq trouve ca normale?
Est ce que qq a une idée a pour résoudre ce probleme?

merci a bientot

Bonjour

je dirait que ton code est faux. tout simplement.

Code:

---

1 2 3 4 5

for (std::map<int, SchedTask*>::iterator i = handlers.begin(); i != handlers.end(); ++i) { handlers.erase(i->second->id);//pourquoi utiliser id??, sinon cela invalide i. De coup il y as un problème avec ++i delete i->second;//indéfinie }

---

tu devrait plustôt faire :

Code:

---

1 2 3 4 5 6

std::map<int, SchedTask*>::iterator end = handlers.end(); for (std::map<int, SchedTask*>::iterator i = handlers.begin(); i != end ; i=handlers.erase(i)) { delete i->second; }

---

ou (je pense plus rapide)

Code:

---

1 2 3 4 5 6 7

std::map<int, SchedTask*>::iterator end = handlers.end(); for (std::map<int, SchedTask*>::iterator i = handlers.begin(); i != end ; ++i) { delete i->second; } handlers.clear();

---

Et bien merci beaucoup par ta réponse.
ca marche super
Encore merci
a bientot

Fais attention, dès que tu modifies un conteneur (ajout ou suppression d'élément) les itérateurs peuvent (si ce n'est pas toujours?) devenir invalides.

Pense aussi à faire des typedef sur tes conteneurs :

Code:

---

1 2	public: typedef std::map<int, SchedTask*> SchedTaskMap

---

Ca évite de réécrire du code lourd et de faire des erreurs.