Discussion :

[Kaelian]

Bonjour,
désolé pour les deux questions le même jours, mais bon ce problème là est un peu plus chiant que l'autre.

Alors voilà, comme d'habitude, je suis l'article de Laurent à propos de la création d'un moteur 3D et là j'ai réalisé le MediaManager.
Là où j'ai un soucis, c'est dans ce code: (j'ai enlevé les parties inutile)

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
 
    class Mesh {
    };
 
    template <typename T>
    struct MediaHolder {
        typedef std::map<std::string, boost::shared_ptr<Loader<T> > > LoaderMap;
 
        LoaderMap loaders; //!< Map contenant les loaders.
    };
 
    //! Liste des medias supportés.
    typedef TYPELIST_1(Mesh) Medias;
 
 
    class MediaManager : public Hierarchy<Medias, MediaHolder> {
        public:
            MediaManager();
 
            ~MediaManager();
 
            FileName findMedia(const FileName& filename);
 
 
        private:
            std::vector<std::string> searchPath;
    };
 
 
    template <typename T>
    Loader<T>* MediaManager::findLoader(const std::string& extension) const {
        // Rechercher l'extension dans la liste des extension supportées par le loader.
        MediaHolder<T>::LoaderMap::iterator it;
 
        return NULL;
    }

Et l'erreur se situe à la ligne " MediaHolder<T>::LoaderMap::iterator it; "
Il me dit: ; expected before it.

Quelqu'un voit d'où peut venir l'erreur ?
En lisant la FAQ on peut y lire des choses à propos de typename au lieu de typedef, mais même en utilisant ce principe mon problème reste le même.

Il faut savoir aussi, j'ai essayé de mettre:
std::map<std::string, boost::shared_ptr<Loader<T> > >::iterator it;
ça n'a pas marché.
Par contre, si je fais:
std::map<std::string, boost::shared_ptr<Loader<Mesh> > >::iterator it;
là ça marche ...

J'espère que quelqu'un pourra m'aider.
Merci d'avance. ^^

[Alp]

Code :

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

typedef typename std::map< ... > tontype;

http://cpp.developpez.com/faq/cpp/?p...LATES_typename

[Kaelian]

C'est justement ce point de la faq dont je faisais référence. Et le soucis est identique: même erreur de compilation.

Il y a eu du nouveau entre temps, après la lecture d'un article, j'ai fait ceci dans ma fonction findLoader:

Code :

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

1
2
3
4
5
6
7
8
9
10
11
 
    //! Trouver le loader adequate par rapport à l'extension.
    template <typename T>
    Loader<T>* MediaManager::findLoader(const std::string& extension) const {
        typedef typename std::map<std::string, boost::shared_ptr<Loader<T> > >::iterator LoaderMapIterator;
 
        // Rechercher l'extension dans la liste des extension supportées par le loader.
        LoaderMapIterator it;
 
        return NULL;
    }

Ca marche, mais c'est sale ...
Je comprend vraiment pas pourquoi le point de la faq ne marche pas dans mon cas. Suis-je obligé de passer par cette methode pas très jolie ? :/

[Alp]

Mais non.

Il suffit de remplacer

Code :

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

typedef std::map<std::string, boost::shared_ptr<Loader<T> > > LoaderMap;

par

Code :

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

typedef  typename std::map<std::string, boost::shared_ptr<Loader<T> > > LoaderMap;

Quelle erreur obtiens-tu quand tu fais cela ?

[Kaelian]

J'obtiens exactement la même erreur:
expected `;' before "it"|

[loufoque]

Les commentaires de Alp (écrire typedef typename std::map< ... > tontype;), c'est n'importe quoi. Il ne faut pas utiliser typename dans ces cas-là.
Quel est le problème avec ton code ? Tu as écrit toi-même la solution quelques messages au-dessus.

typename est uniquement nécessaire quand on accède à un membre statique d'une classe template dont les paramètres ne sont pas connus lors de cette phase (i.e. ils sont issus d'autres paramètres templates).
Par défaut un tel membre est une valeur. S'il s'agit d'un type, il faut donc l'indiquer avec typename.