erreur en parcourant une map assez bizarre

Bonjour,

Voici la fonction ou le compilateur m'insulte.

Code:

---

1 2 3 4 5 6 7 8 9 10 11 12 13 14 15

typedef std::map<std::string /*nom param*/, std::string /*valeur param*/> ListParams; typedef map<std::string, std::string>::iterator ListParamsIt ; ................   void XMLMessageOut::addAttributes(const ListParams& params_list) {   //ligne: 103 for( ListParamsIt iter = params_list.begin(); iter != params_list.end(); iter++ ) {   xmlTextWriterWriteAttribute(writer, BAD_CAST (&((*iter).first)), BAD_CAST (&((*iter).second))); }   }

---

et l'insulte :

]../../../src/proj/XMLMessageOut.cpp: In member function `void XMLMessageOut::addAttributes(const std::map<std::string, std::string, std::less<std::string>, std::allocator<std::pair<const std::string, std::string> > >&)':
../../../src/proj/XMLMessageOut.cpp:103: error: conversion from `std::_Rb_tree_const_iterator<std::pair<const std::string, std::string> >' to non-scalar type `std::_Rb_tree_iterator<std::pair<const std::string, std::string> >' requested



Quelqu'un a une idée pourquoi le compilateur m'insulte ?

Merci en avance

Ben ouais, à priori tu passes ta map par référence constante.
L'itérateur que tu peux en retirer doit être un const_iterator.
Or tu utilises ton typedef ListParamsIt qui lui n'est pas constant.

En général, l'idée de faire un typedef sur une map est très bien, mais je ne fait pas de typedef sur les iterator à cause ce problème, j'utilise plutôt ListParams::iterator ou ListParams::const_iterator dans mon code c'est pas trop lourd.

Salut,

Sinon c'est possible de typedef aussi le const_iterator et d'utiliser la bonne version au bon moment :

Code:

---

1 2 3

typedef std::map< std::string, std::string > ListParams; typedef ListParams::iterator ListParamsIt; typedef ListParams::const_iterator ListParamsConstIt;

---

Mais c'est sûr que ça ne raccourcit pas tant que ça l'écriture...

MAT.

Salut,

Citation:

---

Envoyé par Mat007

Salut,

Sinon c'est possible de typedef aussi le const_iterator et d'utiliser la bonne version au bon moment :

Code:

---

1 2 3

typedef std::map< std::string, std::string > ListParams; typedef ListParams::iterator ListParamsIt; typedef ListParams::const_iterator ListParamsConstIt;

---

Mais c'est sûr que ça ne raccourcit pas tant que ça l'écriture...

MAT.

---

A vrai dire, tout dépend de si tu comptes utiliser régulièrement les iterateurs, constant ou non, dans tes fonctions/méthodes...

Si c'est pour ne l'utiliser que dans une seule fonction, de fait, cela ne vaut pas vraiment la peine...

Si tu as un grand nombre de fonctions qui vont devoir utiliser un (const_)iterator, cela peut devenir intéressant :D

En fait je pense qu'il faut voir plus large que ça, par soucis d'uniformité, il faut à mon avis surtout choisir une solution et s'y tenir.
Quand je vois du code non uniforme avec lui-même avec des conventions folkloriques, je me dis que ça doit être pareil dans la tête du développeur...

MAT.