Discussion :

[ZaaN]

hey hey,

j ai une std::list dont la boucle de parcours (pour le delete) ne joue pas correctement.

Si j ai un element dans la liste, lors du premier passage tout va bien mais la condition de sortie est ma l evalué et je tente de supprimer un auter objet inexistant.

Code :

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

1
2
3
4
5
6
for(l_itProcess=m_ProcessList.begin();l_itProcess!=m_ProcessList.end();l_itProcess++)//read list
{						
	//deleting process in list				
	delete  (*l_itProcess);
	m_ProcessList.erase(l_itProcess);
}

des idées ?

[Laurent Gomila]

http://c.developpez.com/faq/cpp/?pag...ssion_elements

Mais pourquoi ne pas faire un m_ProcessList.clear() après la boucle s'il faut effacer tous les éléments ? Pourquoi ne pas stocker des boost::shared_ptr si ce sont des variables allouées dynamiquement ?

[Sifo-Dyas]

Essaies de parcourir ta liste en sens inverse (du dernier au 1er).
Je pense que ça devrait mieux fonctionner .

[ZaaN]

tout dans la comme d hab !

[Mat007]

Essaies de parcourir ta liste en sens inverse (du dernier au 1er).
Je pense que ça devrait mieux fonctionner .

Je ne suis pas sûr que ça change grand chose, ou alors j'ai raté un truc ?

MAT.

[ZaaN]

Essaies de parcourir ta liste en sens inverse (du dernier au 1er).
Je pense que ça devrait mieux fonctionner .

ptet qu avec cette methode je tomberais sur l'addresse de base du conteneur en remontant et pas dans le vide en decendant. Mais je prefère de loin travaillé propre avec la methode conseillé par Laurent Gomila.

[Laurent Gomila]

Aller en sens inverse ne changera rien, après effacement du dernier élément on se retrouverait tout autant à pointer sur n'importe quoi.

[Sifo-Dyas]

ZaaN a écrit :
ptet qu avec cette methode je tomberais sur l'addresse de base du conteneur en remontant et pas dans le vide en decendant. Mais je prefère de loin travaillé propre avec la methode conseillé par Laurent Gomila.

tu as raison.

ps : Je développe principalement avec les MFC et lorsque je dois détruire un tableau ou une liste de pointeur j'utilise cette méthode, qui marche très bien.