Discussion :

[nicolo011]

Bonjour,

Je rencontre un gros problème d'utilisation de forward déclarations et de namespaces (avec utilisation de singleton), et si quelqu'un pouvait éclairer ma lanterne, ça serait super (à savoir: je suis expérimenté en C++, mais là, je sèche bêtement)

Voici mon problème:
J'ai 2 fichiers Animal.hpp et Animal.cpp.
Dans Animal.hpp:

Code :

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

1
2
3
4
5
6
7
8
9
10
11
 
namespace LeMondeDesAnimaux
{
class Animal
{
public:
    Animal();
    virtual ~Animal();
    virtual void Manger()=0;    // virtuelle pure
};
};

Dans le fichier Animal.cpp, les implémentations de Animaux et de classes classes héritées (déclarées ici volontairement, car pas besoin de mettre ces classes dans l'interface de ma lib):

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
namespace LeMondeDesAnimaux
{
class Lion: public Animal
{
public :
    virtual ~Lion();
    virtual void Manger();
    static Animal* GetInstance();  // singleton
private:
    Lion();
};
};

using namespace LeMondeDesAnimaux;

Animal::Animal()
{}
Animal::~Animal()
{}

Lion::Lion() {}
Lion::~Lion() {}
void Lion::Manger() {}
Animal* Lion::GetInstance()
{
    static Lion monInstance;
    return &monInstance;
}

Rien de bien méchant jusque là (j'ai allégé en virant les #ifndef ANIMAL_HPP dans mon code donné ici pour ne pas vous alourdir la lecture).


Viennent ensuite les fichiers Lieu.hpp et Lieu.cpp:
Dans Lieu.hpp:

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
#include Animal.hpp
 
namespace LeMondeDesAnimaux
{
class Lieu
{
public:
    Lieu();
    virtual ~Lieu();
    virtual void Ouvrir() = 0;   // virtuel pure!
 
protected:
    Animal* m_pCages[3];   // j'ai 3cages, avec des animaux dans chacun, qui sont donnés à la construction
};
};

et Lieu.cpp:

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
#include "Lieu.hpp"

namespace LeMondeDesAnimaux
{
class Zoo : public Lieu
{
public:
    Zoo();
    virtual ~Zoo();
    virtual void Ouvrir();
};
};  // fin namespace

using namespace LeMondeDesAnimaux;
Lieu::Lieu()
{
    m_pCages[0]=0;
    m_pCages[1]=0;
    m_pCages[2]=0;
}
Lieu::~Lieu()
{
}

Zoo::Zoo()
{
    m_pCages[0]=Lion::GetInstance();
    m_pCages[1]=0;
    m_pCages[2]=0;
}
Zoo::~Zoo()
{
}
void Zoo::Ouvrir()
{}

Comme vous le voyez, je fais référence, dans un fichier cpp (Lieu.cpp), à une class dont la définition est donnée dans un autre fichier cpp (Animal.cpp).

Quand je compile, j'obtiens un message d'erreur sur Lion::GetInstance() :
"Lion : must be a class or namespace"

OK, je fais de la forward déclaration... donc avant la ligne de constructeur Zoo::Zoo(), j'ajoute:

Code :

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

class Lion;

Nouvelle erreur sur Lion::GetInstance() :
"Incomplete type is not allowed"

OK... je me dis que ma forward déclaration devrait spécifier le namespace, donc je la fais plus haut, avant le using namespace, en écrivant:

Code :

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

1
2
3
4
5
 
namespace LeMondeDesAnimaux
{
    class Lion;
};

Même erreur!!

Quelqu'un aurait-il une idée à la question, qui se résume donc à:
Comment, dans un fichier A.cpp, dans le namespace N, faire référence à une classe définie uniquement dans un autre fichier B.cpp et appartenant au même namespace N? Quelle forme doit prendre la forward déclaration et l'utilisation de la classe dans le fichier A.cpp?


Si vous avez une idée, ça me sauverait bien...

Merci d'avance pour vos réponses

Nicolas

[Jean-Marc.Bourguet]

Le probleme est que tu ne peux utiliser un membre de Lion que si tu as une definition de la classe Lion visible. Une simple declaration ne suffit pas.

[jblecanard]

Pour éclairer le propos de Jean-Marc, une forward déclaration ne te permet que d'utiliser des pointeurs vers la classe forward déclarée. Là tu essayes carrément d'accéder à une méthode statique !

Cela dit, tu n'as besoin de Lion::GetInstance() que dans l'implémentation et pas dans le header. Donc tu ne devrais pas avoir de problème : il suffit d'ajouter l'include correspondant. Or, tu as déclaré ces classes dans tes fichiers *.cpp. C'est ici que se situe le problème. Si tu ne veux pas exposer ces interfaces, tu n'as qu'à en créer plusieurs : un header public et un header privé que tu ne livres pas à l'utilisateur.