pointeurs d'objets dans des vecteurs

liberation

Sujet résolu

15 août 2007 à 14:13:15

Bonjour,

j'ai une classe qui contient des pointeurs d'objets regroupés dans des vecteurs,
j'ai une methode "clear()" qui doit supprimer ses pointeurs, je fais comme ca :

void CBloc_w::clear(void)
{
for(std::vector<CBloc_w*>::iterator it = m_bloc.begin(); it != m_bloc.end();)
it = m_bloc.erase(it);
for(std::vector<CEntree_w*>::iterator it = m_entree.begin(); it != m_entree.end();)
it = m_entree.erase(it);
for(std::vector<CBouton_w*>::iterator it = m_bouton.begin(); it != m_bouton.end();)
it = m_bouton.erase(it);
for(std::vector<CTexte_w*>::iterator it = m_texte.begin(); it != m_texte.end();)
it = m_texte.erase(it);
}

j'aimerais savoir si c'est la bonne facon de faire

merci

C++ Références | Idiomes | FAQ

Anonyme

15 août 2007 à 14:18:09

les vector ont une méthode qui s'appelle "clear", jettes-y un coup d'oeil. Ca t'aidera

Anonyme

15 août 2007 à 14:18:42

Tu risques d'avoir pas mal de fuites avec cette méthode, car là tu ne fais que supprimer les pointeurs et pas désallouer les objets pointés:

void CBloc_w::clear(void)
{
for(std::vector<CBloc_w*>::iterator it = m_bloc.begin(); it != m_bloc.end(); it++)
delete *it;
for(std::vector<CEntree_w*>::iterator it = m_entree.begin(); it != m_entree.end(); it++)
delete *it;
for(std::vector<CBouton_w*>::iterator it = m_bouton.begin(); it != m_bouton.end(); it++)
delete *it;
for(std::vector<CTexte_w*>::iterator it = m_texte.begin(); it != m_texte.end(); it++)
delete *it;
}

Tu n'as pas besoin de supprimer chaque élément du vecteur par contre, std::vector s'en charge pour toi à sa destruction

Nanoc

15 août 2007 à 14:21:43

Hello,

Alors je te le dis tout de suit comme ça, ça ne marchera pas.

Si tes pointeurs ne sont pas crés via des "new", tu peux utiliser la méthode "clear" de la classe std::vector.

std::vector<*blabla> monVector;//On cree un vector de pointeurs
//Le vector vit plein d'aventures palpitantes
monVector.clear(); //On vide le vector

Si tu obtients tes pointeurs via des "new" c'est autre chose, mais je pense pas que c'est ce que tu voulais faire puisque tu n'uilises pas de delete dans ton code.

EDIT: grilled

Co-auteur du cours de C++. ||| Posez vos questions sur le forum ||| Me contacter.

chlablak

15 août 2007 à 14:25:19

Citation : Cyprien_

Tu risques d'avoir pas mal de fuites avec cette méthode, car là tu ne fais que supprimer les pointeurs et pas désallouer les objets pointés:

ma classe n'alloue rien, elle ne fais que pointée sur des objets deja créé.

maintenant je fais comme cela :

void CBloc_w::clear(void)
{
m_bloc.clear();
m_entree.clear();
m_bouton.clear();
m_texte.clear();
}

c'est OK ?

EDIT: ok merci

C++ Références | Idiomes | FAQ

lmghs

15 août 2007 à 22:25:46

Bof. Regarde la FAQ C++ de developpez.