Discussion :

[mister3957]

Bonjour à tous,

J'ai un petit soucis avec les vector...
En fait quand on fait un push_back, il supprime l'object qu'on vient s'insérer après l'avoir recopié dans sa liste.
Mon objet que j'insère contient un pointeur vers un autre objet, que j'initialise dans l'implémentation avec un new, et que je détruit dans le destructeur s'il existe.

Le soucis est que le fait de l'insérer dans le vector ça le recopier puis le supprime, alors ça apel son destructeur et donc ça supprime son objet membre...
Vous voyez l'truc?

Donc je peux faire des vector de pointeur mais bon, j'aimerai éviter c'est pour ça.

A bientôt

[Laurent Gomila]

Un vecteur va toujours effectuer des copies de tes objets, il ne stockera jamais les originaux.

Donc si ton objet ne peux pas être copié stocke des pointeurs, sinon c'est que ton objet est mal conçu.

[mister3957]

Ben j'ai fait ça:

Code :

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
17
18
19
20
 
class Objet1
{
    public:
        Objet1();
        ~Objet1();
 
    private:
        Objet2* m_objet2;
};
 
Objet1::Objet1()
{
    m_objet2 = new Objet2();
}
 
Objet1::~Objet1()
{
    delete m_objet2;
}

Ensuite autrepart j'ai

Code :

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

1
2
3
4
5
 
{
    Objet1 objet1;
    m_vector.push_back(objet1);
}

Avec ça il passe deux fois dans le destructeur de objet1. Une fois en faisant push_back, et une fois quand on passe l'accolade. Lors de la recopie de l'objet1, il a recopié le pointeur vers l'objet2, mais il pointe toujours vers le même Objet2 qui a été détruit précédemment car le destructeur a été appelé...

Donc c'est normal que ça ne fonctionne pas, mais comment je peux faire pour que ça aille?

Merci LouLou

P.S.: T'as vu mes "m_"? :p

[Laurent Gomila]

Si ta classe est copiable alors il faut lui ajouter un constructeur de copie et un opérateur d'affectation. Ces deux-là géreront en l'occurence la copie correcte (duplication, comptage de référence, clonage, ...) de ton pointeur membre.

Le mieux serait tout de même d'utiliser un pointeur intelligent, ça t'éviterait bien des ennuis.

[mister3957]

C'est quoi un pointeur intelligent? Comment tu fais ça?

Merci Loulou

[bigboomshakala]

http://c.developpez.com/faq/cpp/?pag...S_intelligents

[mister3957]

merci

[Matthieu Brucher]

C'est pas très conseillé d'utiliser les auto_ptr dans ce cas car dans les opérations sur les conteneurs standards, il peut y avoir des recopies sans le dire. Or, si tu utilises un auto_ptr dans un objet qui est recopié pour faire une comparaison par exemple, l'auto_ptr cède la gestion du pointeur à la copie qui sera détruite à la fin de la fonction de comparaison et donc on perd l'objet membre.
Le mieux est d'utiliser les smart pointers de boost.

C'est bien d'utiliser ce genre d'idiome

[mister3957]

Oui j'ai vu dans la fac des trucs sur boost, je vais télécharger la librairie, la doc etc... et l'intégrer au maximum.

Merci beaucoup