Discussion :

[petaf]

Bonjour,

je souhaiterai écrire dans l'idée le code suivant mai je n'arrive pas à écrire le destructeur
si quelqu'un à une idée ou bien me dire si je suis à coté de la plaque
merci

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
21
22
23
24
25
26
27
28
29
30
31
32
33
34
class A
{
public:
    A(int a);
 
private:
    int m_a;
};
 
A::A(int a): m_a(a)
{
 
}
 
class B
{
public:
    B();
    ~B();
 
private:
    std::vector<B> *m_essai;
};
 
B::B():m_essai(new std::vector<B>)
{
 
}
 
B::~B()
{
    for(auto c:m_essai)
        delete C;
}

[Matt_Houston]

B : un objet de type B.
B * : un pointeur vers un objet de type B.
std::vector<B> : un std::vector dont les éléments sont des objets de type B.
std::vector<B> * : un pointeur vers un std::vector dont les éléments sont des objets de type B.

Vois-tu comment déclarer ton vecteur pour obtenir ce que tu souhaitais initialement ?


Lorsque tu auras compris comment déclarer tes pointeurs et collections de pointeurs, nous verrons pourquoi il convient dans ton cas de les remplacer par une construction de type shared_ptr ou unique_ptr..

[nikko34]

Déjà il faut se demander "pourquoi des pointeurs?"

[Bousk]

- tu n'as pas de vector de pointeur mais un vector de B... dans B
- un vector de B dans B, à part si tu écris un truc à base de node, json etc y'a peu de chance que ce soit correct
- m_essai est vector*, ce qui n'a généralement aucun sens
- à fortiori quand on l'initialise via new dans le constructeur
- for(auto c:m_essai) delete C;> appelle delete sur une variable qui n'existe pas
> et qui n'est pas du tout un pointeur, c'est tout au mieux un... B

[orilion]

Voici un exemple de vecteur de pointeur :

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
 
#include <vector>
 
class A{
public:
      int a;
      A(int a){
          this->a = a;
     }
}
 
int main(int argc, char *argv[]){
     vector<A*> list;
     list.push_back(new A(2));
}

Si tu a :

vector<A> *list;

et que tu veux détruire list, tu n'a qu'a faire :

delete(list) ;

mais si tu a :

vector<A*> *list;

Alors il faut faire :

for(A *a : list)
delete(a);
delete(list);

[petaf]

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
21
22
23
24
25
26
27
28
29
30
31
32
33
34
 
class A
{
public:
    A(int a);
 
private:
    int m_a;
};
 
A::A(int a): m_a(a)
{
 
}
 
class B
{
public:
    B();
    ~B();
 
private:
    std::vector<A> *m_essai;
};
 
B::B():m_essai(new std::vector<A>)
{
 
}
 
B::~B()
{
   delete m_essai;
}

merci pour vos réponses
oui Bousk j'ai fait des grosses erreurs dans le code que j'ai tapé (Désoler)
la class A est beaucoup plus grande que ce que j'ai donné comme exemple
mais j'ai voulu simplifier

En utilisant des pointeurs, je pense être dans la bonne direction. Mon but est de ne pas dupliquer des instances d'objets pour ne pas alourdir le programme
en d'utilisant un tableau de pointeurs pour accéder aux donnés encapsulées dans la classe A par exemple.
mais quand j'ai voulu taper mon destructeur en tapant

Code :

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

1
2
3
4
5
 
B::~B()
{
delete m_essai;
}

ça a planté..
(un unique_prt me semblle plus simple en effet)
voilà par contre , ma syntaxe pour mon tableau de pointeur

Code :

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

1
2
 
std::vector<A> *m_essai;

ou

Code :

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

1
2
 
std::vector<A*> m_essai;

je ne saisi pas bien la différence et le cour de C++ que j'ai récupéré
est un peu vague sur ce point.
si vous avez un petit conseil..ce serait sympa
C'est mieux comme ça 😉

[Matt_Houston]

std::<vecto*> m_essai; wat ? C'est quoi ça, une faute de frappe ?

[Médinoc]

Nous sommes en 2017: Si ton vecteur doit "posséder" les objets pointés, alors autant faire un std::vector<std::unique_ptr<a>> plutôt qu'un vecteur de pointeurs nus.

[petaf]

OK j'ai bien compris
merci pour vos conseilles
bon weekend a tous