Discussion :

[Peeter]

Bonjour,

je suis nouveaux sur les templates et je ne comprends pas pourquoi le code suivant de compile pas (avec g++ sous OSX):

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
#ifndef RESSOURCEMANAGER_H
#define RESSOURCEMANAGER_H
 
#include <map>
#include <string>
 
template <class T>
class RessourceManager
{
    std::map<std::string, T> _ressourceDb;
 
public:
    RessourceManager() {}
    /**
      *
      */
    T getRessource(const std::string & ressourceName)
    {
 
	std::map<std::string, T>::iterator it = _ressourceDb.find(ressourceName);
	if(it == _ressourceDb.end())
	{
	    //_ressourceDb.insert(createRessource(ressourceName));
	}
	return it;
    }
 
    //virtual T createRessource(const std::string & ressourceName) = 0;
};
 
 
#endif // RESSOURCEMANAGER_H

Les erreurs de compilation sont les suivantes:

../Alpha/RessourceManager.h: In member function 'T RessourceManager<T>::getRessource(const std::string&)':
../Alpha/RessourceManager.h:20: error: expected `;' before 'it'
../Alpha/RessourceManager.h:21: error: 'it' was not declared in this scope
../Alpha/RessourceManager.h:25: error: 'it' was not declared in this scope

[Mat007]

Salut,

Ajoute typename devant, cf. cette entrée de la FAQ.

MAT.

[Peeter]

Merci!!

[koala01]

Salut, et bienvenue sur le forum.

Fais cependant attention au fait que, classiquement, un itérateur de std::map itère sur... des paires clé / valeurs

Si tu tentes de renvoyer it, tu renverra autre chose qu'un T, car tu renverra... un std::map<std:: string, T>::iterator, c'est à dire un iterator<std::pair<std::string, T> > et le compilateur ne va pas aimer cela

Tu devras donc renvoyer... it->second pour effectivement renvoyer ton T

De plus, vu que tu renvoie un T et non une référence sur T, tu renverras... une copie de l'objet présent dans la std::map, or, je présumes que tu souhaites que tout ce que ut pourrais faire avec l'objet récupéré soit... répercuté sur l'objet géré par ton manager(re ).

Tu devrais donc faire en sorte de renvoyer, non pas un objet de type T, mais une référence sur un tel objet

Ta fonction getRessource devrait donc au final prendre une forme proche de

Code :

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

1
2
3
4
5
6
7
8
9
10
11
12
13
14
    T& getRessource(const std::string & ressourceName)
    {
 
	std::map<std::string, T>::iterator it = _ressourceDb.find(ressourceName);
	if(it == _ressourceDb.end())
	{
	    //_ressourceDb.insert(createRessource(ressourceName));
            /* !!!   si tu insère un nouvel élément dans ta map, tu dois
             * veiller à ce que ton iterateur soit effectivement affecté à
             * cet élément ;) !!!
             */
	}
	return it->second;
    }

PS : pense à cliquer sur le bouton lorsque ton problème sera réglé