Destructeur privé

Bonjour,

J'essaie d'implementer la classe singleton avec un desctructeur privé, sauf qu'à la compilation j'ai cette erreur (GlobalClass::~GlobalClass'*: impossible d'accéder à private membre déclaré(e) dans la classe 'GlobalClass) dans ligne :

Code:

---

GlobalClass sing=*p1;

---

alors que la ligne

Code:

---

GlobalClass &ref=*p1;

---

ne pose aucun problème.
Quelqu'un peut m'expliquer?
Rq: Le programme compile qd le destucteur est publique.

Code:

---

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 39 40 41

class GlobalClass { int m_value; static GlobalClass *s_instance; GlobalClass(int v = 0) { m_value = v; cout<<" construct singleton"<<endl; } ~GlobalClass(){} public: int get_value() { return m_value; } void set_value(int v) { m_value = v; } static GlobalClass *instance() { if (!s_instance) s_instance = new GlobalClass; return s_instance; } }; // Allocating and initializing GlobalClass's // static data member. The pointer is being // allocated - not the object inself. GlobalClass *GlobalClass::s_instance = NULL; int main() { GlobalClass *p1=GlobalClass::instance(); GlobalClass sing=*p1; // le problème est là GlobalClass *p2=p1->instance(); GlobalClass &ref=*p1; p1->set_value(3); return 0; }

---

Ton code est tres bizzard8O
regarde ici pour faire un singleton en C++: http://come-david.developpez.com/tut...Singleton#LIII

dans ton main l'utilisation de ton singleton est aussi tres bizarre:
toutes tes référence et pointeurs sont inutiles.
tu devrais avoir quelque chose du genre:

Code:

---

1 2 3 4 5 6

int main() { GlobalClass::instance().set_value(3); return 0; }

---

Code:

---

1 2	GlobalClass sing=*p1;

---

Ce code utilise le constructeur par recopie, qui n'est pas redefinie dans ta classe, et qui utilise donc le constructeur de copie par defaut, qui est public.

Le probleme est donc que tu peux bien copie ton instance, mais pas la detruire, puis que destructeur est prive.

Hors tu appelles automatiquement le destructeur a la fin du main, puisque sing n'est pas alloue dynamiquement mais dans la pile.

Mettre

Code:

---

1 2	GlobalClass &sing=*p1;

---

resoudra ton probleme

Edit :

Oui j'oubliais juste, tu n'as pas de methode permettant de destruire ton singleton, et tu auras donc des memory leaks. Tu as 2 solutions simple :

Soit tu ajoute la methode, pour la detruire, et a ce moment-la tu peux recreee ton instance a tout moment,

Soit tu met simplement

Code:

---

1 2	static GlobalClass _instance;

---

dans ton singleton, qui sera alors automatiquement detruit a la fin du programme

On pourrait mettre à jour cet article sinon avec:

- Une utilisation thread-safe: ne pas faire de lazy-init, mais une init genre dans le main avant le lancement des threads

- Il y a une solution avec une variable statique et un appel à boost::call_once qui permet de faire du lazy-init thread safe

Salut,
Ceci dit faire un copie avec un DP singleton ... c'est original. C'est un multi-ton alors :mouarf:
Ton singleton devrait être non copiable : F.A.Q. Comment rendre une classe non copiable ?

Citation:

---

Envoyé par nikko34

On pourrait mettre à jour cet article sinon avec:

- Une utilisation thread-safe: ne pas faire de lazy-init, mais une init genre dans le main avant le lancement des threads

- Il y a une solution avec une variable statique et un appel à boost::call_once qui permet de faire du lazy-init thread safe

---

JE ne suis pas certain de ce que tu appelles lazy-init, mais le code :

Code:

---

1 2 3 4 5

Singleton &Singleton::instance() { static Singleton theInstance; return theInstance; }

---

est thread-safe, selon la définition C++0x du langage, et appelle les destructeurs à la fin du programme. Seul problème : Certains compilateurs (visual C++, par exemple) ne respectent pas (encore) cette règle.