Discussion :

[shenron666]

bonjour,

j'ai une classe template de singleton :

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
template <class T>
class CSingletonStatic : public boost::noncopyable
{
protected:
	// ctor / dtor
	CSingletonStatic(){}
	~CSingletonStatic(){}
 
 
public:
	static CSingletonStatic& Instance(void)
	{
		static CSingletonStatic instance;
		return instance;
	}
};

et un resource manager que je déclare en singleton :

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
typedef boost::shared_ptr<unsigned char>		CResourcePtr;
typedef std::map<std::string, CResourcePtr>		CMapResources;
 
 
class CResourcesManager : public CSingletonStatic<CResourcesManager>
{
	friend class CSingletonStatic<CResourcesManager>;
 
private:
	// ctor / dtor
	CResourcesManager(void);
	virtual ~CResourcesManager(void);
 
 
public:
	// récupération d'une resource
	CResourcePtr Get(const std::string& sName);
 
 
protected:
	// chargement d'une resource
	CResourcePtr Load(const std::string& sName);
 
 
protected:
	CMapResources		m_mapResources;
};

mais je ne peux pas appeler la méthode Get dans le main de la manière suivante :

Code :

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

CResourcesManager::Instance().Get("test.dat");

l'erreur du compilateur est simple :

error C2039: 'Get'*: n'est pas membre de 'CSingletonStatic<T>'

qu'est-ce que j'ai fait de travers pour que ça ne fonctionne pas ?

ps: je suis sous vc7

[bolhrak]

Ta méthode statique de récupération de l'instance du singleton retourne un type CSingletonStatic<CResourcesManager>, qui n'a aucune méthode Get. Fais lui retourner un objet de type T (parce que si tu regardes bien, ton paramètre template ne sert à rien du tout dans ta class CSingletonStatic).

[bolhrak]

CResourcesManager::Instance() retourne un CSingletonStatic<CResourcesManager>, lequel ne contient pas de membre get, comme l'indique le compilateur. Pour que ton code fonctionne, il faut passer par un dynamic_cast :

Code :

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

1
2
 
dynamic_cast<CRessourcesManager&>(CResourcesManager::Instance()).Get();

CResourcesManager::Instance crée une variable statique de type CSingletonStatic<ResourcesManager>. Comment veux-tu pouvoir la sous-caster en CResourcesManager sans que ça plante ?

[shenron666]

Merci bolhrak, tu m'as mis sur la voie de la solution

Code :

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

1
2
3
4
5
	static T& Instance(void)
	{
		static T instance;
		return instance;
	}

l'instance que je créais n'était pas du bon type