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.