Discussion :

[alen]

Bonjour,
Je démarre dans l'utilisation des templates. J'ai réalisé quelques fonctions template avec succès. J'essaye maintenant généraliser une de mes classe outil par l'utilisation d'un template, mais je rencontre quelques difficultés. Peut-être pouvez-vous me conseiller?
Cette classe est simpement une réencapsulation d'un multimap avec quelques fonctions de manipulation suplémentaire
Fichier h original:

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
36
37
38
39
40
 
class CRankEx
{
public:
	CRankEx();
	~CRankEx();
 
protected:
	typedef std::pair<_string, LPARAM> ItemPair;
	typedef std::pair<double,ItemPair> MMapPair;
	typedef std::multimap<double, ItemPair> _mmap;
	_mmap::const_iterator m_It;
	_mmap m_mmap;
// Functions
public:
	void Add(const _string& str, double dKey, LPARAM itemdata=0, bool bAddIfExist=true);
....
};
 
et le .cpp
 
CRankEx::CRankEx()
{
	m_It = m_mmap.end();
}
 
 
void CRankEx::Add(const _string& str, double dKey, LPARAM itemdata/*=0*/, bool bAddIfExist/*=true*/)
{
	_mmap::iterator it;
	if (!bAddIfExist)
	{
		it=m_mmap.find(dKey);
		if (it != m_mmap.end())		// Item found
			return;
	};
	m_mmap.insert(MMapPair(dKey,ItemPair(str,itemdata)));
}
 
....

La clef de tri est un double. Mon but est de remplacer celle-ci par un template. J'ai donc:

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
36
37
38
39
40
41
42
43
 
template<typename T>
class CRankEx
{
public:
	CRankEx();
	~CRankEx();
 
protected:
	typedef std::pair<_string, LPARAM> ItemPair;
	typedef std::pair<T,ItemPair> MMapPair;
	typedef std::multimap<T, ItemPair> _mmap;
	_mmap::const_iterator m_It;
	_mmap m_mmap;
// Functions
public:
	void Add(const _string& str, T dKey, LPARAM itemdata=0, bool bAddIfExist=true);
....
};
 
et le .cpp
 
template<typename T>
CRankEx::CRankEx()
{
	m_It = m_mmap.end();
}
 
 
template<typename T>
void CRankEx::Add(const _string& str, T dKey, LPARAM itemdata/*=0*/, bool bAddIfExist/*=true*/)
{
	_mmap::iterator it;
	if (!bAddIfExist)
	{
		it=m_mmap.find(dKey);
		if (it != m_mmap.end())		// Item found
			return;
	};
	m_mmap.insert(MMapPair(dKey,ItemPair(str,itemdata)));
}
 
....

Il semble que la déclaration

Code :

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

typedef std::multimap<T, ItemPair> _mmap;

ne soit pas correct.
Quel serait votre conseil?
Merci

[farscape]

salut,
tu peux indiquer l'erreur ?
je ne suis pas sûr que l'on puisse faire un typedef sur un paramètre ...

[alen]

Code :

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

1
2
3
4
5
6
 
Warning	1	warning C4346: 'std::multimap<T,CRankEx<T>::ItemPair>::const_iterator' : dependent name is not a type	utilities.h	140	Phoenix
 
Error	2	error C2146: syntax error : missing ';' before identifier 'm_It'	utilities.h	140	Phoenix
 
Error	3	error C4430: missing type specifier - int assumed. Note: C++ does not support default-int	utilities.h	140	Phoenix

La ligne 140 étant

Code :

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

1
2
 
_mmap::const_iterator m_It;

[farscape]

et comme ça ?:

Code :

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

1
2
 
std::multimap<T, ItemPair>::const_iterator m_It;

[alen]

non, pas mieux.
J'ai l'impression que ce qui ne passe pas c'est le fait davoir une 'template dans un template' le T dans std::multimap<T,ItemPair>.
Je ne sais pas si une telle opération est permise.

[farscape]

hum en fait ça coince uniquement sur la déclaration de l'iterateur..