Comparer deux structures d'objets

Version imprimable

25/01/2007, 09h41
rozwel

Comparer deux structures d'objets

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?
25/01/2007, 09h44
Baptiste Wicht

Citation:

Envoyé par 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?

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

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

Code:

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

25/01/2007, 10h15
maxf1
Et ta classe EqualsBuilder, et standard elle aussi?

Ce 'if' :
Code:

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. :roll:
25/01/2007, 10h19
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:

.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.
25/01/2007, 10h23
rozwel
Citation:

Envoyé par maxf1

Et ta classe EqualsBuilder, et standard elle aussi?

Presque. Elle fait partie des Commons-Utils d'Apache.
Citation:
Envoyé par maxf1

Ce 'if' :

Code:

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. :roll:
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.
25/01/2007, 10h25
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.

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:

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.

25/01/2007, 10h42
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.
25/01/2007, 10h45
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. :?