Discussion :

[lepoutho]

//quelqu'un pourrait-il m'expliquer pourquoi les destructeurs de
//MonObjet et Singleton ne
//sont pas appelés.

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
37
38
 
class MonObjet
{
public:
	MonObjet() {
		std::cout<<"construction de MonObjet"<<std::endl;
	}
	~MonObjet () {
		std::cout<<"destruction de MonObjet"<<std::endl;
	}
};
 
class Singleton
{
public :
	Singleton()
	{
		std::cout<<"appel constructeur"<<std::endl;
	};
	~Singleton()
	{
		std::cout<<"appel du destructeur"<<std::endl;
	}
	static Singleton * pt;
	static Singleton * Instance()
	{
		if ( !pt) pt = new Singleton;
		return pt;
	}
	MonObjet titi;
};
Singleton * Singleton::pt  = 0;
 
int _tmain(int argc, _TCHAR* argv[])
{
	Singleton::Instance();
	return 0;
}

d'avance merci

[lepoutho]

En fait je précise la question, je voudrais faire de
l'allocation dynamique via une classe singleton et ça m'embete
de la deleter explicitement à la fin de mon prog, il y a sans
doute une autre solution. En fait je ne sais pas vraiment si
l'allocation de mémoire via un pointeur satique est deleté
en fin de prog sans passer par l'appel du destructeur.
A+

[la drogue c'est mal]

bah c'est un peu le principe du singleton. 1 instance unique qui sera détruit en meme temps que ton application.

à une époque lointaine, on appellait ca une classe globale donc imagine que c'est une variable globale

[lepoutho]

oui en fait pour l'appel de destructeur, ce code marche

Code :

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

1
2
3
4
5
6
7
 
int _tmain(int argc, _TCHAR* argv[])
{
	Singleton::Instance();
	delete Singleton::pt;
	return 0;
}

mais ça m'embete, si je n'ajoute pas la ligne de delete, j'ai
bien peur que ma memoire reste ou ne reste pas affecte suivant
tel ou tel OS.

A+

[la drogue c'est mal]

bah oui. Si tu essaies de dire que chaque new doit avoir son delete, tout le monde sera d'accord avec toi

sinon pour un "vrai" singleton :

il faut rendre inaccessible le constructeur ( soit un protected soit en private)
Ajour une methode pour liberer la mémoire, ReleaseInstance ou un nom comme ca qui détruit pt et le met à NULL

[Rafy]

le design patern du singleton c'est pas tout à fait ca.

Normalement c'est

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
 
class MonObjet
{
    public:
        static MonObjet& Instance(void)
        {
                if (!MonObjet::Instance)
                    MonObjet::pInstance = new MonObjet();
                return *this->pInstance;
        }
        static void Destroy(void)
        {
                 delete MonObjet::pInstance;
                 MonObjet::pInstance = NULL;
        }
    private:
    MonObjet(void);
    virtual ~MonObjet(void);
    static MonObjet* pInstance;
};

Il ne faut pas mettre d'autre constructeur en public.
S'il le faut tu fais une methode MonObjetIni(.....) pour initialiser des membre de ta class.

[Rafy]

Par la suit pour utiliser la methode Fonc qui est dans MonObjet il te faut tapper :

Code :

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

1
2
 
MonObjet::Instance().Fonc();             // Si la methode Fonc est private biensur  :twisted:

[Rafy]

Tu ne dois pas faire deux objets différents. ou alors du dérive ta class singleton pour en faire une MonObjet....

[Laurent Gomila]

Ceci devrait t'aider

http://c.developpez.com/faq/cpp/?pag...LASS_singleton

[Rafy]

yep Merci Loulou :o

[sebA]

Salut,

une petit chose en plus. Le GOF presente comme tu l'as decrit le singleton, mais c'est un peu un "pauvre" singleton car il n'y a pas de synchronisation et si ton programme est multi threader tu peux te retrouver avec plusieurs instance du soi disant singleton. Car il me semble que new n'est pas une operation automique donc une 2ieme thread peut rentrer ds la methode instance avant que la premiere n'ait fini d'allouer l'objet.


pour la synchro -> ACE http://deuce.doc.wustl.edu/

@+
Seb

[lepoutho]

merci à tous pour toutes ces précisions.