Discussion :

[zenux]

Bonjour,

J'ai cette classe:

Code :

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

1
2
3
template <class T> class Singleton{
  void destroy();
};

J'ai aussi une map qui contiens toutes mes instances de singletons:

Code :

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

map<string, void *> singletons;

J'aimerais parcourir ma map et appeler la methode destroy() pour tous les singletons, comment faire ?

Code :

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

1
2
3
4
5
for(map<string, void *>::iterator it = singletons.begin(); it!=singletons.end(); ++it)
{
  Singleton *singleton = dynamic_cast<Singleton *>((*it).second); //ne fonctionne pas pcq Singleton n'est pas typé !
  singleton->destroy();
}

Merci d'avance

[gbdivers]

Bonjour

En ajoutant un classe parente ?

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
class SingletonInterface
{
   virtual void destroy();
};
 
template <class T> class Singleton : public SingletonInterface
{
   virtual void destroy();
};
 
{
   map<string, SingletonInterface*> singletons;
 
   for(map<string, SingletonInterface*>::iterator it = singletons.begin(); it != singletons.end(); ++it)
   {
      it->second()->destroy();
   }
}

Ca devrait fonctionner.

[zenux]

Merci pour ta réponse.
Oui en effet ta solution fonctionne surement, j'y avais déjà pensé.
Mais n'existe t-il pas une autre solution sans class supplémentaire ?

[Goten]

Moi j'ai une question, comment tu peux avoir plusieurs instances de singleton et mieux, comment tu peux avoir une méthode non statiques destroy ?

[gbdivers]

Ce n'est pas impossible d'avoir plusieurs singleton. Ici, il a un singleton pour chaque spécialisation de sa classe template.

Pour le static, c'est nécessaire pour une fonction destroy() qui détruit le singleton et affecte NULL au pointeur (pour supprimer le singleton avant le fin du programme).

Si le but est de supprimer les singletons à la fin du programme, pas besoin de destroy(). On peut simplement les détruire avec un delete :

Code :

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

1
2
3
4
for(map<string, Singleton*>::iterator it = singletons.begin(); it != singletons.end(); ++it)
{
   delete it->second();
}

Je vois pas comment créer une fonction destroy() utilisant dynamic_cast. La classe Singleton n'existe pas mais uniquement les classes Singleton<T>.

[Goten]

Ce n'est pas impossible d'avoir plusieurs singleton. Ici, il a un singleton pour chaque spécialisation de sa classe template.

Pour le static, c'est nécessaire pour une fonction destroy() qui détruit le singleton et affecte NULL au pointeur (pour supprimer le singleton avant le fin du programme).

Si le but est de supprimer les singletons à la fin du programme, pas besoin de destroy(). On peut simplement les détruire avec un delete :

Code :

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

1
2
3
4
for(map<string, Singleton*>::iterator it = singletons.begin(); it != singletons.end(); ++it)
{
   delete it->second();
}

Je vois pas comment créer une fonction destroy() utilisant dynamic_cast. La classe Singleton n'existe pas mais uniquement les classes Singleton<T>.

J'ai pas dis que c'était impossible, je mettais juste un doigt sur quelque chose qui peut *à de fortes chances d'* être une erreur de conception (voir l'article sur le blog d'emmanuel Deloget : Etes vous atteint de singletonite?).
Quand à la fonction non statique, alors autant laisser faire le destructeur (comme tu l'as dis avec delete)...