C++

Pointeur intelligent et singleton [résolu]

Recherche :

Mot : Pseudo : Filtrer
Bas de page
Auteur	Sujet : Pointeur intelligent et singleton [résolu]

bqron

We become what we think about.

Salut, j'essai de transposer ce code en utilisant des pointeur intelligent et je sèche...

L'original :

Code :

TextureManager* TextureManager::s_pInstance = NULL; // déclarer en static dans le header
TextureManager* TextureManager::Instance() // idem
{
if (s_pInstance == NULL)
{
s_pInstance = new TextureManager();
return s_pInstance;
}
return s_pInstance;
}

Ma sauce :

Code :

//---------------- TextureManager.h ----------------
class TextureManager
{
private:
std::map<std::string, SDL_Texture*> m_textureMap;
//static TextureManager* s_pInstance; ////
static std::unique_ptr<TextureManager> s_pInstance;
TextureManager();
public:
//static TextureManager* Instance(); // singleton
static std::unique_ptr<TextureManager> Instance();
~TextureManager();
};
//---------------- TextureManager.cpp ----------------
std::unique_ptr<TextureManager> TextureManager::s_pInstance = nullptr; // toujours en static dans le header
std::unique_ptr<TextureManager> TextureManager::Instance() // idem
{
if (s_pInstance == nullptr)
{
std::unique_ptr<TextureManager> s_pInstance(new TextureManager());
return s_pInstance;
}
return std::move(TextureManager::s_pInstance);
}

Le singleton ne fonctionne plus, s_pInstance reçoit une nouveau pointeur a chaque appelle de "Instance()"

Message édité par bqron le 14-12-2014 à 07:17:58

Publicité

Farian

Bonsoir !

Dans la fonction "instance", vous créez une variable locale "s_pInstance", qui masque le membre statique, et c'est à cette variable que vous donnez une valeur, pas au membre, qui vaut donc toujours "nullptr", puisqu'aucune valeur ne lui a été affectée ...

Enlevez le "std::unique_ptr<TextureManager>" et faites une affectation (je n'ai plus la syntaxe du std::unique_ptr en tête, je ne sais pas si il faut faire un "=" ou un "set" pour lui affecter une valeur).

Bonne continuation !

bqron

We become what we think about.

Merci de m'avoir mit sur la bonne voie, le problème est résolu :

Code :

std::shared_ptr<TextureManager> TextureManager::s_pInstance = nullptr;
std::shared_ptr<TextureManager> TextureManager::Instance()
{
if (s_pInstance == nullptr)
{
s_pInstance = std::make_shared<TextureManager>(TextureManager());
return s_pInstance;
}
return s_pInstance;
}

Une question tout de même à propos du code original si possible, j'ai remarqué que le destructeur n’étais jamais appeler du coup le pointeur renvoyé par "new TextureManager();" n’étais jamais libéré (a mon avis), même si j'utilisais le code ci-dessous, quelqu'un aurait-il une explication ?

Code :

TextureManager::~TextureManager()
{
if (s_pInstance != NULL)
delete s_pInstance;
}

Farian

Dans l'ancienne version, l'attribut "s_pInstance" restait toujours à NULL, vu que c'était une variable locale (qui n'était a priori jamais libérer) qui était allouée, en lieu et place de l'attribut.

En revanche, un destructeur public pour une classe singleton, c'est vraiment très étrange, pas du tout dans l'état de l'art (et casse-gueule dans environ 100 % des cas)

Bonne continuation !

Message édité par Farian le 15-12-2014 à 08:03:49

FORUM HardWare.fr

Programmation

C++

Pointeur intelligent et singleton [résolu]

Sujets relatifs
[Resolu][VBA]pointeur sur une sous-feuille dans un sous formulaire	Aide sur une fonction retournat un pointeur sur char. Merci
Pointeur sur tableau de structure	tableau de pointeur sur structure dans une structure
déclaration d'un vector de pointeur	Différence entre pointeur de pointeur et vector de pointeur
vecteur de vecteur ou pointeur de pointeur	Comment récupérer la forme du pointeur souris n'importe où sous window
Générateur de chiffres à la suite d'une formule	pointeur de tableau 2D d'entiers...
Plus de sujets relatifs à : Pointeur intelligent et singleton [résolu]

Page générée en 0.067 secondes