Anular el método java equals () - ¿no funciona?

Me encontré con un problema interesante (y muy frustrante) con el equals()método hoy que causó el bloqueo de lo que pensé que era una clase bien probada y causó un error que me llevó mucho tiempo rastrear.

Solo para completar, no estaba usando un IDE o depurador, solo un buen editor de texto antiguo y System.out. El tiempo era muy limitado y era un proyecto escolar.

De todos modos

Que estaba desarrollando un carrito de la compra básica que podría contener un ArrayListde Booklos objetos . Con el fin de poner en práctica los addBook(), removeBook()y hasBook()métodos de la Cesta, quería comprobar si el Bookya existía en el Cart. Entonces me voy

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

Todo funciona bien en las pruebas. Creo 6 objetos y los lleno con datos. Realiza muchas operaciones de adición, eliminación, has () en Carty todo funciona bien. Leí que puedes tener equals(TYPE var)oequals(Object o) { (CAST) var } asumir que, dado que funcionaba, no importaba demasiado.

Luego me encontré con un problema: necesitaba crear un Bookobjeto con solo el contenido IDdentro de la clase Libro. No se ingresarán otros datos. Básicamente lo siguiente:

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

De repente, el equals(Book b)método ya no funciona. Esto tomó MUCHO tiempo para rastrear sin un buen depurador y suponiendo que la Cartclase se probó y corrigió correctamente. Después de cambiar el equals()método a lo siguiente:

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

Todo comenzó a funcionar nuevamente. ¿Hay alguna razón por la cual el método decidió no tomar el parámetro Libro a pesar de que claramente era un Bookobjeto? La única diferencia parecía ser que se instanciaba dentro de la misma clase y solo se llenaba con un miembro de datos. Estoy muy muy confundido Por favor, arrojar algo de luz?

En Java, el equals()método que se hereda Objectes:

public boolean equals(Object other);

En otras palabras, el parámetro debe ser de tipo Object. Esto se llama anulación ; su método public boolean equals(Book other)hace lo que se llama sobrecarga al equals()método.

Los ArrayListusos sobrescritos equals()métodos para comparar los contenidos (por ejemplo, para su contains()y equals()métodos), no sobrecargados queridos. En la mayoría de su código, llamar al que no anuló correctamente Objectlos iguales estuvo bien, pero no es compatible con ArrayList.

Por lo tanto, no anular el método correctamente puede causar problemas.

Anulo es igual a lo siguiente cada vez:

@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...
}

El uso de la @Overrideanotación puede ayudar mucho con errores tontos.

Úselo siempre que piense que está anulando el método de una superclase o interfaz. De esa manera, si lo hace de manera incorrecta, obtendrá un error de compilación.

Si usa eclipse solo vaya al menú superior

Fuente -> Generar equals () y hashCode ()

Ligeramente fuera de tema a su pregunta, pero probablemente vale la pena mencionar de todos modos:

Commons Lang tiene algunos métodos excelentes que puedes usar para anular equals y hashcode. Consulte EqualsBuilder.reflectionEquals (...) y HashCodeBuilder.reflectionHashCode (...) . Me ahorró mucho dolor de cabeza en el pasado, aunque, por supuesto, si solo quiere hacer "iguales" en la identificación, puede que no se ajuste a sus circunstancias.

También estoy de acuerdo en que debe usar la @Overrideanotación siempre que anule iguales (o cualquier otro método).

Otra solución rápida que ahorra código repetitivo es la anotación Lombok EqualsAndHashCode . Es fácil, elegante y personalizable. Y no depende del IDE . Por ejemplo;

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;

Vea las opciones disponibles para personalizar qué campos usar en los iguales. Lombok está disponible en maven . Simplemente agréguelo con el alcance proporcionado :

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

en Android Studio es alt + insert ---> equals y hashCode

Ejemplo:

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

Considerar:

Object obj = new Book();
obj.equals("hi");
// Oh noes! What happens now? Can't call it with a String that isn't a Book...

la instanceOfdeclaración se usa a menudo en la implementación de iguales.

¡Esta es una trampa popular!

El problema es que el uso instanceOfviola la regla de simetría:

(object1.equals(object2) == true) si y solo si (object2.equals(object1))

si el primer igual es verdadero y object2 es una instancia de una subclase de la clase a la que pertenece obj1, ¡el segundo igual devolverá falso!

si la clase considerada a la que pertenece ob1 se declara como final, entonces este problema no puede surgir, pero en general, debe probar lo siguiente:

this.getClass() != otherObject.getClass(); si no, devuelve falso, de lo contrario, pruebe los campos para comparar la igualdad.

recordId es propiedad del objeto

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