Discussion :

[cashmoney]

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 :

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

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

[we.are.the.storm]

Bonjour,

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

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.

[Invité]

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 :

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

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

[adiGuba]

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++

[cashmoney]

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

Code :

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

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

[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.

[cashmoney]

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

[JohnNC]

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 ?

[adiGuba]

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++

[Uther]

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 :

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

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);
}