Discussion :

[kass28]

Bonjour,

Voici la fonction ou le compilateur m'insulte.

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

[NiamorH]

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.

[Mat007]

Salut,

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

Code :

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

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.

[koala01]

Salut,

Salut,

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

Code :

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

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

[Mat007]

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.