Discussion :

[Gebudi.]

Bonjour à tous,

je suis en train de m'entrainer en C++ et j'ai un soucis avec un exercices (je n'ai pas les corrigés).
Voilà mon code qui est une fonction membre de la classe "Abonne". Le programme est la gestion d'une bibliothèque. Les livres disposent de leur propre classe "Livre".

Code :

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

1
2
3
4
5
6
7
8
9
10
11
void Abonne::rendre_livre(Livre l)
{
     list<Livre>::iterator it;
 
     for (it = livres_empruntes.begin(); it != livres_empruntes.end(); it++)
         if (*it == l)
         {
            livres_empruntes.erase(it);
            break;
         }
}

Je pense que la fonction est claire, elle s'exécute lorsqu'un abonné doit rendre un livre et donc le retirer la liste de livres empruntés. livres_empruntes est donc une liste de "Livre". Le problème se situe au : if (*it == l) Le compilateur n'arrive pas à comparer ces deux éléments.

Voilà l'erreur:
no match for 'operator==' in '(&it)->std::_List_iterator<_Tp>::operator* [with _Tp = Livre]() == l'

Quelqu'un aurait une idée?
Merci

[boromir73]

Le message d'erreur te dit que tu dois définir l'opérateur '==' pour ta classe Livre. Un truc du genre :

Code :

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

1
2
3
4
 
bool operator==(const Livre& l1, const Livre& l2) {
 return l1.getId()==l2.getId(); // Tu dois expliquer comment comparer 2 objets de type Livre
}

Renseigne-toi sur la fonction std::find(), cela simplifiera ton code (si tu as du courage, tu peux même essayer std::remove_if()).

[Gebudi.]

Merci bien, je regarde ça.

[Gebudi.]

Merci ça marche. Je n'avais pas compris le message d'erreur.
J'ai quand même utilisé find:

Code :

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

1
2
3
4
5
list<Livre>::iterator it = find (livres_empruntes.begin(), livres_empruntes.end(), l);
     if (it != livres_empruntes.end())
     {
        livres_empruntes.erase(it);
     }

Et j'ai quand même du surcharger l'opérateur ==.

Merci encore

[Laurent Gomila]

Et pourquoi pas

Code :

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

livres_empruntes.remove(l);

Si ton livre est unique dans la liste, bien sûr.

[Eusebe]

Pour une list, on peut probablement se passer de remove_if, et se contenter de remove :

Code :

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

livres_empruntes.remove(l)

Mais tout dépend si tu veux supprimer tous les éléments égaux à l ou juste le premier...

[Gebudi.]

Ah ouais nickel avec remove.
Merci à tous.