Discussion :

[Devlin111]

Bonjour,

J'utilise la méthode contains sur une liste d'objets, mais cela ne fonctionne pas.
En cherchant un peu, je me suis aperçu que c'était sûrement du au fait que je n'ai pas redéfini les méthodes equals et hascode.
Pour equals, je l'ai faite, mais comment redéfinir hashcode?

Merci

[Invité]

Soit plus précis.

Je n'ai pas compris grand chose à ton probleme

[spottt]

Es-tu sûr que cela vient du fait que tu n'aies pas redéfini ces méthodes?

Je demande cela car je n'ai réellement jamais eu de souci avec Contains()...

[Devlin111]

J'ai une liste de langue. Ma classe Langues ne contient pas grand chose : nom, niveau et les getters/setters pour ces deux propriétés.

Je veux demander a un user quelles langues il parle ... donc dans un formulaire, j'ai une dropdownlist avec les langues, et l'user peut choisir une langue, cliquer sur le bouton "ajouter langue" et lorsqu'il fait ceci, je rajoute à la liste de langues. Mais je veux vérifier que la liste ne contient pas deja la langue en question en faisant

Code :

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

if (user.sesLangues.Contains(new Langues(dropdown1.SelectedItem.Text)))

sesLangues est la liste de langues (qui ne contient que le nom, pas le niveau)
et je compare avec la langue choisie dans la dropdownlist.
J'ai aussi essayé comme ca :

Code :

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

1
2
Langues l1 = new Langues (dropdown1.SelectedItem.Text);
if (user.sesLangues.Contains(l1))

Lorsque je teste et que j'ajoute deux fois la même langue, ça ne passe pas dans le if.

Merci

[spottt]

Un contains ne renvoit true que si les deux objets testés sont équivalents.

Tu nous dis que ta classe Langue a deux propriétés (nom et niveau), mais lorsque tu fais ton Contains tu instancies une langue uniquement via un nom.

Le problème pourrait venir de là...

[SoBaKa]

Bonjour,

Le "Contains" ne va vérifier uniquement les références des objets contenus dans cette liste... Toi tu instancies un nouvel objet de type "Langues", donc à tous les coup cette méthode te renverra false...

Dans ton cas, je passerais par linq :

Code :

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

1
2
3
4
5
6
7
8
if(user.sesLangues.Any(l => l.Nom == dropdown1.SelectedItem.Text))
{
  // langue est déjà dans la liste
}
else 
{
  // OK on peut ajouter...
}

[Devlin111]

Non, car j'ai précisé que dans cette liste ci il n'y a pas le niveau.
J'ai deux constructeurs, un qui prend nom + niveau et l'autre uniquement le nom.
J'utilise le 2e ici.

Je ne vois pas pourquoi ça ne marche pas, à part cette histoire de hashcode que je ne sais pas régler ...

[Devlin111]

SoBaKa, merci

Ça marche !!