Discussion :

[El Zeru]

Bonjour, j'ai le code suivant :

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
 
class A{
public:	
	virtual void Draw(){std::cout<<"lalala A\n";}
        A(){};
       ~A(){};
};
 
class B :public A{
public:	
	virtual void Draw(){std::cout<<"lalala B\n";}
        B(){};
        ~B(){};
};
 
 
int _tmain(int argc, _TCHAR* argv[])
{
	B *b=&B();
	b->Draw();
	return 0;
}

Je ne comprends pas pourquoi c'est le A::draw qui est appelé et non B::draw. Et pourquoi quand j'enlève le destructeur de la classe A, là ça marche...
Merci de m'aider !
a+

[Seelass]

Salut

D' après ce que j' ai vu tu ne fait pas par exemple d' allocation mémoire dans ta classe.je ne vois pas à quoi cela pourrait te servir?.

[JolyLoic]

Code :

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

1
2
 
	B *b=&B();

Il faut comprendre ce que fait cette ligne :
- B() : Tu crées un objet B temporaire
- &B() : Tu en prends l'adresse
- B *b = &B() : Tu stoke l'adresse dans un pointeur
- B *b = &B(); : A la fin de l'instruction, le temporaire est détruit. Donc faire b-> devient un comportement indéfini. Il se trouve que dans ton cas, selon la manière dont les destructeurs sonts définis, tu as l'impression de voir quelque chose ou autre chose, mais à la base, le programme aurait tout autant le droit de planter sans autre forme de procès. Ou, plus sournois, de donner l'impression de marcher, et de causer un bug plus tard, à un endroit qui n'a aucun rapport.

[El Zeru]

Merci de vos réponses. Dans mon programme j'ai une classe mère A et deux filles B et C. J'ai fait un container qui contient à la fois de B et des C et je veux faire un traitement automatisé du container quelques soient ses éléments. J'ai donc pensé faire un container de pointeurs sur A qui pointent vers des B ou des C, mais comme vu dans mon post, ça ne marche pas comme je le voudrais...
Que dois-je faire ?

Merci d'avance.
a+

[3DArchi]

Salut,
Tout comme dans l'exemple simplifié il y avait une erreur, montres nous comment tu définis puis construit ton conteneur et comment tu invoques la méthode.

[El Zeru]

J'ai donc la hierarchie suivante de classes :
Obj--->Objfich--->Obj3D
|--->Objprim

L'idée est de pouvoir stocker et utiliser indifféremment des Obj3D et des Objprim...Ici le constructeur permet d'initialiser une géométrie 3D à partir d'un objet. Objprim concerne les primitives de Glut.

En global :

Code :

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

1
2
 
std::vector<Obj *> Objs;

Puis dans une fonction :

Code :

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

1
2
3
4
 
     char * fich="stanbunny.mxa";
     Objs.push_back(&Obj3D(fich));
     Objs[0]->Draw();

Merci d'avance.

[3DArchi]

Code :

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

1
2
 
     Objs.push_back(&Obj3D(fich));

Exactement la même cause que t'a expliqué JolyLoic juste au-dessus.

[El Zeru]

Haha, oui j'ai mis des "new" ça marche beaucoup mieux ! Merci à tous !

[hiura]

Au cas où : as-tu pensé au delete ?

[3DArchi]

Attention à bien gérer tes pointeurs. Pour cela tu peux t'aider des pointeurs intelligent ... et faire une petite lecture du tutoriel Pointeurs intelligents de Loïc et consulter l'entrée de la FAQ.

[El Zeru]

Je regarde ça, merci.