Discussion :

[Troopers]

Bonjour,

J'aimerai créer une classe abstraite A et deux classe dérivante AA et AB
Je veux que la classe A contienne une méthode static pour la création des objets de type AA et AB et une méthode virtuel implémenté dans AA et AB

J'ai donc le code suivant

Code :

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

1
2
3
4
5
6
7
8
9
class A
{
public:
	static A Create(BOOL bIsB = false);
	~A(void);
protected:
	A(void);
	virtual void Run()=0;
};

Pour AA et AB

Code :

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

1
2
3
4
5
6
7
8
9
 
class AA :
	public A
{
public:
	AA(void);
	~AA(void);
	virtual void Run();
};

Pour la méthode Create j'ai:

Code :

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

1
2
3
4
5
6
7
A A::Create(BOOL bIsB)
{
	if(bIsB)
		return AB();
	else
		return AA();
}

Et à la compilation j'ai une erreur : error C2259: 'A'*: impossible d'instancier une classe abstraite


J'ai loupé quelque chose???

Merci pour vos lumières

[bertry]

Salut,

J'ai loupé quelque chose???

Oui en effet! La classe A est une classe abstraite (c'est même toi qui le dit ) alors

Code :

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

1
2
3
static A Create(BOOL bIsB = false);
...
A A::Create(BOOL bIsB)

n'est pas correct, car 'Create' renvoie un objet de la classe A : Comme A est une classe abstraite, on ne peut pas créer d'objets de cette classe. C'est d’ailleurs ce que dit ton message d'erreur.

[Troopers]

Oui mais je crée pas réellement un objet de type A, je crée un objet de type AB ou AA et je renvoie un objet de type A.

En c#, cela est possible. Masi en C++ je dois apparemment renvoyé un pointeur de A :

Code :

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

1
2
3
4
5
6
7
8
 
A* A::Create(BOOL bIsB)
{
	if(bIsB)
		return new AB();
	else
		return new AA();
}

[bertry]

Code :

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

je crée un objet de type AB ou AA et je renvoie un objet de type A.

Non, non et non on ne peut pas!!!

Les AA et les AB tu en instancie tant que tu veux, mais pour les A ce n'est pas possible, ou sinon, il faut que A ne soit pas abstraite

De plus, si A n'était pas abstraite, tu pourrais être confronté au problème de 'slicing' développé ici

[Troopers]

Les AA et les AB tu en instancie tant que tu veux, mais pour les A ce n'est pas possible

Argf, justement je n'instancie pas de A: tu as vu ça où???

[koala01]

Salut,

Argf, justement je n'instancie pas de A: tu as vu ça où???

Mais si, justement...

Comme tu ne renvoie ni un pointeur ni une référence, ce que tu renvoie est... une instance de type A (et non de type AA ou de type AB), ce qui est strictement interdit...

Pour que cela fonctionne, et pour profiter du polymorphisme au passage, tu dois renvoyer une référence ou un pointeur sur un objet "passant pour être" de type A.

Tu dois, en outre, faire attention au fait que l'objet référencé ne soit pas détruit au moment de sortir de la fonction, si tu décide de renvoyer une référence.

La signature de ta fonction doit donc être de l'ordre de

Code :

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

1
2
3
4
5
class A
{
public:
	static A& Create(BOOL bIsB = false);
};

ou de

Code :

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

1
2
3
4
5
class A
{
public:
	static A Create(BOOL bIsB = false);
};

et son implémentation proche de

Code :

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

1
2
3
4
5
6
7
A* A::Create(BOOL bIsB)
{
	if(bIsB)
		return new AB();
	else
		return new AA();
}

ou, si cela a un sens, t'orienter vers quelque chose de proche de

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
class A
{
public:
    static A& Create(BOOL bIsB = false)
    {
        if(! m_member)
        {
            if(bIsB)
                m_member = new AB();
            else
               m_member = new AA();
            }
        return *m_member;
    }
    private:
    A* m_member;
};