Discussion :

[doudou87]

slt,je suis sur un mini projet de distributeur de boisson,j'ai essayé de faire une map qui contient comme clé le boisson et comme valeur sa quantité dans le stock,or j'ai eu des erreurs.
code source:

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
bool stock_boissons::retirer_boisson(string mark)
{
	ItB it;
 
for(it=stokBoissons.begin();it!=stokBoissons.end();it++)
{
 
	if(*(it).first.getMarque()==mark)
	{
		stokBoissons.erase(it);
		return true;
 
	}
}
return false;
}
 
void stock_boissons::consulter_stockBoissons()
{
if(!stokBoissons.empty())
{
 
ItB it;
for(it=stokBoissons.begin();it!=stokBoissons.end();it++)
{
	cout<<"la marque="<<(*it).first.consulter_boisson()<<"  "<<(*it).second<<endl;
}
}
}

les erreurs:
error C2039: 'first' : is not a member of 'iterator'.
error C2662: 'consulter_boisson' : cannot convert 'this' pointer from 'const class boisson' to 'class boisson &' Conversion loses qualifiers.
j'attend votre aide
merçi d'avance

[loufoque]

Les messages d'erreur sont explicites.

Sinon ton retirer_boisson est assez risible : une recherche linéaire dans une map...

[Invité]

Le premier message provient de ta ligne

Code :

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

if(*(it).first.getMarque()==mark)

*(it).first est interprété par le compilateur comme *(it.first), et celui ci te répond que l'itérateur it (un pointeur, si tu veux) n'a pas de membre first (ce sur quoi il pointe, en revanche, l'entrée dans ta map, a un membre first).

Il faudrait donc écrire

Code :

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

(*it).first

mais on fait plus souvent

Code :

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

it->first

ca évite toute confusion... Note d'ailleurs que tu fait bien (*it).first quelques lignes plus bas, dans la fonction consulte_stockboisson().

Francois

[3DArchi]

Bonjour,
Pour compléter la réponse de loufoque sur retirer_boisson, sache que std::map possède une méthode find qui te renvoie l'itérateur de la clé donnée en paramètre ou stokBoissons::end() s'il n'y a pas d'entrée correspondante. La recherche est plus performante que la recherche linéaire que tu fais. Cela fait parti de l'intérêt des conteneurs.