Discussion :

[mister3957]

Bonjour à tous,


Je connais bien l'utilité d'avoir un Singleton (pour les gestionnaires etc...).

En revanche, je tombe régulièrement sur deux implémentations.

L'une :

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
 
template<class T>
class CSingleton
{
public:
   T* GetInstance()
   {
       if(!m_pInstance)
          m_pInstance = new T();
       return m_pInstance;
   }
 
   void Release()
   {
       if(m_pInstance)
       {
           delete m_pInstance;
           m_pInstance = 0;
       }
   }
 
private:
   T* m_pInstance;
};

Et l'autre :

Code :

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

1
2
3
4
5
6
7
8
9
10
11
 
template<class T>
class CSingleton
{
public:
   T* GetInstance()
   {
       static T instance;
       return instance;
   }
};

Laquelle est préférable ?

La première différence que je vois est qu'on a pas à gérer la destruction dans la second (l'instance statique étant détruite à l'arrête du programme). L'inconvénient étant qu'on ne contrôle pas le moment de la destruction.



Merci

[Jan Rendek]

Je suis souvent partisan de la seconde version, pour sa simplicité généralement.

Elle présente par contre quelques inconvénients:
1) Elle n'est possible que si la construction du singleton n'est pas paramétrée.
2) Le singleton ne doit pas dépendre d'une autre ressource "static". Le compilateur n'offrant aucune garantie sur l'ordre d'instanciation des variables globales.

En dehors de ça, c'est kif-kif. Ton code est en revanche faux dans les 2 versions. La fonction membre GetInstance() et la variable membre m_instance doivent être déclarées statiques.

[mister3957]

Exacte oui, j'voulais juste mettre en valeur le pointeur ou la variable globale.


Merci

[Mat.M]

Exacte oui, j'voulais juste mettre en valeur le pointeur ou la variable globale.


Merci

Jan Redek a raison : le premier exemple de code donne ce n'est pas un singleton le second oui

[Luc Hermitte]

Les diverses implémentations possibles sont discutées in-extenso dans Modern C++ Design et dans Imperfect C++.

Et finalement, l'implémentation que je préfère est encore sous forme d'un fichier squelette qui produit ce genre de choses -- j'ai toujours besoin de fonctionner en environnement MT, j'ai régulièrement besoin de passer des arguments pour initialiser la variable globale, et la construction paresseuse s'est systématiquement avérée être un faux besoin.