Discussion :

[sylvain.cool]

Bonjour,
J'ai creer un vector de pointeur vers une classe CHole de mon cru:

Code :

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

1
2
 
vector<CHole*> m_pHoles;

Apres avoir utilise ce vector, je m'apercois que certains elements doivent etre fusionnés avec d'autres. Je dois donc supprimer certains elements de mon vector. Ce que je fais comme ceci:

Code :

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

1
2
3
4
5
6
7
8
9
10
11
 
// delete regions which isn't source
vector<CHole*>::iterator it=m_pHoles.begin();
for(i=0;i<saveNumberOfRegion;i++){
	if(tabRegion[i]!=-1){
		delete m_pHoles[i];
		it=m_pHoles.erase(it);
	}
	else
		it++;
}

Mais malheureusement, ca plante a l'execution.
Peut-etre est-ce mon destructeur de la classe CHole qui ne marche pas? En executant en mode debug, je pense que cela ne vient pas de la....
Aurais-je fais quelque chose qu'il ne fallait pas?
Merci d'avance pour vos reponses.

[koala01]

Salut,

La suppression d'un itérateur rend celui-ci invalide.
Du coup, la solution passe par plusieurs étapes:

• déplacer les éléments en fin de tableaux en récupérant l'itérateur du premier élément à supprimer à l'aide de remove_if et std::mem_fun
• libérer la mémoire allouée dynamiquement des éléments à supprimer
• suprimer définitivement les éléments du tableau.

Tu trouveras un exemple de laurent gomila sur =>ce post<=

[sylvain.cool]

J'ai trouve d'ou venait l'erreur!

C'est au niveau du delete que j'ai fait une erreur:

Code :

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

1
2
 
delete m_pHoles[i];

ceci ne marche apparement pas correctement, mais je ne sais pas trop pourquoi.

Par contre en remplacant par:

Code :

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

1
2
 
delete (*it);

Ca marche nickel!

Merci pour ta reponse, meme si elle ne m'a pas trop aide... quoique c'est grace au post que j'ai resolu le probleme.

[Laurent Gomila]

Ta boucle est de toute façon incorrecte. A la première suppression, ton itérateur it et ton indice i ne correspondront plus.

Si tu as deux tableaux dont les éléments sont liés deux à deux, il faut veiller à ce qu'ils restent synchronisés (ie. une suppression dans l'un doit donner lieu à une suppression dans l'autre). D'ailleurs, pourquoi maintenir deux tableaux en parallèle ? Ou bien est-ce que j'ai mal compris ce que tu faisais ?

[sylvain.cool]

En fait, j'ai un tableau qui me dit quelle region doit etre fusionnee avec qui (tabRegion) et un tableau contenant ces regions (m_pHoles).
Dans le premier tableau, si la valeur est differentes de -1, je dois supprimer la region correspondante.
Je parcours donc le premier tableau a la recherche de regions devant etre supprimees. L'iterateur est incremente a chaque fois.
Effectivement, dans la premiere version, le i et l'iterateur n'etait pas synchro. Mais je crois que c'est maintenant le cas:

Code :

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

1
2
3
4
5
6
7
8
9
10
11
 
// delete regions which isn't source
vector<CHole*>::iterator it=m_pHoles.begin();
for(i=0;i<saveNumberOfRegion;i++){
	if(tabRegion[i]!=-1){
		delete (*it);
		it=m_pHoles.erase(it);
	}
	else
		it++;
}

non?

[Laurent Gomila]

Oui et non. Ta boucle fournira le résultat attendu, mais ton tableau tabRegion ne sera plus à jour, il comportera toujours les -1 alors que les trous correspondant auront été supprimés de m_pHoles.