[STL]destruction d'objet d'un vector

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

Code:

---

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:

---

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.

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<=

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

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

Code:

---

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:

---

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.

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 ?

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:

---

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?

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.

Ca, ce n'est pas grave puisqu'il est detruit juste apres.

Merci pour ton aide.