Discussion :

[Sunsawe]

Salut à tous,

Je cherche simplement à recopier une std::map<std::string,std::string> dans une autre.

J'ai tenté cette approche:

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
std::map<std::string,std::string> map_source;
std::map<std::string,std::string> map_destination;
 
/*remplissage de la source */
 
std::map<std::string,std::string>::const_iterator it = map_source.begin();
while(it!=map_source.end())
{
 
///////////////////////////////////////////////
///////// la ligne qui ne compile pas
 
   map_destination[it->first] = it->second;
 
// OU ENCORE
 
   map_destion.insert(std::pair<it->first,it->second>);
 
// OU ENCORE
 
   std::string cle = it->first;
   std::string valeur = it->second;
   //puis
   map_destination[cle] = valeur;
   //ou
   map_destion.insert(std::pair<cle,valeur>);
 
///////////////////////////////////////////////
   it++;
}

et j'obtiens systématiquement du compilateur, le message suivant:

passing «const std::map<std::basic_string<char, std::char_traits<char>, std::allocator<char> >, std::basic_string<char, std::char_traits<char>, std::allocator<char> >, std::less<std::basic_string<char, std::char_traits<char>, std::allocator<char> > >, std::allocator<std::pair<const std::basic_string<char, std::char_traits<char>, std::allocator<char> >, std::basic_string<char, std::char_traits<char>, std::allocator<char> > > > >» as «this» argument of «void std::map<_Key, _Tp, _Compare, _Alloc>::insert(_InputIterator, _InputIterator) [with _InputIterator = std::basic_string<char, std::char_traits<char>, std::allocator<char> >, _Key = std::basic_string<char, std::char_traits<char>, std::allocator<char> >, _Tp = std::basic_string<char, std::char_traits<char>, std::allocator<char> >, _Compare = std::less<std::basic_string<char, std::char_traits<char>, std::allocator<char> > >, _Alloc = std::allocator<std::pair<const std::basic_string<char, std::char_traits<char>, std::allocator<char> >, std::basic_string<char, std::char_traits<char>, std::allocator<char> > > >]» discards qualifiers|

Je ne sais pas si ça parle à quelqu'un mais pour récapituler, j'aimerai bien savoir ce que je fais de mal et aussi le moyen le plus efficace de faire une copie d'une map dans une autre map.

Merci à tous

[Sunsawe]

Je viens de lire dans un post qu'en cas de remplacement total de la map, l'opérateur d'affection suffit...
C'est tellement simple... mais est ce le plus efficace?

Et j'aimerai toujours l'explication sur le pourquoi mon code précédent ne compilait pas. (j'ai peut être introduit une erreur de syntaxe en rapportant le code de mémoire, je ne l'ai pas sous les yeux. Mais c'est bien les lignes indiquées qui ne compilent pas et avec le message que j'ai fournit).

Merci

[Sylvain Togni]

Code :

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

1
2
3
4
5
6
7
8
9
10
11
12
13
std::map<std::string,std::string> map_source;
std::map<std::string,std::string> map_destination;
 
/*remplissage de la source */
 
std::map<std::string,std::string>::const_iterator it = map_source.begin();
while(it!=map_source.end())
{
 
///////////////////////////////////////////////
///////// la ligne qui ne compile pas
 
   map_destination[it->first] = it->second;

Tout ceci compile chez moi, poste un vrai code minimal qui ne compile pas.

Code :

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

1
2
3
// OU ENCORE
 
   map_destion.insert(std::pair<it->first,it->second>);

La première erreur ici est que map_destion n'est pas défini. La seconde est que std::pair prend des types entre < >, pas des valeurs. Il faut utiliser std::make_pair pour cela :

Code :

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

map_destination.insert(std::make_pair(it->first, it->second));

Code :

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

1
2
3
4
5
6
7
8
// OU ENCORE
 
   std::string cle = it->first;
   std::string valeur = it->second;
   //puis
   map_destination[cle] = valeur;
   //ou
   map_destion.insert(std::pair<cle,valeur>);

idem :

Code :

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

map_destination.insert(std::make_pair(cle, valeur));

Code :

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

1
2
3
///////////////////////////////////////////////
   it++;
}

[loufoque]

Je viens de lire dans un post qu'en cas de remplacement total de la map, l'opérateur d'affection suffit...
C'est tellement simple... mais est ce le plus efficace?

C'est le mieux oui. Ça peut potentiellement être plus efficace, mais certainement pas moins.

[Sunsawe]

Salut,
merci pour vos réponses.
@Sylvain Togni :
En ce qui concerne l'utilisation de pair, c'est l'une des erreurs de syntaxe dont je parlais dans mon second commentaire.
Le véritable code que j'ai testé est:

Code :

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

map_destination.insert(std::pair<const std::string, std::string>(it->first, it->second));

Par contre je ne comprend pas pourquoi tu dis que la map_destination n'est pas définit dans la boucle.

@loufoque:
J'ai essayé le simple opérateur d'affectation = et l'erreur obtenue est la même.

--------------------------------------------------------------------------

En produisant le code minimal, j'ai trouvé que l'erreur provenait d'un const que j'avais oublié quelque part en faisant un copier coller....


Désolé pour le dérangement, mais merci tout de même pour le coup de main.