[LINKAGE] probleme avec Singleton

Bonjour,

Voici mon souci,
J ai créée une classe avec un singleton (ManagerSingleton). il est hyperbasique
il est dans un fichier ManagerSingleton.h

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 42

#pragma once // Include files #include "stdhdrs.h" class ManagerSingleton { protected: // Constructeur/destructeur ManagerSingleton (); ~ManagerSingleton (); // Variables membres int _value; static ManagerSingleton *_singleton;   public: // Interface publique // Fonctions de création et destruction du singleton static inline ManagerSingleton* getInstance () { if (_singleton == NULL ) { // std::cout << "creating singleton." << std::endl; _singleton = new ManagerSingleton; } else { // std::cout << "singleton already created!" << std::endl; }   return _singleton; }   static void inline kill () { if (NULL != _singleton) { delete _singleton; _singleton = NULL; } }   };

---

Tous va bien jusque la
le je souhaite appele ce singleton dans une autre class
J ajoute un include dans le header de ma class

Code:

---

#include "ManagerSingleton.h"

---

puis dans le .cpp
je fais

Code:

---

ManagerSingleton* ManagerSingleton = ManagerSingleton::getInstance ();

---

à partir du moment ou j ajoute cette ligne de code j ai l erreur suivante :

Code:

---

Error 116 error LNK2019: unresolved external symbol "protected: __thiscall ManagerSingleton::ManagerSingleton(void)" (??0ManagerSingleton@@IAE@XZ) referenced in function "public: static class ManagerSingleton * __cdecl ManagerSingleton::getInstance(void)" (?getInstance@ManagerSingleton@@SAPAV1@XZ)

---

et j arrive pas en m en depetrer :aie:

essaie de remplacer:

Code:

---

1 2 3

ManagerSingleton (); ~ManagerSingleton ();

---

par:

Code:

---

1 2 3

ManagerSingleton () {} ~ManagerSingleton () {}

---

Bien le bonjour,

Par le plus grand des hasards, n'aurais-tu pas oublié de définir ta donnée statique static ManagerSingleton *_singleton; ?

:arrow: http://c.developpez.com/faq/cpp/?pag..._static_erreur

Merci
oui khayyam90 c est fort probablement ça.
mais alors comment faire pour le declarer dans le .cpp ?
Alors que ma classe est est declare dans un Header ?

Citation:

---

Envoyé par magnus2005

alors comment faire pour le declarer dans le .cpp ?

---

en mettant quelque chose comme

Code:

---

ManagerSingleton *ManagerSingleton::_singleton;

---

dans ton .cpp

pas dans une fonction particulière, par exemple (comme dans la faq) juste après les #include. Et cette définition devra être unique, elle devra être dans une seule unité de compilation.

pas besoin de mettre static dans la declaration ?

avec ou sans static j ai l erreur suivante :

Code:

---

Error 2 error C2039: '_singleton' : is not a member of 'ManagerSingleton' d:\workspace\vnc_winsrc\winvnc\managersingleton.cpp 5

---

Je suis desolé mais je trouve
http://c.developpez.com/faq/cpp/?pag...SS_init_static

qui est juste avant et qui declare sa variable static dans son header 8O

En suivant la FAQ C++ sur les static
http://c.developpez.com/faq/cpp/?pag...SS_init_static
j ai recode mon fichier .h

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

#ifndef MANAGERSINGLETON_H #define MANAGERSINGLETON_H #pragma once // Include files #include "stdhdrs.h" class ManagerSingleton { protected: // Constructeur/destructeur ManagerSingleton (); ~ManagerSingleton (); // Variables membres int _value; static ManagerSingleton *_singleton;   public: // Interface publique // Fonctions de création et destruction du singleton static ManagerSingleton* getInstance (); static void Init (); static void inline kill ();   };   #endif // Initialisation du singleton à NULL ManagerSingleton* ManagerSingleton::_singleton = NULL;

---

mais je me retrouve avec l erreur suivante comme quoi ma variable _singleton est déja défini.

Code:

---

Error 143 fatal error LNK1169: one or more multiply defined symbols

---

Pourtant je la défini bien qu'une fois dans mon code.
Je ne comprend pas tres bien :roll:

Citation:

---

Je suis desolé mais je trouve
http://c.developpez.com/faq/cpp/?pag...SS_init_static

qui est juste avant et qui declare sa variable static dans son header

---

Pas dans son en-tête non, tu as mal regardé.

Autant pour moi,
Merci beaucoup
ça fonctionne (du moins ça compile et ça LINK)

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

#ifndef MANAGERSINGLETON_H #define MANAGERSINGLETON_H #pragma once // Include files #include "stdhdrs.h" class ManagerSingleton { protected: // Constructeur/destructeur ManagerSingleton (); ~ManagerSingleton (); // Variables membres int _value; static ManagerSingleton *_singleton;   public: // Interface publique // Fonctions de création et destruction du singleton static ManagerSingleton* getInstance (); static void Init (); static void inline kill ();   };   #endif

---

:resolu: ?

Oui et non

J'ai toujours un probleme de linkage (la meme erreur qu au debut mais un peu plus loin dans le code) :

Code:

---

1 2	Error 117 error LNK2019: unresolved external symbol "public: void __thiscall ManagerSingleton::StartSharing(void)" (?StartSharing@ManagerSingleton@@QAEXXZ) referenced in function "public: static long __stdcall vncDesktop::KeybdProc(int,unsigned int,long)" (?KeybdProc@vncDesktop@@SGJHIJ@Z)

---

Voici mon header

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

#pragma once // Include files #include "stdhdrs.h" class ManagerSingleton { protected: // Constructeur/destructeur ManagerSingleton (); ~ManagerSingleton (); // Variables membres int _value; static ManagerSingleton *_singleton;   public: // Interface publique // Fonctions de création et destruction du singleton static ManagerSingleton* getInstance (); static void Init (); static void Kill (); public: void StartSharing(); void StopSharing(); void ControlBack();   };   #endif

---

J ai l erreur de linkage quand j appelle "StartSharing" qui n est pas pas une methode static pourtant.

Code:

---

1 2 3

ManagerSingleton* singl; singl = ManagerSingleton::getInstance(); singl->StartSharing();

---

[EDIT] Le code que j ai posté etait le bon j avais une erreur dans le .cpp
Merci