Discussion :

[dragonjoker59]

Bonjoir !

Je rencontre un problème avec mes spécialisations de templates, sous Gcc mais pas sous Visual Studio.
Comme un bout de code est plus parlant...

Dans Factory.hpp j'ai ceci :

Code :

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

1
2
3
4
5
6
7
8
9
10
   template <class Obj, class Key=String>
   class Factory
   {
   private:
      typedef shared_ptr<Obj> obj_ptr;
      typedef std::map<Key, obj_ptr> obj_map;
      static obj_map m_mapRegistered;
   public:
      // des fonctions qui vont bien
   };

Dans Light.cpp j'ai ceci :

Code :

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

Factory<LightCategory, eLIGHT_TYPE>::obj_map Factory<LightCategory, eLIGHT_TYPE>::m_mapRegistered;

Si j'essaie de compiler comme ça il me sort comme erreur:

Code :

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

error: specializing member 'Factory<LightCategory, eLIGHT_TYPE>::m_mapRegistered' requires 'template<>' syntax

Je me dis "Ok, on va faire ce qu'il demande" :

Code :

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

template <> Factory<LightCategory, eLIGHT_TYPE>::obj_map Factory<LightCategory, eLIGHT_TYPE>::m_mapRegistered;

Et là il me met l'erreur de link suivante :

Code :

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

référence indéfinie vers Factory<LightCategory, eLIGHT_TYPE>::m_mapRegistered

???
Qu'est-ce que j'ai fait de mal ? (au cas où la question vous viendrait à l'esprit, le fichier Light.cpp est bel et bien inclus dans le projet).
Je reprécise que ces erreurs arrivent uniquement sous gcc et pas sous VisualStudio.

[BigNic]

vue ta 1ere déclaration c'est classe qui est template pas les membre de ta classe, donc tu ne peux pas spécialiser les méthodes seulement tu dois spécialiser ta classe entièrement ex:

Code :

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

1
2
3
4
5
 template<>
class Factory<LightCategory, eLIGHT_TYPE>
{
   // des fonctions qui vont bien
};

et tes fonctions pourront manipuler m_mapRegistered comme s'il était un map de LightCategory indéxés par des eLIGHT_TYPE

[3DArchi]

Salut,
Faut toujours écouter son compilateur :

Code :

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

1
2
template <class Obj, class Key>
typename Factory<Obj, Key>::obj_map Factory<Obj, Key>::m_mapRegistered;

donc :

Code :

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

1
2
3
 
template<>
Factory<LightCategory, eLIGHT_TYPE>::obj_map Factory<LightCategory, eLIGHT_TYPE>::m_mapRegistered;

[dragonjoker59]

@3DArchi : Ben c'est ce que j'ai mis mais ça fait une erreur de link.

@BigNic : Y a vraiment pas d'autre moyen ? Parce que ça m'enlève tout l'intérêt de cette façon de faire ...

[3DArchi]

Salut,

@3DArchi : Ben c'est ce que j'ai mis mais ça fai une erreur de link.

Excuses moi, j'ai lu et répondu un peu trop vite...

Le problème est que lors d'un instanciation (différent d'une spécialisation) d'une classe template, les membres statiques ne sont pas implicitement instanciés. Il faut donc leur donner une définition explicitement :

Code factory.h :

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

1
2
3
4
5
6
7
8
9
10
11
12
13
14
15
 
   template <class Obj, class Key=String>
   class Factory
   {
   private:
      typedef shared_ptr<Obj> obj_ptr;
      typedef std::map<Key, obj_ptr> obj_map;
      static obj_map m_mapRegistered;
   public:
      // des fonctions qui vont bien
   };
 
template <class Obj, class Key>
typename Factory<Obj, Key>::obj_map 
     Factory<Obj, Key>::m_mapRegistered;

[dragonjoker59]

Merci, je testerai ça lundi !

[dragonjoker59]

Bon, après essai, j'ai toujours la même erreur ...

@BigNic : en fait je ne veux pas spécialiser les méthodes, elles font le travail générique que je veux (enregistrement, création de nouvelle entité) et je n'ai pas besoin de les spécialiser.

EDIT :
Du coup je vais enlever la partie statique de Factory, de toute façon les static c'est mal...

[3DArchi]

Bon, après essai, j'ai toujours la même erreur

Euh, tu peux montrer ton code. Car j'ai testé et je n'ai plus d'erreur de link (alors que je l'avais bien reproduit avec ton code précédent).

[dragonjoker59]

Factory.hpp :

Code :

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

1
2
3
4
5
6
7
8
9
10
11
12
   template <class Obj, class Key=String>
   class Factory
   {
   private:
      typedef shared_ptr<Obj> obj_ptr;
      typedef std::map<Key, obj_ptr> obj_map;
      static obj_map m_mapRegistered;
   public:
      // des fonctions qui vont bien
   };
 
   template <class Obj, class Key> typename Factory<Obj, Key>::obj_map  Factory<Obj, Key>::m_mapRegistered;

Rien de changé ailleurs.

En tout cas, j'ai viré les variables statiques, qui de toute façon me poseront problème dès que j'appellerai mes factories dans des Dll (j'ai pas mal de plugins dans mon moteur et ces plugins sont dll / shared lib)

[koala01]

Salut,

Factory.hpp :

Code :

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

1
2
3
4
5
6
7
8
9
10
11
12
   template <class Obj, class Key=String>
   class Factory
   {
   private:
      typedef shared_ptr<Obj> obj_ptr;
      typedef std::map<Key, obj_ptr> obj_map;
      static obj_map m_mapRegistered;
   public:
      // des fonctions qui vont bien
   };
 
   template <class Obj, class Key> typename Factory<Obj, Key>::obj_map  Factory<Obj, Key>::m_mapRegistered;

Rien de changé ailleurs.

En tout cas, j'ai viré les variables statiques, qui de toute façon me poseront problème dès que j'appellerai mes factories dans des Dll (j'ai pas mal de plugins dans mon moteur et ces plugins sont dll / shared lib)

Heuuu... Est ce une erreur de copier / coller, ou bien tu ne considères pas que m_mapRegistered est une variable statique

[dragonjoker59]

Je ne comprends pas ce que tu veux dire.

[Emmanuel Deloget]

Je ne comprends pas ce que tu veux dire.

template <class Obj, class Key=String>
class Factory
{
private:
typedef shared_ptr<Obj> obj_ptr;
typedef std::map<Key, obj_ptr> obj_map;
static obj_map m_mapRegistered;
public:
// des fonctions qui vont bien
};

template <class Obj, class Key> typename Factory<Obj, Key>::obj_map Factory<Obj, Key>::m_mapRegistered;

m_mapRegistered est une statique de classe, d'où la remarque de koala01.

[dragonjoker59]

Oui, et je ne comprends toujours pas ce qui ne va pas, à part le fait d'une variable statique membre d'une classe template que je ne sais pas comment implémenter (la méthode que j'avais utilisée ne fonctionnant pas)

EDIT :
J'ai testé dans un projet vide et je n'ai effectivement plus l'erreur de link.
Donc ça marche, mais comme j'ai passé m_mapRegistered en non statique, ça ne m'est plus utile.
Je passe en résolu.

[Emmanuel Deloget]

Oui, et je ne comprends toujours pas ce qui ne va pas

Dans ton message, tu dis que tu as viré les statiques de classes, hors celle-ci est présente. D'où la réponse de koala01.

Après, quand à savoir si ça va ou pas...

Si tu instancie la classe dans un programme et dans une dll, tu auras deux versions de la même variable.