Discussion :

[karimot]

Bonjour tout le monde

je voudrais supprimer un élément depuis un vecteur

Voilà la methode que j'ai fai :

en c# je fai le même principe et ça marche ...

Code :

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

1
2
3
4
5
6
7
8
9
10
11
12
void Equipe::SuppimerJoueurs(Joueur const &j)
{
     for(size_t i=0;i<ListJoueurequipe.size();i++)
{
    if ( ListJoueurequipe[i].Getnum()=j.Getnum())
    {
    ListJoueurequipe.erase(i);
    }
 
     }
 
   }

Voilà les deux erreurs que je recois

Erreur2error C2664: 'std::_Vector_iterator<_Myvec> std::vector<_Ty>::erase(std::_Vector_const_iterator<_Myvec>)' : impossible de convertir le paramètre 1 de 'size_t' en 'std::_Vector_const_iterator<_Myvec>'

Erreur1error C2662: 'Joueur::Getnum' : impossible de convertir un pointeur 'this' de 'const Joueur' en 'Joueur &'

[Bousk]

Bonjour,

peut-être faudrait-il commencer par lire la doc de vector::erase ?
Où il apparait très clairement que erase annule l'itérateur supprimé et le rend inutilisable. Mais retourne l'itérateur suivant justement.
Et que si tu fais un it++ sur un itérateur supprimé... tu as vu le résultat.

[koala01]

Salut,

Fais aussi attention à ton test, car

Code :

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

if ( ListJoueurequipe[i].Getnum()=j.Getnum())

ne fait surement pas ce que tu crois (à condition que cela compile... en fonction du fait que GetNum est une fonction constante ou non et en fonction du type de retour (valeur ou référence ) de la fonction).

Dans le meilleur des cas (comprends : si GetNum est déclarée comme étant une fonction constante renvoyant une référence constante), le compilateur se plaindra de l'impossiblité de modifier la valeur.

Dans le pire des cas, (si GetNum est déclarée comme une fonction non constante et qu'elle renvoie une référence non constante), tu modifiera la valeur renvoyée par GetNum de tous les éléments de ListJoueurequipe, et ils seront tous supprimés

Entre les deux (fonction (constante ou non) renvoyant une valeur), le test vaudra true si j.GetNum est différent de 0. Ce qui risque d'arriver dans la très grande majorité des cas, et qui provoquera donc l'effacement de la très grosse majorité des éléments (pour ne pas dire de tous, selon que tu acceptes ou non qu'un joueur ait le numéro 0).

L'égalité se teste toujours avec l'opérateur == (deux fois le symbole '=' qui se suit), sinon c'est une affectation!!!

[oodini]

Pose-toi aussi la question de savoir si le vecteur est la structure de données la plus adaptée.

Par ailleurs, ta fonction est nommée SuppimerJoueurs, alors que je pense qu'un seul joueur sera supprimé (à moins que tu autorises 2 joueurs à voir le même numéro). Auquel cas, ta fonction est mal nommée (pluriel indésirable) et sous-optimisée (tu parcours les éléments de ton vecteur même après avoir trouvé le joueur en question).

Je te suggérerais plutôt d'utiliser un std::find(), suivi d'un erase().