Discussion :

[rozwel]

Je suis en train d'écrire un test unitaire qui est censé vérifier que deux structures d'objets sont identiques. J'ai redéfini les méthodes equals() et hashCode() de tous les noeuds de ma structure, mais quand je lance le test, il me dit juste que les deux structures ne sont pas égales.

Est-ce que quelqu'un connait un outil ou une méthode pour faire une espèce de diff entre deux structures d'objets?

[Baptiste Wicht]

Je suis en train d'écrire un test unitaire qui est censé vérifier que deux structures d'objets sont identiques. J'ai redéfini les méthodes equals() et hashCode() de tous les noeuds de ma structure, mais quand je lance le test, il me dit juste que les deux structures ne sont pas égales.

Est-ce que quelqu'un connait un outil ou une méthode pour faire une espèce de diff entre deux structures d'objets?

Montre nous ta méthode equals() qu'on voie s'il y a quelque chose à changer

[rozwel]

C'est une méthode equals standard générée par CommonClipse:

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
 
/**
     * @see java.lang.Object#equals(Object)
     */
    public boolean equals(Object object)
    {
        if (object == this) {
            return true;
        }
        if (!(object instanceof ProcedureDetail)) {
            return false;
        }
        ProcedureDetail rhs = (ProcedureDetail) object;
        return new EqualsBuilder()
                .appendSuper(super.equals(object))
                .append(this.sigle, rhs.sigle)
                .append(this.title, rhs.title)
                .append(this.associatedDocuments, rhs.associatedDocuments)
                .append(this.activeReferral, rhs.activeReferral)
                .append(this.baseDocument, rhs.baseDocument)
                .isEquals();
    }

[maxf1]

Et ta classe EqualsBuilder, et standard elle aussi?

Ce 'if' :

Code :

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

1
2
3
if (object == this) {
            return true;
        }

n'est à mon avis pas très utile, il ne sera vrai que dans un cas, si tu compares ton objet avec lui meme. A moins que j'ai zappé quelque chose.

[maxf1]

Ok je viens de voir le but de EqualsBuilder.
Vu que dans l'exemple ils font le test que j'ai dit qu'il ne servait à rien peut-etre sert-il mais je ne vois pas trop à quoi.

Je pense que ton probleme vient du faite que tu fais

Code :

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

.appendSuper(super.equals(object))

ProcedureDetail est bien une classe à toi? Elle "implements" ou "extends" quelque chose?

Peutx-tu mettre le code de procedureDetail ou au moins que l'on voit de quelle nature sont ces attributs.

[rozwel]

Et ta classe EqualsBuilder, et standard elle aussi?

Presque. Elle fait partie des Commons-Utils d'Apache.

Ce 'if' :

Code :

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

1
2
3
if (object == this) {
            return true;
        }

n'est à mon avis pas très utile, il ne sera vrai que dans un cas, si tu compares ton objet avec lui meme. A moins que j'ai zappé quelque chose.

Non c'est tout à fait à ça que ça sert. Mais juste pour être un peu plus précis. Cet object est lié à d'autres objets, eux-mêmes liés à d'autres, etc. Tout ça dans une structure assez complexe qui heureusement possède une racine unique. Et c'est en appelant equals sur cette racine que toute la structure est parcourue.

[rozwel]

Non le appendSuper() est justifié parce qu'effectivement ProcedureDetail est une classe à moi qui en étend une autre.

Au fait, je n'ai aucun problème avec la méthode equals en elle-même, elle fait très bien son boulot. C'est juste que quand j'exécute mon test unitaire pour vérifier l'égalité des deux structures, il me manque de l'info pour vérifier où se trouvent les différences.

[maxf1]

Ok. Donc tu es dans ton test unitaire, et il te renvoi faux alors qu'il devrait te renvoyait vrai, c'est bien ca ton problème?

EN faite je pensais que ProcedureDetail était toute simple et qu'elle héritait Object par exemple et que la méthode equals n'était pas bien défini sur celle si.

En fait pour ton test unitaire, le plus simple serait de faire ceci:

Code :

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

1
2
3
4
5
6
7
8
9
 
boolean res = true;
res = res && new EqualsBuilder().appendSuper(super.equals(object)).isEquals();
res = res && new EqualsBuilder().append(this.sigle, rhs.sigle).isEquals();
res = res && new EqualsBuilder().append(this.title, rhs.title).isEquals();                      
res = res && new EqualsBuilder().append(this.associatedDocuments, rhs.associatedDocuments).isEquals();
res = res && new EqualsBuilder().append(this.activeReferral, rhs.activeReferral).isEquals();
res = res && new EqualsBuilder().append(this.baseDocument, rhs.baseDocument).isEquals();
return res;

Ca te permettra de voir ou est ce que ton equals est faux.

[rozwel]

Difficilement envisageable pour des raisons à la fois pratiques et esthétiques.

Pratiques parce que j'ai environ une trentaine de classes dans mon modèle et que j'ai déjà galéré en générant automatiquement le equals() pour chacune de ces classes avec CommonClipse dans Eclipse, et en repassant sur chaque méthode générée pour éviter les références circulaires. Alors si je dois commencer à réécrire tous les equals à la main, j'ai pas fini.

Esthétiques parce que je perds tout l'intérêt d'utiliser EqualsBuilder à ce moment là.

Je vais essayer de trouver autre chose. Merci quand même.

[maxf1]

OUi mais en faite je voulais te faire réécrire cette methode uniquement pour savoir d'ou venait le probleme et de pouvoir mieux le cibler. Donc pour te donner une piste de ou cherchait "l'erreur".
Car si tu ne sais pas pourquoi il te renvoi faux c'est un peu génant.