Construction d'un element dynamique dans une classe.

**deubelte** · 22/03/2011, 16h53

Bonjour

Dans le cas d'un dev professionnel, je dois r�aliser la conception suivante.
Nous avons une classe A de base, et une classe d�riv�e B.

Nous avons enfin une classe C, qui poss�de un attribut qui peut etre de type A ou de type B selon la valeur d'un param�tre.

Alors voila ce que j'ai fait. Toutes les critiques sont pour moi bonnes a prendre.

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
35
36
37
38
39
40
41
42
43
44
45
class A{
public:
	A(int u):a(u){};
	A(){};
private:
int a;
};
 
class B:public A{
public:
	B(int x1,char* x2):A(x1),x(x2){};
	char* getunChar(){
		return x;
	}
private:
	char* x;
};
 
class C{
public:
	C(int i,char*);
	char* getChar(){
			return	((B*)elem)->getunChar();
	}
private:
A* elem;
};
 
C::C(int u,char* chaine){
 
	if(u==1)
	{
	   elem =new A(1);
	}
	else
	{
	  elem = new  B(4,chaine);
	}
}
 
int main(){
	char *x="ee";
	C unobjet(2,x);
	std::cout<<unobjet.getChar();
}

La classe B h�rite bien de A. La classe C poss�de en attribut un pointeur A. Mais ce pointeur peut avoir pour type dynamique le type B. La valeur du param�tre est pass� dans le constructeur. Il s'agit dans ce cas u. Si u vaut 1 alors elem est de type A, sinon il est de type B. D'un point de vue th�orique, cela me semble bien.

Il faudrat bien sur dans le destructeur faire un delete de elem, ce que j'ai fait ici:

Code :

S�lectionner tout - Visualiser dans une fen�tre � part

1
2
C::~C()
{	delete elem;  }

J'ai enfin une derni�re question. Pourquoi ne faut il pas faire ceci:

Code :

S�lectionner tout - Visualiser dans une fen�tre � part

1
2
3
4
5
6
7
8
9
10
C::C(int u,char* chaine){	
	if(u==1)
	{
	   elem =&A(1);
	}
	else
	{
	  elem = &B(4,chaine);
	}
}

Je vous rermercie pour votre aide.

**JolyLoic** · 22/03/2011, 17h02

Il manque le destructeur virtuel dans A : http://cpp.developpez.com/faq/cpp/?p...UCTEUR_virtuel

elem = &B(4,chaine);
Cr�e un objet temporaire de type B, non nomm�, avec comme param�tres du constructeur 4 et chaine. On prend ensuite l'adresse de cet objet que l'on stocke dans elem. Puis cet objet temporaire est d�truit, et donc elem pointe sur de la m�moire qui a �t� d�sallou�e...

**deubelte** · 22/03/2011, 17h28

ok merci pour ta r�ponse.
En tout cas, il n'y a rien de dangereux ou de choquant a faire une telle impl�mentation?

**N i h i l** · 22/03/2011, 17h40

Envoy� par deubelte

En tout cas, il n'y a rien de dangereux ou de choquant a faire une telle impl�mentation?

Le risque c'est de cr�er un C qui dont l'elem pointe sur un A et d'invoquer get. Remplace ton cast C par un dynamic cast et teste la valeur de retour.

**deubelte** · 22/03/2011, 17h44

Le risque c'est de cr�er un C qui dont l'elem pointe sur un A et d'invoquer get. Remplace ton cast C par un dynamic cast et teste la valeur de retour.

oui, ok d'une mani�re g�n�rale, j'ai tendance a privil�gier les static cast sur les dynamiques, pour des raisons de perf.

**deubelte** · 22/03/2011, 19h15

J'ai un autre probl�me. La classe que j'ai faite est un peu simpliste.
En r�alit�, dans la classe C nous avons quelque chose de ce style:
(la classe A et B reste inchang�e).

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
template<typename T>
class MaClasse{
public:
	MaClasse<T>(double x1,double x2,double x3):a(x1),b(x2),c(x3){};
private:
	double a; 
	double c;
	double b;
};
 
class C{
public:
	C(int i,char*);
	char* getChar(){
			return	((B*)elem)->getunChar();
	}
private:
MaClasse<A>* elem;
};
 
C::C(int u,char* chaine){
 
	if(u==1)
	{
	   elem = new MaClasse<A>(1,2,3);
	}
	else
	{
	  elem = new MaClasse<B>(1);
	}
}

cela ne fonctionne pas car le C++ voit deux objets compl�tement diff�rents, l'un de type MaClasse<A>*, l'autre de type MaClasse<B>* qu'il ne peut pas convertir bien que A soit la classe m�re de B.

Alors comment puis-je faire, pour avoir ceci:

Code :

S�lectionner tout - Visualiser dans une fen�tre � part

1
2
3
4
5
6
7
8
	if(u==1)
	{
//Instanciation d'un objet de type MaClasse<A>	, dans mon code, cela marche parfaitement. 
	}
	else
	{
//Instanciation d'un objet de type MaClasse<B>. C'est là que ca plante !!
	}

Je ne vois pas trop comment contourner le probl�me du template qui met en �chec tout ce que j'ai fait avant. Peut etre une classe-interface sup�rieure � A.

Je vous remercie.