Discussion :

[david lefaivre]

Salut,

J'ai une petite question. Ça fait longtemps que j'ai fait du c++ et j'ai oublié comment on fait (si c'est possible) pour déclarer une fonction static dans le header et la définir dans le cpp d'implémentation. En fait, j'aimerais faire une classe singleton et j'ai abouti avec le code suivant :

header:

Code :

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

1
2
3
4
5
6
7
8
9
10
11
12
 
class Yo
{
	private:
		Yo();
 
	public:
		static Yo* getInstance();
 
	private:
		static Yo* instance;
};

cpp

Code :

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

1
2
3
4
5
6
7
8
 
Yo* Yo::getInstance()
{
	if (instance == NULL)
		instance = new Yo;
 
	return instance;
}

Le problème est que ce bout de code ne compile pas. J'ai une erreur de linkage, en fait.

Error 1 error LNK2001: unresolved external symbol "public: static class Yo * __cdecl Yo::getInstance(void)" (?getInstance@Yo@@SAPAV1@XZ)

Pour ce que j'en comprend, la function instance ne semble pas être reconnue. Je supposes que le problème est que la méthode getInstance est static. Donc, je réitère ma question : Comment on fait (si c'est possible) pour déclarer une fonction static dans le header (le prototype) et la définir (le corps de la fonction) dans le cpp d'implémentation ?

Merci

[BlueMonkey]

Bonsoir,

Le message d'erreur n'est pas explicite. Mais ce qu'il manque c'est l'initialisation du pointeur static Yo* instance;.
Pour ne pas se compliquer la vie sur l'allocation/libération de mémoire, je suggère ceci.

.CPP

Code :

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

1
2
3
4
5
6
Yo Yo::instance;
 
Yo* Yo::getInstance()
{
	return &instance;
}

.H

Code :

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

1
2
3
4
5
6
7
8
9
class Yo
{
    private:
    		Yo(){};
	public:
		static Yo* getInstance();
	private:
		static Yo instance;
};

Autre solution avec une référence et variable statique dans getInstance : FAQ Singleton

[3DArchi]

Salut,

- Il manque la définition de la variable statique membre
- Il manque l'interdiction de la copie (opérateur = et constructeur de copie privés ou =delete si C++11)
- Si l'allocation peut être dynamique, il vaut mieux retourner un objet par référence static Yo& getInstance();.
- Pour rappel, ce genre de création n'est pas thread-safe (et ne peux l'être en c++03)
- Pour rappel, les singletons peuvent être considérés un anti-pattern (c'est exactement les mêmes inconvénients qu'une variable globale)

[david lefaivre]

Merci pour la réponse.