Discussion :

[Nyko17]

Bonjour/soir

Je voudrais faire une fonction qui demande le nom d'un element d'une std::list et qui supprime cet element.
J'ai donc fais ceci: La fonction supprimeVin() appel la fonction supprime(), qui se sert de analyseNom().

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
42
43
44
45
46
47
48
49
50
51
 
 
void ListeVins::supprimeVin()
{
    std::cout << "Entrez le nom du vin a supprimer" << '\n';
    std::string leNom;
    std::cin >> leNom;
 
     supprime(leNom);
 
    std::list<Vin>::iterator iter;
    std::ofstream uneListeVins("list.txt");
    uneListeVins << sesVins.size() << '\n';
    uneListeVins << '\n';
 
    for(iter=sesVins.begin(); iter!=sesVins.end(); iter++)
    {
        toStringF(uneListeVins,(*iter));
    }
 
    std::cout << "Vin " << leNom << " supprime" <<'\n';
    return;
 
}
 
 
 
void ListeVins::supprime(std::string nom)
{
    std::list<Vin>::iterator ite;
    for(ite=sesVins.begin(); ite!= sesVins.end(); ite++)
                {
                    if( analyseNom(*ite,nom))
                    sesVins.erase(ite);
                }
    return;
}
 
 
 
bool ListeVins::analyseNom(Vin v,std::string & nom)
{
    bool reponse;
 
    if( v.getLeNom()==nom)
    reponse=true;
 
    else reponse=false;
 
    return reponse;
}

Ca compile mais ça marche pas. J'ai beau chercher je ne trouve pas pourquoi.
J'ai testé de rajouter des std::cout à différents endroits, et apparemment le bug arrive apres la fonction erase() dans la boucle for.

Voilà, merci de votre aide.

[koala01]

Salut,

Tu devrais te tourner vers la méthode remove_if et les prédicats.

Voici un peu de lecture (en anglais), afin de te permettre de comprendre le principe

[Nyko17]

J'avais dejà essayé avec remove_if mais ça plantait aussi.

[Luther13]

Normal.
La boucle for suppose que la taille de la liste ne change pas.

Tu devrais réinitialiser ta boucle à chaque fois qu'un élément est effectivement supprimé.

[yan]

Salut
http://cpp.developpez.com/faq/cpp/?p...ssion_elements
http://r0d.developpez.com/articles/algos-stl/#LII-B-7

[Nyko17]

Merci ça marche. Mais je suis finalement confronté à un autre problème.
Si 2 elements ont le même nom, la fonction supprime() supprimera le 1er des 2.
Il faudrait donc plutot que je numerote les elements de la liste et que je demande le numero à l'utilisateur.
J'aurais pensé à un enum, mais ça me semble difficilement fesable etant donné que le nombre d'element n'est pas fixe, je ne vois donc pas comment je pourrais les déclarer.

EDIT: resolu