Discussion :

[scary]

Bonjour,

Voila je n'arrive pas à voir pourquoi lorsque je fais un erase puis un delete sur ma map mon application plante. Voici le petit bout de code :

Code :

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

1
2
3
4
5
6
7
8
9
10
11
12
void Image::_add_span (int old_id_span, int new_id_span) throw () {
	std::map<int, Span *>::iterator old_it = this->_listSpan.find (old_id_span);
	std::map<int, Span *>::const_iterator new_it = this->_listSpan.find (new_id_span);
	PersoTypes::ListPoints old_pixels = old_it->second->content ();
 
	for (PersoTypes::ListPoints::const_iterator it=old_pixels.begin ();it!=old_pixels.end ();it++) {
		this->_pixel (*it).id_span (new_id_span);
		new_it->second->content ().push_back (*it);
	}
	this->_listSpan.erase (old_it);
	delete old_it->second;
}

Ce sont les deux dernières lignes de la méthode qui foire. Quelqu'un aurait une idée ? Merci d'avance

[Oussapik]

Je suppose que ta variable this->_listSpan est de type std::List<Span> ou std::List<Span *>.

Si c'est bien le cas, l'appel de la méthode erase avec un iterator en paramètre enlève bien l'élément de la liste, mais il en profite aussi pour appeler le destructeur de l'élément en question.

Tu as donc détruit la variable old_it. Tu ne peux donc pas utiliser l'opérateur -> dessus après. Je pense que ton erreur vient de là.

Source : This effectively reduces the list size by the number of elements removed, calling each element's destructor before. (cplusplus.com)

[scary]

non mon "this->_listSpan" est un "std::map<int, Span *>" et même si je ne met pas le "delete" à la fin l'application plante

[Oussapik]

pourrais-tu donner le message d'erreur qui apparait quand l'application plante ?

[scary]

C'est une application graphique et cela ce traduit par une erreur de ma dll. Il n'y a donc pas de message d'erreur précisément.

[3DArchi]

Salut,
Tu n'es pas obligé de mettre 'this->' pour accéder aux membres de ton objet dans une fonction de ta classe. Cela nuit un peu à la lisibilité.
Ceci dit, je soupçonne moins un problème de ta map que plutôt un problème de gestion de pointeurs bruts dans ta map. Cf Les Pointeurs intelligents.

[nowahn]

Salut,

je n'arrive pas à voir pourquoi lorsque je fais un erase puis un delete sur ma map mon application plante.

Pour répondre à cette question, après ton erase(), la paire pointée par l’itérateur olt_it n’existe plus, tu ne peux donc pas récupérer son membre second.
Il te faut simplement faire le delete avant le erase. Mais comme le dit 3DArchi, tu devrais aller voir du côté des pointeurs intelligents.

et même si je ne met pas le "delete" à la fin l'application plante

Là, je comprend pas pourquoi, ça devrait « juste » provoquer une fuite de mémoire.

[scary]

Oui j'avais vu ma première erreur car d'après ce que j'ai pu lire dans la doc du erase, au moment du erase chaque destructeur de ce que l'on veut effacé est appellé donc cela en effet aurait créé une fuite mémoire.

Ensuite je ne comprends pas pourquoi en effet cela plante...

Je vais relire mon appli depuis le début et regardé si j'ai pas fais une bétise plus haut.

Pour les smart pointer en effet c'est très intéressant je vais regarder cela de plus près

[scary]

Voila j'ai résolue mon problème, en fait il était plus haut, j'analysais une partie d'un tableau qui n'existait plus et donc quand je voulais accéder au span corespondant à cet id (contenu dans le tableau) dans la map et bien il plantait

Autant pour moi et merci