Discussion :

[bowow]

Bonjour à tous,

je rencontre des problèmes pour insérer des données avec un it.first déjà existant dans une map. J'ai essayé de synthétiser et simplifier au maximum le besoin.

Exemple:

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
37
38
39
40
41
#pragma once
 
#include <string>
#include <vector>
#include <map>
 
struct example
{
	int id;
	std::string name;
};
 
struct data
{
	int number;
	std::string string;
};
 
class c
{
	private:
	std::map<example, std::vector<data>> m;
 
	public:
	void add(example &ex)
	{
		data d;
		bool state = false;
		for (const auto& its : m)
		{
			if (its.first.id == ex.id)
			{
				m[its.first].push_back(d);
				//m_.emplace(std::make_pair(its.first, d));
				state = true;
				break;
			}
		}
		if (!state) m.emplace(std::make_pair(ex, d));
	}
};

Merci bien.

[Bousk]

Quel est le problème ?
Pourquoi ne pas simplement faire m[ex.id].push_back(d); ? L'opérateur [] de la map commence par créer la valeur si elle n'existe pas.

[bowow]

Bonjour Bousk,

ça ne compile pas ! et les erreurs ne sont pas du tout explicites et me renvoient dans xutility et xstddef.

Le but étant de faire un test sur un champ (id) et de stocker l'intégralité de la structure dans la map.

En fait je ne vois rien d'anormal, je me demande si le soucis ne viendrait pas d'ailleurs.

[bacelar]

les erreurs ne sont pas du tout explicites

Pour vous.
Laissez-nous en juger.
Les messages d'erreur, SVP.

[ternel]

pour moi, le problème pourrait bien être que tu n'as pas défini operator <, au moins dans le code que tu nous montres.

[Danny-k]

Si tu veux stocker ta class example dans ta map, il faut que tu définise un opérateur de comparaison (à mettre en fonction libre):

Code :

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

1
2
3
4
5
6
 
//trie sur l'id
bool operator< (const example & e1, const example & e2)
{
	return e1.id < e2.id;
}

Et tu peux utiliser ta map comme ceci :

Code :

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

1
2
3
4
5
6
7
 
std::map<example, std::vector<data>> m;
example e{1, "toto"};
 
data d{5, "titi"};
 
m[e].push_back(d);

[bowow]

Bonjour,

En effet, le problème viendrait de l'absence de la surcharge d'opérateur.

:\program files (x86)\microsoft visual studio 14.0\vc\include\xstddef(239): error C2678: '<' binaire*: aucun opérateur trouvé qui accepte un opérande de partie gauche de type 'const Event' (ou il n'existe pas de conversion acceptable)
1> c:\program files (x86)\microsoft visual studio 14.0\vc\include\system_error(436): note: est peut-être 'bool std::operator <(const std::error_condition &,const std::error_condition &) noexcept'
1> c:\program files (x86)\microsoft visual studio 14.0\vc\include\system_error(427): note: ou 'bool std::operator <(const std::error_code &,const std::error_code &) noexcept'
1> c:\program files (x86)\microsoft visual studio 14.0\vc\include\xstddef(239): note: lors de la tentative de mise en correspondance de la liste des arguments '(const Event, const Event)'
1> c:\program files (x86)\microsoft visual studio 14.0\vc\include\xstddef(238): note: lors de la compilation de la fonction membre '<Inconnu>' de la classe <Inconnu>
1> c:\program files (x86)\microsoft visual studio 14.0\vc\include\xutility(812): note: voir la référence à l'instanciation de la fonction modèle 'bool std::less<_Kty>::operator ()(const _Ty &,const _Ty &) const' en cours de compilation

Oui mais le fait de créer une fonction libre aura forcément de gros impacts sur la solution, il faut prendre en compte les effets de bords.. et ça ne compile toujours pas (erreurs de linkage).