soucis avec collection.contains

Bonjour a tous,
j'ai une ArrayList<MaClasse>
MaClasse a un attribu id et un attribu name avec les getters et setter qui vont avec
MaClasse a aussi cette methode

Code:

---

1 2 3 4 5 6 7 8 9 10 11 12 13

@Override public boolean equals(Object obj){ System.out.println("Comparaison de 2 objets"); MaClasse maClasse = (MaClasse) obj; if(maClasse.getName().equals(name)){ System.out.println("ils sont égaux"); return true; } else{ System.out.println("ils sont differants"); return false; } }

---

Mon soucis est que lorsque j'utilise la méthode contains sur mon arrayList celle ci n'execute pas ma méthode equals, le débug n'a rien donné non plus :-(
Si vous avez des pistes je suis preneur

On peut voir le code de MaClasse ?

Voici le code complet de MaClasse, elle est on ne peut plus simple

Code:

---

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

public class MaClasse {   private int id; private String name;   public MaClasse(){   }   public MaClasse(String name){ this.name = name; }   public MaClasse(int id, String name) { super(); this.id = id; this.name = name; }   public int getId() { return id; }   public void setId(int id) { this.id = id; }   public String getName() { return name; }   public void setName(String name) { this.name = name; }   @Override public boolean equals(Object obj){ System.out.println("Comparaison de 2 objets"); MaClasse objet= (MaClasse) obj; if(objet.getName().equals(name)){ System.out.println("ils sont égaux"); return true; } else{ System.out.println("ils sont differants"); return false; } } }

---

Le code de test :

Code:

---

1 2 3 4 5 6

Arraylist<MaClasse> maCollection; //chargement de ma collection grace a Hibernate if(maCollection.contains("un text recherché")){ //un traitement qui n'est jamais executé }

---

Mon code s'execute sans erreur, le soucis est que le programme n'entre jamais dans le if malgres les bonnes données vu dans le mode debug

Une remarque : il se pourrait que "name" soit null et ta méthode "equals" se planterait

Sinon, peux-tu également montrer ton code de test ?

dans getName et setName, tu as une variable pName qui correspond à rien et donc la méthode "equals" rend toujours false

Citation:

---

Envoyé par OButterlin

Une remarque : il se pourrait que "name" soit null et ta méthode "equals" se planterait

Sinon, peux-tu également montrer ton code de test ?

---

name ne peut etre null

Si, comme ceci

Code:

---

1 2	MaClasse c = new MaClasse();

---

Ton test est à l'envers

Code:

---

1 2 3 4 5

if (maCollection.contains("un text recherché")) { //un traitement qui n'a jamais été executé }

---

il faudrait

Code:

---

1 2 3 4 5

if ( ! maCollection.contains("un text recherché")) { //un traitement qui n'a jamais été executé }

---

(si c'est bien un traitement à faire une seule fois que tu veux)

merci pour la réponse,
j'avais déjà essayé la solution que tu m'a donné mais cela donne toujours le même résultat, le test est fait comme si j'avais pas ré-écris la methode equals. Je me demandais s'il fallait pas que j'implémente une interface ou que j’hérite d'une classe meme si normalement cela devrait fonctionner sans

Salut,


Ce comportement est normal puisque tu ne passes pas un objet de type MaClasse mais un objet de type String :

Code:

---

1 2 3

maCollection.contains("un text recherché") // Il faut faire : maCollection.contains(new MaClasse("un text recherché"))

---

Je suppose que contains() doit utiliser la méthode equals() de l'objet reçu en paramètre...


a++

Citation:

---

Envoyé par adiGuba

Je suppose que contains() doit utiliser la méthode equals() de l'objet reçu en paramètre...

---

Dans le cas d'une ArrayList, oui. Ça va de soi et c'est ce que dit la doc. (Correction : la doc ne le dit pas mais, par élimination, on peut logiquement conclure qu'il n'existe pas d'autre moyen de faire ce que dit la doc.)

Merci adiGuba ta solution est la bonne !