Discussion :

[yupa]

Bonjour

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

Code :

Sélectionner tout - Visualiser dans une fenêtre à part

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

[yan]

Bonjour

je dirait que ton code est faux. tout simplement.

Code :

Sélectionner tout - Visualiser dans une fenêtre à part

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 :

Sélectionner tout - Visualiser dans une fenêtre à part

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 :

Sélectionner tout - Visualiser dans une fenêtre à part

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();

[yupa]

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

[NiamorH]

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 :

Sélectionner tout - Visualiser dans une fenêtre à part

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

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