Discussion :

[LiquidHuk]

Bonjour,

Je souhaite réaliser une classe permettant de gérer le chargement des ressource sons et images avec sfml. Le chargement étant identique au type prêt (sf::Image ou sf::SoundBuffer) entre les deux, j'ai décidé de créer une classe template singleton qui charge et conserve les médias en mémoire. L'objectif est donc de pouvoir charger des médias de la manière suivante :

Code :

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

1
2
3
 
MediaManager<sf::SoundBuffer>::getInstance()->getMedia("chemin vers le son");
MediaManager<sf::Image>::getInstance()->getMedia("chemin vers l'image");

Pour réaliser cette classe, je m'y suis pris de la manière suivante :

MediaManager.hpp :

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
 
#include <SFML/Audio.hpp>
#include <iostream>
#include <map>
 
template<class T> class MediaManager
{
public:
	static MediaManager<T>* getInstance()
	{
		if (NULL == _singleton)
		{
			std::cout << "DEBUG: MediaManager | Creation du singleton." << std::endl;
			_singleton = new MediaManager<T>;
		}
		return _singleton;
	}
 
private:
	static MediaManager<T>* _singleton;			//Pointeur du singleton
 
	MediaManager();
 
};
 
 
template<class T> MediaManager<T>* MediaManager<T>::_singleton = NULL;

Avec simplement la création de la classe sous forme de singleton, je me retrouve avec une erreur de link sur getinstance() :

Erreur 2 error LNK2019: symbole externe non résolu "private: __thiscall MediaManager<class sf::SoundBuffer>::MediaManager<class sf::SoundBuffer>(void)" (??0?$MediaManager@VSoundBuffer@sf@@@@AAE@XZ) référencé dans la fonction "public: static class MediaManager<class sf::SoundBuffer> * __cdecl MediaManager<class sf::SoundBuffer>::getInstance(void)" (?getInstance@?$MediaManager@VSoundBuffer@sf@@@@SAPAV1@XZ)

Je ne vois pas ce que je fais de mal ici, une idée pour me guider ?

[Trademark]

Je pense que tu n'as pas implémenté le constructeur par défaut (on ne voit que la signature dans l'exemple) ou que tu l'as implémenté dans un .cpp, vu que c'est une classe template tout doit être dans le .hpp.

Aussi, si tu utilises plusieurs threads, sache que ton code n'est pas thread-safe, si tu utilises C++11, je te conseille de faire simplement ça :

Code :

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

1
2
3
4
5
static MediaManager<T>& getInstance()
{
  static MediaManager<T> manager;
  return manager;
}

Et même si tu n'utilises pas plusieurs threads, ce code est quand même plus clair, je ne vois pas l'avantage de passer par un pointeur...

[LiquidHuk]

En effet, je me suis concentré sur les templates alors que l'erreur était toute bête, c'était bien le constructeur.
Je ne connaissais pas cette implémentation pour les singletons et je vais l'utiliser même si je n'ai pas besoin de thread, c'est en effet beaucoup plus clair et léger comme ça.

Merci beaucoup !