Substitution de la java de la méthode equals () - pas de travail?

J'ai couru dans une intéressante (et très frustrant) problème avec le equals() méthode aujourd'hui, ce qui a causé ce que je pensais être un bien testé la classe de crash et de provoquer un bug qui m'a pris un temps très long à repérer.

Juste pour être complet, je n'étais pas à l'aide d'un IDE ou un débogueur - juste de la bonne vieille éditeur de texte et du Système.out. Le temps était très limité et que c'était un projet d'école.

De toute façon -

J'ai été l'élaboration d'une base panier qui pourrait contenir un ArrayList de Book objets. Afin de mettre en œuvre les addBook(), removeBook(), et hasBook() méthodes de la Charrette, je voulais vérifier si le Book existait déjà dans le Cart. Donc me voilà -

public boolean equals(Book b) {
    ... //More code here - null checks
    if (b.getID() == this.getID()) return true;
    else return false;
}

Tous fonctionne très bien dans les tests. J'ai créer 6 objets et de les remplir avec des données. Faire beaucoup de ajoute, supprime, a() les opérations sur la Cart et tout fonctionne bien. J'ai lu que vous pouvez soit equals(TYPE var) ou equals(Object o) { (CAST) var } mais suppose que depuis qu'il travaillait, il n'a pas trop d'importance.

Puis j'ai rencontré un problème - j'avais besoin de créer un Book objet avec seulement la ID à partir du Livre de classe. Pas d'autres données serait entré en elle. Essentiellement les suivantes:

public boolean hasBook(int i) {
    Book b = new Book(i);
    return hasBook(b);
}

public boolean hasBook(Book b) {
    //.. more code here
    return this.books.contains(b);
}

Tout d'un coup, le equals(Book b) méthode ne fonctionne plus. Cela a pris un temps TRÈS long à repérer sans un bon débogueur et en supposant que le Cart classe a été testé correctement et de les corriger. Après swaapping la equals() méthode à la suivante:

public boolean equals(Object o) {
    Book b = (Book) o;
    ... //The rest goes here   
}

Tout a commencé à travailler à nouveau. Est-il une raison, la méthode décidé de ne pas prendre le Livre paramètre, même si c'est clairement la a un Book objet? La seule différence semble être que c'était instancié à partir de l'intérieur de la même classe, et remplis avec un membre de données. Je suis très très confus. S'il vous plaît, jeter un peu de lumière?

Je suis conscient que j'ai violé le "Contrat" concernant la substitution de l'égal des méthodes en cours de réflexion cependant j'ai besoin d'un moyen rapide de vérifier si l'objet existe dans la liste de tableaux sans l'aide de médicaments génériques.
C'est une bonne leçon à apprendre Java et est égale à

InformationsquelleAutor Josh Smeaton | 2008-10-09

323

En Java, le equals() méthode est héritée de Object est:
```
public boolean equals(Object other);
```
En d'autres termes, le paramètre doit être de type Object.

La ArrayList utilise la bonne méthode equals, où vous avez été toujours à l'appel de celui qui n'avait pas bien remplacer Objects'est égal.

Pas de substitution de la méthode correctement peut causer des problèmes.

Je remplace est égale à la suivante à chaque fois:
```
@Override
public boolean equals(Object other){
    if (other == null) return false;
    if (other == this) return true;
    if (!(other instanceof MyClass))return false;
    MyClass otherMyClass = (MyClass)other;
    ...test other properties here...
}
```
L'utilisation de la @Override annotation peut aider une tonne avec des erreurs stupides.

De les utiliser quand vous pensez que vous êtes remplacement d'une super-classe ou de l'interface de la méthode. De cette façon, si vous le faites mal, vous obtiendrez une erreur de compilation.
- C'est un bon argument en faveur de l' @Override annotation... si l'OP a utiliser @Override son compilateur aurait dit de lui qu'il n'était pas réellement de la substitution d'un parent de la classe de la méthode...
- Tout à fait d'accord!!!!! Je vais ajouter ça à ma réponse dans un sec
- N'a jamais été au courant de l' @Override, merci pour ça! J'aimerais aussi ajouter que primordial hashCode() ne devrait vraiment avoir été fait et peut-être repéré l'erreur plus tôt.
- Certaines IDEs (par exemple, Eclipse) peut même générer automatiquement equals() et hashcode() méthodes pour vous sur la base des variables de membre de classe.
- Eclipse fait généralement un bon travail aussi.
- if (!(other instanceof MyClass))return false; retourne false si MyClass s'étend de l'autre classe. Mais il ne serait pas de retour false si l'autre classe étendue MyClass. Ne devrait pas equal être moins contradictoires?
- Lors de l'utilisation de instanceof la précédente nullcheck est redondante.
- À partir de la question je n'étais pas à l'aide d'un IDE ou un débogueur - juste de la bonne vieille éditeur de texte et du Système.out. Le temps était très limité et que c'était un projet d'école. Les points sur les IDEs aider avec @Override mettre en évidence que l'utilisation d'un IDE et débogueur sont probablement encore plus important quand le temps est très limité.
- commentaire et @Nikel8000 's réponse de soulever un point TRÈS important. À moins que votre classe est définitive, vous êtes de rompre le contrat sur equals() que o1.equals(o2) doit retourner true uniquement si o2.equals(o1) renvoie également vrai. si o1 est une instance d'une sous-classe de o2, o1 instanceof o2 retourne vrai (et donc ne o1.equals(o2)), mais o2 instanceof o1 renvoie la valeur false, o2.equals(o1) retourne false, et le contrat est rompu.
- instanceof o1 n'est pas encore légal de la syntaxe. Le seul problème potentiel serait le cas d'une sous-classe en œuvre equals() de manière plus stricte.
InformationsquelleAutor
108

Si vous utilisez eclipse, allez simplement dans le menu du haut

Source --> Générer equals() et
hashCode()
- Je souhaite que je pourrais donner cette réponse 10 upvotes.
- Je suis d'accord! Ce que je ne savais jamais de l'avant et de générer les rend moins enclins à faire des erreurs
- Même ici. Merci Fred!
- Dans l'Ide, vous trouvez cela sous le Code → Générer... ou ctrl+N. 🙂
- Dans Netbeans, vous allez à la barre de menu > Source (ou clic droit) > Insérer du Code (ou Ctrl-I), et cliquez sur Générer equals()...
InformationsquelleAutor Fred
11

Légèrement hors-sujet à votre question, mais c'est probablement la peine de mentionner quand même:

Commons Lang a obtenu quelques excellentes méthodes que vous pouvez utiliser en remplaçant equals et hashcode. Découvrez EqualsBuilder.reflectionEquals(...) et HashCodeBuilder.reflectionHashCode(...). M'a sauvé beaucoup de maux de tête dans le passé - même si bien sûr, si vous voulez juste faire "est synonyme de" ID il peut ne pas convenir à votre situation.

Je suis aussi d'accord que vous devriez utiliser le @Override annotation lorsque vous êtes primordial est égal à (ou toute autre méthode).
- Si vous êtes un utilisateur d'eclipse, vous pouvez aussi aller right click -> source -> generate hashCode() and equals(),
- Suis-je le droit de ces méthode est exécutée au moment de l'exécution? Ne veut-on pas avoir des problèmes de performances dans le cas où si nous traversant une grande collection avec les éléments de la vérification de l'égalité de quelque autre élément à cause de la réflexion?
InformationsquelleAutor

Une autre solution rapide qui permet d'économiser du code réutilisable est Lombok EqualsAndHashCode annotation. Il est facile, élégant et personnalisable. Et ne dépend pas de l'IDE. Par exemple;

import lombok.EqualsAndHashCode;

@EqualsAndHashCode(of={"errorNumber","messageCode"}) //Will only use this fields to generate equals.
public class ErrorMessage{

    private long        errorNumber;
    private int         numberOfParameters;
    private Level       loggingLevel;
    private String      messageCode;

Voir le options disponible pour personnaliser les champs à utiliser dans les égaux. Lombok est dispo dans maven. Il suffit de l'ajouter avec fourni portée:

<dependency>
    <groupId>org.projectlombok</groupId>
    <artifactId>lombok</artifactId>
    <version>1.14.8</version>
    <scope>provided</scope>
</dependency>

InformationsquelleAutor borjab

dans Android Studio est
alt + inser ---> equals et hashCode

Exemple:

    @Override
public boolean equals(Object o) {
    if (this == o) return true;
    if (o == null || getClass() != o.getClass()) return false;

    Proveedor proveedor = (Proveedor) o;

    return getId() == proveedor.getId();

}

@Override
public int hashCode() {
    return getId();
}

InformationsquelleAutor David Hackro

1

Considérer:
```
Object obj = new Book();
obj.equals("hi");
//Oh noes! What happens now? Can't call it with a String that isn't a Book...
```
- Ce serait une bonne chose.
- Comment alors? obj est déclaré comme un Object. Le point de l'héritage est que vous pouvez ensuite affecter un Book à obj. Après que, à moins que vous suggèrent qu'une Object ne doit pas être comparable à un String via equals(), ce code doit être parfaitement légal et retour false.
- Je suggère exactement que. Je crois qu'il est assez largement accepté.
InformationsquelleAutor bcsb1001
0

la instanceOf déclaration est souvent utilisé dans la mise en œuvre d'égal à égal.

Ce populaire est un piège !

Le problème est que l'utilisation de instanceOf viole la règle de symétrie:

(object1.equals(object2) == true) si et seulement si (object2.equals(object1))

si le premier est égal à est vrai, et objet2 est une instance d'une sous-classe de
la classe où obj1 appartient, alors que la seconde est égal retournera false!

si l'considérée classe où l'ob1 appartient à est déclarée comme définitive, alors ce
le problème ne peut pas se poser, mais en général, vous devez tester comme suit:

this.getClass() != otherObject.getClass(); sinon, retourne false, sinon le test
les champs de comparer pour l'égalité!
- Voir Bloch, Efficace Java, Article 8, une grande section de discuter des problèmes avec substitution de la equals() méthode. Il recommande à l'encontre de l'aide getClass(). La raison principale est qu'il rompt le Principe de Substitution de Liskov pour les sous-classes, qui n'affectent pas l'égalité.
InformationsquelleAutor Nikel8000

-1

recordId est la propriété de l'objet

@Override
    public boolean equals(Object obj) {
        if (this == obj)
            return true;
        if (obj == null)
            return false;
        if (getClass() != obj.getClass())
            return false;
        Nai_record other = (Nai_record) obj;
        if (recordId == null) {
            if (other.recordId != null)
                return false;
        } else if (!recordId.equals(other.recordId))
            return false;
        return true;
    }

InformationsquelleAutor vootla561

Vous devez vous connecter pour publier un commentaire.