vector d'objets ou vector de pointeurs sur objets

**manitor** · 15/02/2012, 19h12

Bonjour,

En résolvant un bug m'est venue cette question: j'ai deux classes A et B. Dans les membres de B, il y a un container type vector d'objets A...
Vaut-il mieux que B soit un vecteur d'objets A ou un vector de pointeurs sur des objets A?
Je précise aussi que la classe A est éventuellement une classe assez "complexe" et possédant un membre type pointeur sur une troisième classe...

**Ekleog** · 15/02/2012, 19h39

Si le polymorphisme n'est pas utilisé, et que les objets de la classe A ne sont pas présents ailleurs que dans le vector en mémoire, alors le vector d'objets est à préférer.
Sinon, un vector de smart pointers (shared_ptr, unique_ptr, [auto_ptr si tu es au c++98 sans boost]), ... est à envisager, selon les cas.

**manitor** · 15/02/2012, 20h35

ok, et en termes compréhensibles (si possible) pour le débutant que je suis, pourquoi cette préférence pour les vector d'objets (plutôt que vector sur pointeurs)?

**Ekleog** · 15/02/2012, 21h04

Parce qu'un vector de pointers n'a pas un destructeur qui fonctionne.

**manitor** · 15/02/2012, 21h16

mais si on construit le destructeur de la classe B de telle sorte qu'il parcourt le vector et détruise les pointeurs sur A:

Code :

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

1
2
3
4
B::~B(){
for(std::vector<A*>::iterator it=m_VectorA.begin(); it!=m_VectorA.end();it++) 
delete (*it);
}

(m_VectorA est le membre de B étant le vector de pointeurs sur A). Cela revient-il au même alors?

**3DArchi** · 16/02/2012, 07h22

Bonjour,

La gestion des pointeurs est affaire délicate surtout si tu es débutant. Je te conseille vivement de ne plus utiliser de pointeurs nus dans ton code pour pouvoir te concentrer sur l'essentiel de ton projet. Et quand tu seras moins débutant, ... et ben tu sauras pourquoi tu ne veux pas utiliser de pointeur nus de toute façon

Quelques lectures :
Présentation des pointeurs intelligents en C++ par Loïc Joly
Gérer ses ressources de manière robuste en C++ par Aurélien Regat-Barrel

Donc ton code peut ressembler à

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
class C;
 
class B
{
 
   // ...
private:
   std::unique_ptr<C> ptr_to_c;
};
 
class A
{
   // ...
private:
   std::vector<std::unique_ptr<B>> vec_ptr_b;
};