Discussion :

[root76]

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

[le y@m's]

As-tu bien réécris la méthode equals() ? si oui montre nous son code.

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

[root76]

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

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

[root76]

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

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

[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

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.

[thibaut]

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

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

Il ne faut pas confondre:

Code :

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

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 :

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

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+