Discussion :

[xillibit]

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 :

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

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.

[djo.mos]

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 :

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

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.

[xillibit]

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 :

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

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

[pracede2005]

Peux-tu nous poster ton main stp!

[adiGuba]

Salut,

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

[xillibit]

Voila mon main :

Code :

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

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

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

[xillibit]

Oui, maintenant cela fonctionne

merci de votre aide