Discussion :

[Hyssgrif]

Bonjour,

Le tuto n'offre pas de solutions pour réaliser des singletons en passant par des smart pointer.
Comment peut-on s'y prendre?

Actuellement mes tests donne:

.h

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 Ressources_Manager : boost::noncopyable{
public:
	static boost::shared_ptr<Ressources_Manager> Instance();
	virtual ~Ressources_Manager();
 
 
private:
//	boost::shared_ptr<Ressources_Manager> operator= (const boost::shared_ptr<Ressources_Manager>){}
	Ressources_Manager();
	Ressources_Manager (const Ressources_Manager&);
 
	static boost::weak_ptr<Ressources_Manager>  m_instance;
 
	sf::Texture H_Sp_Prairie;
	sf::Image   Img_H_Prairie;
 
	void Load_Textures();
	void Load_Images();
};

.cpp

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
 
	boost::shared_ptr<Ressources_Manager> Ressources_Manager_ptr = Ressources_Manager::Instance();
//	Ressources_Manager Ressources_Manager::m_instance=Ressources_Manager();
 
		Ressources_Manager::Ressources_Manager() {
 
			Load_Images();
			Load_Textures();
		}
 
		Ressources_Manager::~Ressources_Manager() {
 
		}
 
		boost::shared_ptr<Ressources_Manager> Ressources_Manager::Instance()
		{
 
			  boost::shared_ptr<Ressources_Manager> instance = m_instance.lock();
			  if (!instance) {
			    instance.reset(new Ressources_Manager());
			    m_instance = instance;
			  }
			return instance;
		}
 
		void Ressources_Manager::Load_Images()
		{
			Img_H_Prairie.loadFromFile("Image/HexaTestSize.bmp");
			Img_H_Prairie.createMaskFromColor(sf::Color(255,255,255));
		}
 
		void Ressources_Manager::Load_Textures()
		{
			H_Sp_Prairie.loadFromImage(Img_H_Prairie);
		}

Erreur affichée:
../RessourcesManager.cpp:29: undefined reference to `itpro::ress::Ressources_Manager::m_instance'

[dragonjoker59]

Bon, pour commencer, ton instance ne devrait pas être stockée dans un weak_ptr, car une fois sorti de ta fonction Instance(), le shared_ptr est détruit, et comme il est la seule strong reference sur l'instance, celle-ci est détruite.
Ensuite, utilise boost::make_shared plutôt que ptr.reset( new... ).
Pour ton erreur de link, c'est simplement dù au fait que tu déclarais m_instance en tant que weak_ptr et tu l'implémentais en tant que shared_ptr


EDIT:

Code :

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

1
2
	boost::shared_ptr<Ressources_Manager> Ressources_Manager_ptr = Ressources_Manager::Instance();
//	Ressources_Manager Ressources_Manager::m_instance=Ressources_Manager();

Que cherches-tu à faire là? Pourquoi m_instance a-t-elle changé de nom?

[JolyLoic]

Je ne suis pas certain que singleton et shared pointers aillent bien ensemble (sans compter que le singleton st souvent un anti-pattern) : Tu va avoir un singleton qui va être détruit dès que personne ne l'utilise, et se reconstruire à chaque utilisation. Difficile dans ces cas là de garder de l'état d'une utilisation à l'autre, et sans état à garder et à partager, à quoi sert un singleton ?

Pour ton problème de link, j'imagine que c'est parce que tu as déclaré ta variable statique dans ton .h, mais que tu ne l'as définie nulle part. Ajoute dans ton .cpp:

Code :

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

boost::weak_ptr<Ressources_Manager>  Ressources_Manager::m_instance;

http://cpp.developpez.com/faq/cpp/?p...tion-des-liens

Sinon, ta fonction Instance n'est pas thread-safe : Qu'arrive-t-il si, alors que ton weak_ptr est vide, deux personnens demandent simultanément accès à Instance ? Tu risques de créer deux shared_ptr différents...

[Hyssgrif]

Bonjour,

Merci pour vos réponses,
En faite je n'ai pas besoins d'utiliser des threads pour cette class, mais je souhaite en faire un singleton pour ne l'initialisée q'une seule fois et pouvoir l’instancier un peut partout dans mon projet comme s'il s'agissait d'une variable globale.
Je me suis un peut aventuré avec les weak_ptr sans savoir ou j'allais en suivant un code trouvé sur internet.
En revanche je souhaiterais vraiment faire appel aux smart_ptr soit les shared_ptr pour partager cette classe avec les autres ressources de mon programme.
Je n'ai pas encore trouvé la solution.

Merci à vous

[Hyssgrif]

Pb résolue,

Merci à vous

[koala01]

Salut,

Bonjour,

Merci pour vos réponses,
En faite je n'ai pas besoins d'utiliser des threads pour cette class, mais je souhaite en faire un singleton pour ne l'initialisée q'une seule fois et pouvoir l’instancier un peut partout dans mon projet comme s'il s'agissait d'une variable globale.
Je me suis un peut aventuré avec les weak_ptr sans savoir ou j'allais en suivant un code trouvé sur internet.
En revanche je souhaiterais vraiment faire appel aux smart_ptr soit les shared_ptr pour partager cette classe avec les autres ressources de mon programme.
Je n'ai pas encore trouvé la solution.

Merci à vous

Je me permet de revenir sur cette discussion, mais, à lire ton intervention, j'ai envie de te poser une seule question:

Pourquoi voudrais tu pouvoir utiliser une variable comme si elle était globale (que ce soit sous la forme d'un singleton ou non)

Car, d'expérience:

1. aucune variable ne doit effectivement être accessible depuis partout dans le code
2. le simple fait de permettre à "n'importe quelle fonction" d'aller chipoter à une variable rend le code particulièrement difficile à débuger

Le plus simple, si tu veux t'assurer qu'une variable ne sera créée qu'une fois, c'est encore de faire en sorte d'avoir cette garantie par la construction de ton code (en veillant à ce qu'il n'y ai qu'une seule déclaration de la variable du type indiqué) et de veiller à la transmettre sous la forme d'une référence (éventuellement constante) à "tout ce qui pourrait en avoir besoin"

[Hyssgrif]

Salut Koala01,

En réalité, l'objectif finale et de créer une classe dans une bibliothèque qui aurait pour rôle de charger une fois et une seule des ressources au début du programme, les stocker et les rendre disponibles un peut partout dans le code.
Ici je charge des images, textures et autres..
Je veux que ces ressources soient accessibles par certaine parties du programme sans avoir a recréer la classe et donc recharger ces ressources