Discussion :

[julien25]

Bonjour à tous,

Dans le but d'effectuer une copy en C++, j'aimerais créer une map qui va contenir ma map que je veux copier. J'aimerais donc iterer sur deux map simulatement.

Dans l'idée ca donne :

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
 
 
 
if(map_a_remplir!= NULL)
	{
		map_a_remplir->clear();
	}
	else
	{
		map_a_remplir= new MapOfStringDouble();
	}
for ((MapOfStringDouble::iterator iter_ret =map_a_remplir->begin(); iter_ret !=map_a_remplir->end(); iter_ret ++) && (MapOfStringDouble::iterator iter =map_a_copier->begin(); iter!=map_a_copier->end(); iter++))
	{
		iter_ret->first = iter->first;
		iter_ret->second= iter->second;
	}

Auriez vous une idée?
Merci à tous.

[Mat007]

Salut,

Une sorte de pari ne connaissant pas les types exacts de tes maps :

Code :

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

*map_a_remplir = *map_a_copier;

?
Au passage c'est bizarre de manipuler des maps par pointeurs...

MAT.

[julien25]

Salut,

non ce que tu proposes est un copie shadow alors que je veux faire une copie profonde, c'est à dire prendre ce qu'il y a à l'interieur des pointeurs.

Une precision mes map sont de types : map<string, double>

[Mat007]

(...) c'est à dire prendre ce qu'il y a à l'interieur des pointeurs.

D'où le déréférencement avec * non ?

MAT.

[julien25]

Salut,

Je pense que ce que tu proposes doit en partie marcher avec le dereferencement.
Cela dit, je préfère copier chaque partie de ma map (first, seconde) comme je l'ai dit plus haut.
Par ailleurs ca me permettrais de comprendre comment faire ce double iterateur.

Merci encore.

[Mat007]

Mais map_a_remplir est vide, il n'y a rien à itérer...
Si vraiment tu veux le faire à la main :

Code :

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

1
2
for( MapOfStringDouble::iterator iter = map_a_copier->begin(); iter != map_a_copier->end(); ++iter )
  map_a_remplir->insert( *iter );

Mais bon c'est un peu absurde dans la mesure où une affectation directe fait au pire la même chose...

MAT.
ps : et oui, ça copie complètement vraiment profondément tout à fait le contenu

[koala01]

Salut,

D'autant plus que, si tu déclare la map à copier au moment opportun, tu peux même directement passer par le constructeur par copie:

Code :

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

std::map<key, value> map_a_remplir(map_a_copier);

est justement fait pour que tout se passe correctement

Ceci dit, et juste pour être sur d'avoir toutes les cartes en main pour t'aider efficacement...
Je présumes que MapOfStringDouble est en réalité un typedef de std::map utilisant std::string comme clé et double comme valeur, mais est-ce réellement le cas

Pourquoi passer par un pointeur sur la map et par l'allocation dynamique

Est-ce juste par masochisme, ou y a-t-il une raison particulière

[Alp]

A noter qu'un std::for_each utilisant le foncteur back_inserter règleraient tout aussi bien le problème. Ainsi que std::copy. Et bien d'autres façons, la plus simple étant le constructeur de copie.

Si tu veux programmer correctement en C++ moderne, ne met des pointeurs que quand tu es obligé, ce qui n'est probablement pas le cas lorsque tu travailles sur des std::map, mais il nous faudrait en savoir plus pour en être sûr.

[julien25]

Merci pour vos réponses.

Première chose : il n' y a aucune raison particulière d'utiliser des pointeurs sur des map. J'avais commencer de la sorte en me disant que ce serait mieux mais sans vraiment comprendre pourquoi. J'ai par contre fait très attention à tout mes delete. Je ferais attention à ce point pour mes futurs softs. Maintenant ce programme est quasi fini et ca me prendrait bcp trop de temps de tout remplacer. (qq 100taines de map)...Est-ce que c'est vraiment important??

Deuxième chose :

Suivant tout vos conseils, j'ai finalement fait comme Mat l'avait suggeré :

Code :

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

1
2
3
4
5
 
        MapOfStringDouble* tmpDict;
	this->myObject->myfunction(tmpDict);
	this->Map_a_remplir->clear();
	(*Map_a_remplir) = (*tmpDict)

PS : Ma Map_a_remplir est declare en global.
myfunction est une methode de myOject est prnd en parametre la ma rempli.

koala01 oui c'est un typdef avec une map de string et double.

Merci encore
++

[koala01]

Merci pour vos réponses.

Première chose : il n' y a aucune raison particulière d'utiliser des pointeurs sur des map. J'avais commencer de la sorte en me disant que ce serait mieux mais sans vraiment comprendre pourquoi. J'ai par contre fait très attention à tout mes delete. Je ferais attention à ce point pour mes futurs softs. Maintenant ce programme est quasi fini et ca me prendrait bcp trop de temps de tout remplacer. (qq 100taines de map)...Est-ce que c'est vraiment important??

Heuu... oui et non...

C'est important dans le sens où cela te libère de tous les soucis qui apparaissent systématiquement lorsque tu manipule des pointeurs et que tu as recours à l'allocation dynamique.

Mais, évidemment, c'est une chose à prendre en compte... au début du programme...

Et, si le programme est presque terminé, que tu as la certitude de n'avoir ni double libération de la mémoire ni fuites mémoire, il est un peu tard pour dire quoi que ce soit