Egalité entre 2 objets (equals)

Bonjour, je voudrais écrire une méthode equals qui dit que 2 salariés sont égaux s'ils ont le même nom et le même numéro de matricule.

Code:

---

1 2 3 4

public boolean equals(Object ob) { return ob instanceof Salarie && ((Salarie) ob).matricule && ((Salarie) ob).nom;

---

Et dans eclipse, il me souligne toute la ligne en rouge et mentionne The operator && is undefined for the types boolean, int.

j'arrive pas à comprendre merci

Bonjour,

Il faut que tu teste les egalites des attributs entre ton instance courante et celle passée en paramètre.

Citation:

---

public boolean equals(Object ob) {
return ob instanceof Salarie && ((Salarie) ob).matricule.equals(this.matricule)
&& ((Salarie) ob).nom.equals(this.nom);

---

Il faudra remplacer equals par == si l'attribut est un type de base.
Attentionégalement à tester quand les attributs sont null, ici si le matricule de l'instance passée en paramètre est null par exemple tu auras une exception.

Salut,
Je suppose que matricule est un int et nom un String... Ici tu n'effectues aucun test sur tes valeurs, que veux dire "int et string" ? Je pense que tu veux tester si le matricule et le nom de l'autre salarié sont égaux aux matricule et nom de ce salarié, non ?
Je ferais comme ca :

Code:

---

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

public boolean equals(Object ob) { if(ob==this) return true; if(ob == null || !(ob instanceof Salarie)) return false;   Salarie other = (Salarie)ob;   return other.matricule == this.matricule && other.nom.equals(this.nom);   }

---

De plus surcharge la méthode hashCode pour être sûr que tu puisses faire des tests d'égalité correct avec des Set si besoin, voir le contrat ici
Bon courage

Salut,


A noter que la FAQ propose un modèle pour cela : Pourquoi et comment redéfinir la méthode equals() ?

A noter que lorsqu'on redéfini equals() il est plus propre d'en faire de même pour hashCode()...

a++

C'est tout a fait normal, tu ne compares pas le matricule et le nom à la valeur courante de l'objet. Il te faut faire:

Code:

---

1 2 3 4 5

public boolean equals(Object ob) { return ob instanceof Salarie && ((Salarie) ob).matricule == this.matricule && ((Salarie) ob).nom.equals(this.nom); }

---

Merci, pour vos reponses et j'ai aussi pensé à ça

Code:

---

1 2 3 4

public boolean equals(Object ob) { Salarie sal =(Salarie)ob; return (sal.nom == this.nom && sal.matricule == this.matricule);

---

ça marche ou dois-je absolument faire des testes de if avant?
merci

Je suppose que nom est un String il faut donc utiliser .equals(...) et non == pour faire la comparaison.

Pour ce qui est du test if(ob == null || !(ob instanceof Salarie)). Il permet d'eviter une ClassCastException si l'objet comparé n'est pas de la Classe Salarié ou d'une classe héritée de Salarie. Il evite également une NullPointerException si on compare à null.

Merci Uther, j'ai très bien compris. Merci

Citation:

---

Envoyé par Uther

Je suppose que nom est un String il faut donc utiliser .equals(...) et non == pour faire la comparaison.

Pour ce qui est du test if(ob == null || !(ob instanceof Salarie)). Il permet d'eviter une ClassCastException si l'objet comparé n'est pas de la Classe Salarié ou d'une classe héritée de Salarie. Il evite également une NullPointerException si on compare à null.

---

Heu juste pour information, il me semble que "obj instanceof Class" renvoie false si obj est null, et non pas un NullPointerException, je me trompe ?

Citation:

---

Envoyé par JohnNC

Heu juste pour information, il me semble que "obj instanceof Class" renvoie false si obj est null, et non pas un NullPointerException, je me trompe ?

---

Tu ne te trompes pas c'est tout à fait cela ;)

a++