Discussion :

[delire8]

Bonjour,

j'ai une classe CSingleton somme toute classique (c'est à dire avec une fonction GetInstance et une autre Kill). Dans mon programme, j'écrit par exemple:

CExcel obj;
Obj = CExcel::Get();


et à la fin, je ne dois surtout pas oublier un petit CExcel::Kill();

Oui mais voilà, je suis tête en l'air... Alors je me demandais s'il n'y a pas moyen, par hasard, de rajouter cela quelque part (dans un destructeur par exemple), de manière à ne plus à me préoccuper de cette ligne. Par exemple, à la fin du programme, l'objet "obj" (CExcel obj) est automatiquement libéré, et ~CExcel() est appelé. J'ai essayé d'ajouter Kill() dedans, mais c'est sans effet.

Bref, comment peut on rendre automatique cet appel ?

Merci

Cordialement

[raptor70]

Code :

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

1
2
CExcel obj;
Obj = CExcel::Get();

A vérifier ... mais je crois que tu fais une copie de ton objet singleton CExcel.. ce qui ne devrait pas arrivé si ton constructeur par copie est privé.

De plus, il n'y a pas de méthode générale pour détruire automatique un singleton seul. Il faut un système a part d'enregistrement de ton singleton et de destruction à la fin de l'application. Mais c'est un système à part du pattern singleton

[Alp]

Là tu cherches quelque chose proche du RAII.
Dans ton programme il n'y a pas une classe qui est instanciée au début et qui est détruite à la fin du programme ? Elle pourrait porter la responsabilité de ton Singleton et le détruire donc à sa propre destruction.

[loufoque]

Code :

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

1
2
3
4
5
static CExcel& CExcel::Get()
{
    static CExcel instance;
    return instance;
}

Et écrire

Code :

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

CExcel& obj = CExcel::Get();

Et non

Code :

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

1
2
CExcel obj;
obj = CExcel::Get();

ou

Code :

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

CExcel obj = CExcel::Get();

[raptor70]

Un singleton, normalement, c'est un pointeur static :

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
 
class CExcel
{
public :
    static CExcel & Get( void )
    {
        if( instance == NULL )
        {
            instance = new CExcel();
        }
        return *instance;
    }
 
    static void Kill( void )
    {
        if( instance != NULL )
        {
            delete instance;
        }
    }
 
private :
    CExcel(){}
 
    static CExcel * instance;
};

[Alp]

L'implémentation proposée par loufoque est aussi très utilisée raptor