Discussion :

[Wilgard]

Bonjour,

Il y a une notion que je ne comprends pas sur les itérators.
J'ai une classe A ainsi qu'une class B qui hérite de A et une class C qui hérite aussi de A.

Je crée des Objets B (donc via new B(...)) que j'ajoute dans un vector de A.

typedef std::vector<A> a_vector; via la commande push_back. Jusque là tout est normal.

Par contre ce que je ne comprends pas c'est que si je fais :

Code :

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

1
2
3
4
for (unsigned int i = 0; i < a_vect->size(); i++)
{
             A a = a_vect[i];
}

Il me dit qu'il ne peut pas convertir a_vector en A.
Là je ne comprends pas pourquoi il y arrive pas ....


Donc du coup je passe par les itérateurs et là ...

Code :

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

1
2
3
4
5
for (a_vector::iterator it = a_vect->begin() ; it != a_vect->end(); it++)
{
             A a = *it; // ca, ca marche sans probleme
             B* b = dynamic_cast<B*> (&a); // et la ca me donne toujours null et ca je ne comprends pas vu que je n'ai jamais fait de new A mais uniquement des new B ....
}

Dans le débugger le _vptr vaut bien B avant l'ajout dans le vector et mais il faut A en tant qu'itérateur. C'est comme ca le fait que je déclarait un itérateur de type A faisait que seulement la partie A de mes objets B serait visible.

Avez-vous une idée de ce problème d'héritage ?

Merci

Cordialement,

[Bousk]

Bonjour,

c'est un vector de A ou de A* ?
Le typedef indique vector<A>, ce qui interdit d'y mettre des pointeurs, et encore moins d'utiliser du polymorphisme.

Code :

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

a_vect->size();

Comment est déclarer a_vect ? C'est un std::vector* ? y'a-t-il un réel intérêt ?

Code :

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

A a = a_vect[i];

a est une copie de a_vect[i], et a ne pourra jamais être transtypé vers B, il n'a aucune information sur B, ni même que B existe.

De plus les dynamic_cast sont généralement à éviter et sont souvent révélateurs d'erreur d'interface ou architecture.
B est-il vraiment un A ?

[cob59]

Tu viens du monde Java Wilgard, non ?
Comme le suggère Bousk, tu fais à plusieurs reprises des copies là où tu pensais faire des références...
Je ne sais pas ce que désignent ces classes, mais il me semble que A ne devrait même pas être instanciable, donc avec ses constructeurs en protected.

[white_tentacle]

Comment as-tu déclaré a_vect ?

Code :

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

1
2
3
4
5
6
7
8
9
 
typedef std::vector<A> vector_a;
 
int f()
{
  vector_a monVect;
   .... // remplir monVect
  A unA = monVect[0]; // OK
}

doit fonctionner.

Sinon, il y a un gros problème dans ton code : si ton vecteur contient des A, il ne faut pas y mettre des B dedans : c’est trop gros, ça ne tiendra pas. Et c’est ce qu’il se passe : tes B sont convertis et copiés en A quand tu les stocke dans ton vecteur. Quand tu les récupères, ce sont des A, ce qui explique que tu ne puisses plus les convertir en B.

Si tu veux stocker tes B dans un vecteur de A, il faut stocker en réalité des pointeurs vers des B dans un vecteur de pointeurs de A. Et plutôt que des pointeurs nus, je te conseilles d’utiliser des shared_ptr.

[Wilgard]

Je vous remercie tous les 3 pour vos réponses rapides et utiles.

En effet cob59 je viens du monde Java , c'est sûrement pour ça que je pose des questions sur les pointeurs.

Effectivement ma classe A est abstraite alors que B et C ne le sont pas d'où le fait que je veux mettre des B ou bien des C dans un vecteur de A.

J'avais oublié le fait que a_vect[i] était une copie et non un récupération de référence... Merci Bousk pour cette info.

Enfin merci white_tentacle car en effet le fait de passer par des pointeurs de B dans un vecteur de pointeurs de A fait que ca marche.

Merci à tout les 3.