Souci pour comparer deux objets avec equals()

Version imprimable

Bonjour,

j'ai réécrit une méthode equals pour pouvoir comparer deux objets, mais lorsque que je fais un test cela me retourne toujours que les objets sont identiques.

La méthode equals :

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
 
private String nom;
    private String prenom;
    public enum Sexe {
        Masculin, Feminin;
    }
    private Sexe sexe;
 
 
    public Individu(String leNom, String lePrenom, Sexe leSexe){
        nom = leNom.toUpperCase();
        StringBuffer sb= new StringBuffer(lePrenom);
        sb.setCharAt(0,Character.toUpperCase(lePrenom.charAt(0)));
        String str=sb.toString();
        prenom=str;
        sexe = leSexe;
    }
 
 public boolean equals(Object obj){
        boolean tempo=false;
        if(obj instanceof Individu){
            Individu test=(Individu)obj;
            tempo=(this.nom==test.nom)&&(this.prenom==test.prenom)&&this.sexe.equals(test.sexe);
 
        } 
            return tempo;
 
    }

J'aimerais bien savoir ce qui ne va pas dans mon code.

Merci d'avance.

Bonjour.
Déjà, il faut commencer par laisser tomber la comparaison des chaines via l'operateur == et utiliser plutôt le equals.
FAQ : Comparaison des chaines

Sinon, tu peux aussi faire un petit peu de débogage, du genre :
Code:

1 2 3 4 5 6 7 8 9 10 public boolean equals(Object obj){ boolean tempo=false; if(obj instanceof Individu){ Individu test=(Individu)obj; System.out.println(this.nom + "=="+ test.nom + " : " + this.nom.equals(test.nom); //tu fais de même pour les autres champs : : }
ça devrait vite te permettre de voi là où ça bloque.
FAQ : equals

Bonne chance.

Je ne peux pas modifier ma classe comme vous le suggérer car j'ai deux fichiers : le premier ou se trouve la classe equals et un second ou il y a les test et la méthode main.

J'ai suivi la FAQ et modifié comme suit, mais je comprends pas pourquoi cela me retourne toujours que les deux objets sont identiques , j'ai du loupé quelque chose :

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
 
public boolean equals(Object obj){
        boolean tempo=false;
        if(obj instanceof Individu){
            Individu test=(Individu)obj;
            if (this.nom != test.nom) {
                // Si les références ne sont pas identiques
                // on doit en plus utiliser equals()
                if (this.nom == null || !this.nom.equals(test.nom)) {
                    return false; // les attributs sont différents
                }
            }
 
            if (this.prenom != test.prenom) {
                // Si les références ne sont pas identiques
                // on doit en plus utiliser equals()
                if (this.prenom == null || !this.prenom.equals(test.prenom)) {
                    return false; // les attributs sont différents
                }
            }
 
            if (this.sexe != test.sexe) {
                // Si les références ne sont pas identiques
                // on doit en plus utiliser equals()
                if (this.sexe == null || !this.sexe.equals(test.sexe)) {
                    return false; // les attributs sont différents
                }
            }
 
            // Si on arrive ici c'est que tous les attributs sont égaux :
            return true;
 
        }
        return false;
    }

30/09/2007, 08h28
pracede2005

Peux-tu nous poster ton main stp!
30/09/2007, 10h25
adiGuba

Salut,

Citation:

Envoyé par pracede2005

Peux-tu nous poster ton main stp!

+1

Sinon pour les enums tu peux te contenter d'une comparaison avec == puisque chaque enum ne peut être instanciée qu'une seule fois (grosso modo equals() se contente d'utiliser ==)

a++

Voila mon main :

Code:

1
2
3
4
5
6
7
8
9
10
11
12
13
14
15
16
17
18
 
public static void main(String[] args) {
        Individu moi = new Individu("dubois","benoit",Individu.Sexe.Masculin);
        Individu oim = new Individu("durand","marcel",Individu.Sexe.Masculin);
        Individu chepa = new Individu("cassel","vanessa",Individu.Sexe.Feminin);
 
        if(moi.equals(chepa)) {
           System.out.println("Les individus sont identiques") ;
        }
        else {
            System.out.println("Les individus sont différents") ;
        }
 
 
 
 
 
    }

30/09/2007, 13h14
adiGuba

Es-tu sûr que tu exécutes bien le code que tu as posté ? Car je ne reproduit pas le problème.

a++
30/09/2007, 16h41
xillibit

Oui, maintenant cela fonctionne

merci de votre aide