Discussion :

[hirobym]

Salu tt le monde

j'ai un probleme avec cette methode :
[balise ajouté par mongaulois]

Code C++ :

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

1
2
3
4
5
6
7
8
9
10
11
12
13
14
15
16
void Labyrinthe::leCombat(Chambre *chr)
{
	multimap<occupant,Occupant*>::iterator iter;
	iter=chr->getPerso()->begin();
	while (iter!=chr->getPerso()->end())
	{
		if(iter->first!=hero)
		{
		  	if(hiro->combattre(iter->second,iter->first))
				//delete iter->second;
				chr->getPerso()->erase(iter);
		}
		iter++;
	}
 
}

ca compile parfaitement mais je reçois une erreur d'execution liée a erase lorsque je change erase par delete ca marche parfaitement le 1er coup et au 2eme passage ca coince on dirait que l'iterateur passe par l'objet supprimé.

PS : la methode combattre retourne une valeur booleenne

[hirobym]

aparament erase marche et le probleme lorsque je fais iter++ il trouve pas iter et genere probleme d'execution

[JolyLoic]

Quand on fait erase sur un itérateur, il devient invalide. La solution : Incrémenter avant.

Code :

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

1
2
3
multimap<occupant,Occupant*>::iterator elemAEffacer = iter;
iter++;
chr->getPerso()->erase(elemAEffacer);

[yan]

bonjour,
1-

2-Parfois erase de map/multimap ...(suivant l'implementation) retourne un pointeur valide,il faut donc utiliser celui la

Code :

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

1
2
3
4
5
6
7
8
9
if(iter->first!=hero && hiro->combattre(iter->second,iter->first))
               {
				delete iter->second;//obligatoire
				iter =chr->getPerso()->erase(iter);
		}
else
{
		iter++;
}

Pour ce que t'as dit JolyLoic,se sera faux pour un vector. Tu zappera des éléments.
http://cpp.developpez.com/faq/cpp/?p...ssion_elements

[hirobym]

merci pour votre aide,

le probleme a bien ete resolu