Discussion :

[flaharrague]

Bonjour,
J'ai un problème pour faire un agrégation d'un héritage d'une classe abstraite... je la refais en plus imagé:
Je possède une classe A qui a une relation d'agrégation avec une classe B, abstraite.

J'ai un certain nombre de classes filles (C, D, E F...) héritées de B. Maintenant, je voudrais pouvoir, dans A, créer une methode du style AddClasseB( B*,x,y) qui me permettrais d'ajouter n'importe quel objet C, D, E.. Mais je vois pas comment faire...

Code :

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

1
2
3
4
5
6
7
8
9
10
class A{
 
public:
 
void AddClassB(B* Obj, int x, int y);
 
private :
 
B** tabB;
};

J'ai redéfini l'opérateur égal de la classe Mère et des classes filles, mais ça me sert à rien...

Est-ce que c'est possible ce que je veux faire??
Merci d'avance pour vos réponses,

[ep31]

Tout d'abord, l'objet B dans ta classe A n'est pas une relation d'agrégation mais de composition car à la destruction de ton instance de A, l'objet B ne sera pas détruit car c'est un pointeur.
Ensuite, il me semble pas que tu puisses stocker dans tabB un tableau d'éléments différents hérités de B car les instances de tes classes C,D, E, ... filles de B n'auront pas forcément la même taille. Donc naviguer avec un pointeur sur des blocs de taille différente va se révéler très dur.

[Caine]

Tu veux une matrice de classe B, c'est bien ça? Et utiliser le polymorphisme, tu dois alors déclarer TabB comme suit:

Code :

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

1
2
 
B*** TabB

Le polymorphisme implique d'avoir un pointeur sur la classe de base.
Si X représente les lignes et Y les colonnes:

Code :

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

1
2
3
4
 
void AddClassB(B* Obj, int x, int y) {
  TabB[y][x] = Obj;
}

[flaharrague]

Pardon j'ai mal expliqué ma fonction, je veux juste ajouter un objet, pas une matrice de classe, donc c'est bien:

Code :

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

 B** tabB

çà j'en suis sûr, mais effectivement le problème est dans :

Code :

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

1
2
3
4
void AddClassB(B* Obj) { 
int i;  
TabB[i] = Obj; 
}

Ca fait appel à l'opérateur égal de B, et moi je voudrais pouvoir affecter dans le tableau un objet C, D, E...hérité de B! Et je pense pas que çe soit possible, Dites moi que si!!

[Laurent Gomila]

Ca fait appel à l'opérateur égal de B, et moi je voudrais pouvoir affecter dans le tableau un objet C, D, E...hérité de B! Et je pense pas que çe soit possible, Dites moi que si!!

Tu n'affectes pas un B, mais un B*. Donc pas d'appel à un quelconque opérateur =, ce qui est tout à fait logique avec une telle sémantique. Donc qu'est-ce qui ne marche pas ? Ton code est correct pour l'instant. (pense quand même à std::vector et aux pointeurs intelligents -- voir la FAQ C++).

Tout d'abord, l'objet B dans ta classe A n'est pas une relation d'agrégation mais de composition car à la destruction de ton instance de A, l'objet B ne sera pas détruit car c'est un pointeur.

Ca c'est un détail d'implémentation, rien n'empêche de lier la durée de vie d'un objet à son contenant, qu'il soit alloué dynamiquement ou non. C'était juste pour pinailler.

[xxiemeciel]

Salut,

Pour repondre a ta question ce que tu veux faire est possible. Maintenant si ca ne marche pas c que tu as une erreure quelquepart.
Comment as tu allouer la memoire de tabB ?


XXiemeciel

[flaharrague]

Tu n'affectes pas un B, mais un B*. Donc pas d'appel à un quelconque opérateur =, ce qui est tout à fait logique avec une telle sémantique. Donc qu'est-ce qui ne marche pas ? Ton code est correct pour l'instant.

Ok, merci pour la réponse, je pense que j'ai des gros problèmes d'initialisation et de destruction c'est ce qui fait planter...

Comment as tu allouer la memoire de tabB ?

C'est ça mon problème, j'alloue dans le constructeur de la classe A:

Code :

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

1
2
 
_tabB= new B*[_nbPattern];

Ensuite je voudrais instancier chaque B*, ce qui est impossible avec une classe abstraite, donc je pense que je vais laisser tomber mes méthodes virtuelles pures.. et faire:

Code :

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

1
2
3
4
5
6
 
_tabB= new B*[_nbPattern];
 for (int i =0 ; i<_nbB; i++)
 {
	_tabB[i]=new B();
 }

Autre petit problème, quand on fait de l'héritage, que se passe t'il lorsqu'on appelle le destructeur d'une classe fille, ça fait d'abord appel au destructeur de la classe mère? mes problèmes pourraient aussi venir de la...
En tout cas merci pour vos réponses!

[Fiquet]

Autre petit problème, quand on fait de l'héritage, que se passe t'il lorsqu'on appelle le destructeur d'une classe fille, ça fait d'abord appel au destructeur de la classe mère? mes problèmes pourraient aussi venir de la...
En tout cas merci pour vos réponses!

http://c.developpez.com/faq/cpp/?pag...explicite_base

[bigquick]

Tout d'abord, l'objet B dans ta classe A n'est pas une relation d'agrégation mais de composition car à la destruction de ton instance de A, l'objet B ne sera pas détruit car c'est un pointeur.

Ca c'est un détail d'implémentation, rien n'empêche de lier la durée de vie d'un objet à son contenant, qu'il soit alloué dynamiquement ou non. C'était juste pour pinailler.

Juste pour pinailler un peu plus .... Il me semble que flaharrague avait raison ... si les durées de vie ne sont pas liées, c'est de l'aggrégation. La composition reflète un embarquement : le contenu est créé et détruit avec le contenant.

C'est tout
(je crois que tout a déjà été dit, mais pense vraiment à utiliser la STL, ça va te simplifier la vie... enfin, le code surtout )

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
#include <vector>
 
class Test
{
public:
 
	void AddClassB( B* Obj )
	{
		m_tabB.push_back(Obj);
	}
 
private :
 
	std::vector<B*> m_tabB;
};
 
 
Test t;
t.AddClassB( new C );
t.AddClassB( new D );
...

[flaharrague]

Ok merci à tous pour vos réponses, je vais tenter avec la STL.

[xxiemeciel]

Salut,

Je pense qu'une std::vector resoudrais tes problemes.

Code :

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

1
2
 
std::vector<B*>* vec = new std::vector<B*>;

XXiemeciel

[Jan Rendek]

Autant éviter l'allocation dynamique du vecteur.