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..

[alen]

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

Je pense bien, j'ai également le problème dans le bout de code suivant, cela doit être le même genre de problème

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
 
template<typename T>bool CDB_StringID::GetIDstrAs(UINT index, T &val)
{
	if (index>1) 
	{
		ASSERT (0);
		return false;
	}
	_string str;
	index?str=GetIDstr2():GetIDstr();
	if (str.empty()) return false;
	return from_string<T>(str,val);
}
 
 
avec 
 
// Convert a string to a specified type
template<typename T>bool from_string( const _string & Str, T & Value )
{
    // Build an input string with given string
#ifdef UNICODE
	std::wistringstream iss( Str );
#else
	std::istringstream iss( Str );
#endif
	// Try to make conversion 
	return (( iss >> Value ) != 0);
}

Ici le link ne passe pas

Code :

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

from_string<T>(....)

ne doit pas être valable

alors qu'un appel du type

Code :

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

from_string<double>(...)

fonctionne parfaitement.
Je ne dois pas avoir tout compris des template...

[farscape]

re,
tu as vu ce tuto ?

[alen]

re,
tu as vu ce tuto ?

Interressant (surtout la library BOOST, une vraie mine d'or), j'utilise l'approche SL (mon code doit être pour certaines parties compatible VS2008 et GCC).
Pour vérifier j'ai implémenté la version FromString du tuto sans plus de succès.
Le problème que j'ai ici est un problème de visibilité de la fonction qui est déclarée de la manière suivante:

Code :

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

1
2
// Convert a string to a specified type
template<typename T>extern bool from_string( _string & Str, T & Value );

Le fichier contenant cette déclaration est bien inclus dans le fichier contenant la classe CDB_StringID ....
J'ai une erreur LNK1120: unresolved external!
Si je place le code de la fonction from_string juste avant CDB_StringID::GetIDstrAs() le link est ok

[farscape]

pour un template ou une fonction modèle ,la déclaration et l'implémentation doit être dans un seul fichier...
si c'est dans le .h tout doit être dans le .h.

[alen]

Merci Farscape,

Cela résoud mon problème from_string.
Donc dans un code contenant des fichiers cpp et h toutes les fonctions templates qui doivent être vue à l'extérieur devront être déclarées le fichier .h. N'est ce pas un peu lourd? Il y a t-il un code de bonne pratique?

[farscape]

pour l'instant il n'y a pas d'autre solution ...
une pratique courante:
mettre la définition de sa classe dans le .h
et l'implémentation dans le même nom avec une extension differente comme .hpp
ensuite en dessous de la classe on rajoute un include de ce fichier.

[alen]

Ok, je vais me tenir à cela
Merci