prob parcours map clé pointeur

Boonjour à tous et bonne année,

J'aimerais avoir votre aide concernant un problème auquel je suis confronté actuellement en c++.
J'ai une map définie de la facon suivante :

Code:

---

1 2	typedef map<Quantite*, Ingredient*> quantite_ingredient_map;

---

Dans ma classe Recette j'ai un attribut de ce type :

Code:

---

1 2	quantite_ingredient_map _m;

---

Pour ajouter dedans j'utilise cette méthode :

Code:

---

1 2 3 4

void Recette :: ajouter_composant(Quantite* q,Ingredient* i){ _m.insert(make_pair(q,i)); }

---

Mon problème survient lorsque je souhaite parcourir ma map de cette manière :

Code:

---

1 2 3 4 5 6 7

if( !_m.empty() ) { quantite_ingredient_map::iterator itt; for (itt = _m.begin(); itt != _m.end(); itt++) { ((*itt).first).afficherDec(os,1); } }

---

J'ai le droit d'avoir cette magnifique erreur :

Code:

---

1 2	no match for 'operator=' in 'itt = (((const quantite_ingredient_map*)((const Recette*)this)) + 20u)->std::map<_Key, _Tp, _Compare, _Alloc>::begin [with _Key = Quantite*, _Tp = Ingredient*, _Compare = std::less<Quantite*>, _Alloc = std::allocator<std::pair<Quantite* const, Ingredient*> >]()'

---

Là je suis complètement bloqué depuis 2h et je ne comprends pas d'où peut venir ce problème.

Merci d'avance à tous ceux qui prendront le temps de me lire et de me répondre.

Louis

Bonjour,
pour commencer une map où la clef et un pointeur... ca ne doit etre ce que tu veut faire.... Il va etre trié en fonction de l'adresse du pointeur....

sinon,
(*itt).first est un pointeur donc :

Code:

---

1 2 3 4

quantite_ingredient_map::iterator itt; for (itt = _m.begin(); itt != _m.end(); itt++) (*itt).first->afficherDec(os,1);

---

Oui mais le tri n'est pas une priorité, il me faut jsute une liste d'ingrédient associé à des quantités.

Citation:

---

Envoyé par allserv

Oui mais le tri n'est pas une priorité, il me faut jsute une liste d'ingrédient associé à des quantités.

---

Comme le trie ne te sert a rien, utilise un vector de pair. La si tu veut associé deux Ingredient a une même instance d'une Quantite ca ne marchera pas.

Code:

---

std::vector< std::pair<Quantite*, Ingredient*> > quantite_ingredient_association;

---

Pourquoi doit tu utiliser des pointeur???

J'utilise des pointeurs car en fait il est impossible d'instancier un objet de type Quantité, il s'agit d'une classe qui est utilisée pour dériver tous les autres types (gramme, litre etc..). Or si je me souviens bien des erreurs que j'ai pu avoir, il était impossible de passer par référence un objet de type Quantité car il fallait qu'il soit instancié.

Je me trompe ?

Citation:

---

Envoyé par allserv

J'utilise des pointeurs car en fait il est impossible d'instancier un objet de type Quantité, il s'agit d'une classe qui est utilisée pour dériver tous les autres types (gramme, litre etc..). Or si je me souviens bien des erreurs que j'ai pu avoir, il était impossible de passer par référence un objet de type Quantité car il fallait qu'il soit instancié.

Je me trompe ?

---

Si gramme, litre, ... sont des class enfants de quantités,oui tu doit faire ca. Peut etre dans boost il y as des choses plus propre.
Par contre ca m'étonnerai que map soit ce qu'il te faut

Es ce qu'un enum ne serait pas mieux pour les quantité??

Code:

---

1 2 3 4 5 6

enum Quantite { litre, gramme, ... }

---

Je vais t'avouer que le schéma UML des classes est déjà donné et donc l'implémentation est déjà choisie :) Ce doit être des classes dérivées.

Un vector de Pair m'a l'air pas trop mal en fait mais j'aurais bien aimé avoir la solution pour l'iterator :oops:

Merci encore de ton aide

Ne voulais-tu pas plutôt utiliser une multimap avec comme clef une Quantite* ?

Pouvoir récupérer tous les ingrédients dont la quantité est égale à...

Pour que ta multimap fonctionne avec un pointeur comme clef, il faudra modifier son prédicat de comparaison (plein d'exemples donnés en ce moment sur le forum avec std::sort, std::set etc.)

Citation:

---

Envoyé par allserv

Code:

---

1 2 3 4 5 6 7

if( !_m.empty() ) { quantite_ingredient_map::iterator itt; for (itt = _m.begin(); itt != _m.end(); itt++) { ((*itt).first).afficherDec(os,1); } }

---

---

La méthode qui appelle ce code serait-elle déclarée const ?
Dans ce cas là utilise plutôt un const_iterator.

Au passage :

Code:

---

((*itt).first)->afficherDec(os,1);

---

et :

Code:

---

++itt

---

Citation:

---

Envoyé par NiamorH

Ne voulais-tu pas plutôt utiliser une multimap avec comme clef une Quantite* ?

Pouvoir récupérer tous les ingrédients dont la quantité est égale à...

Pour que ta multimap fonctionne avec un pointeur comme clef, il faudra modifier son prédicat de comparaison (plein d'exemples donnés en ce moment sur le forum avec std::sort, std::set etc.)

---

A la limite une multimap pourquoi pas c'est vrai. Mais si le trie ne lui sert a rien...faut mieux un vector.

D'ailleurs il faudrait qu'il regarde:
http://cpp.developpez.com/faq/cpp/?p...hoix_conteneur

L'iterator const ainsi que l'accès aux fonctions via la -> a résolu le problème, cela fonctionne.

Merci encore pour votre patience

Louis

Si il n'est pas nécessaire de répondre à cette question :

Citation:

---

Pouvoir récupérer tous les ingrédients dont la quantité est égale à...

---

autant avoir recours à un vector c'est sur.

Citation:

---

Envoyé par Mongaulois

sinon,
(*itt).first est un pointeur donc :

Code:

---

1 2 3 4

quantite_ingredient_map::iterator itt; for (itt = _m.begin(); itt != _m.end(); itt++) (*itt).first->afficherDec(os,1);

---

---

J'avais pas vu cette partie de ton post :lol: