comparer deux objets

Version imprimable

15/01/2007, 14h14
root76

comparer deux objets

Bonjour,

J'ai deux instances de la même classe à comparer, j'utilise equals() afin de comparer la valeur, or en prenant un exemple ou j'instancie mes deux objets avec les mêmes paramétres (cad ils sont égaux), equals me retourne false je comprends pas pourquoi
15/01/2007, 14h33
le y@m's

As-tu bien réécris la méthode equals() ? si oui montre nous son code.
15/01/2007, 14h33
adiGuba

Salut,

Il faut redéfinir la méthode equals() afin qu'elle prenne en compte tes conditions d'égalité.

Plus d'info : http://www.developpez.net/forums/sho...4&postcount=33

a++
15/01/2007, 14h39
root76

Non je l'ai aps encore surchargé la méthode equals car je croyais qu'il comparait les attributs
15/01/2007, 14h47
spekal

Le fonctionnement standard de la méthode equals, défini dans la classe Object, est simplement de comparer les références des objets.

Donc, même lorsque tu crées deux objets censés être identiques dans leurs composantes, equals renvoie faux, puisque les références ne sont evidemment pas les mêmes.

Il faut donc modifier ce comportement dans de nombreux cas. C'est ce que fait, par exemple, la classe String, qui redéfinit la méthode equals pour que, merci bien, deux chaînes identiques soit égales du point de vue de la méthode equals.

Attention : la méthode hashcode fonctionne en parrallèle avec la méthode equals. Donc, quand tu redéfinis la méthode equals il est de très bon ton de redéfinir également la méthode hashcode.

Quand à la façon de procéder il y a de la doc là dessus partout. Cependant, si tu as des questions, n'hésite pas.

voila mon code pour la methode equals, ça marche pour le moment, mais je voudrais savoir est ce que c 'est propre ça:

Code:

1
2
3
4
5
6
7
8
9
10
11
12
13
14
15
16
 public boolean equals(CommandeDAO obj) {
        // TODO Auto-generated method stub
        Field myField[] =CommandeDAO.class.getFields();
       try {
        for (int i = 0; i < myField.length; i++) {
            if (myField[i].get(this) != myField[i].get(obj)) {
                return false;
            }
        }
    } catch (Exception e) {
        // TODO: handle exception
        e.printStackTrace();
    }
       return true;
    }
}

15/01/2007, 18h27
spekal

NON !

1) Pourquoi utilises-tu la reflexivité ?... Tu ne connais pas les champs de ta classe ?... Tu voudrais définir une méthode equals générique ?... Méfie toi que, si la méthode Object ne l'a pas fait, c'est que peut être c'est plus difficile qu'on ne le pense...

2) Le paramètre de la méthode equals est de type Object.

3) Redéfinir la méthode equals pour n'utiliser que les opérations == ou != pour rechercher l'égalité entre deux attributs est un peu comme prendre un verre de bière pour boire du vin ; ça va pas terrible ensemble. Tu devrais faire à tout le moins if (myField[i].get(this).equals(myField[i].get(obj))).

4) Etc :D

Je t'aide un peu : Common Lang contient des classes utilitaires qui t'aideront à faire des equals génériques comme tu sembles vouloir le faire ; regarde leurs sources.

Citation:

Envoyé par root76

Non je l'ai aps encore surchargé la méthode equals car je croyais qu'il comparait les attributs

Juste pour jouer le chieur :lol:, j'aime bien ça !!!

Il ne faut pas confondre:

Code:

1
2
3
4
5
6
7
8
9
10
11
12
13
14
public class Surcharge {
    public void methodeSurchargee(int a) {
       // ...
    }
 
    public void methodeSurchargee(String chaine) {
       // ...
    }
 
    public void methodeSurchargee(int a, String chaine) {
        // ...
     }
 
}

qui est de la surcharge

et:

Code:

1
2
3
4
5
6
7
8
9
10
public class Parent {
    public void methodeSurchargee(int a) {
       // ...
    }
 
public class Fils extends Parent {
     public void methodeSurchargee(int a) {
        // ...
     }
}

qui est de la ré-écriture...

A+