Переопределение метода java equals () - не работает?

В Java метод equals(), унаследованный от Object, имеет следующий вид:

public boolean equals(Object other);

Другими словами, параметр должен иметь тип Object. Это называется преобладающий; ваш метод public boolean equals(Book other) выполняет то, что называется перегрузка по отношению к методу equals().

ArrayList использует переопределенные методы equals() для сравнения содержимого (например, для его методов contains() и equals()), нет перегруженных. В большей части вашего кода вызов того, который не переопределяет должным образом Object, был нормальным, но несовместимым с ArrayList.

Таким образом, неправильная переопределение метода может вызвать проблемы.

Я переопределяю каждый раз:

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

Использование аннотации @Override может помочь множеству глупых ошибок.

Используйте его всякий раз, когда считаете, что переопределяете метод суперкласса или интерфейса. Таким образом, если вы сделаете это неправильно, вы получите ошибку компиляции.

Немного не по теме вашего вопроса, но, вероятно, все равно стоит упомянуть:

У Commons Lang есть несколько отличных методов, которые вы можете использовать для переопределения равенства и хэш-кода. Проверьте EqualsBuilder.reflectionEquals (...) и HashCodeBuilder.reflectionHashCode (...). В прошлом избавил меня от многих головных болей - хотя, конечно, если вы просто хотите сделать «равное» по идентификатору, это может не соответствовать вашим обстоятельствам.

Я также согласен с тем, что вам следует использовать аннотацию @Override всякий раз, когда вы переопределяете равенство (или любой другой метод).

Если вы используете eclipse, просто перейдите в верхнее меню

Source --> Generate equals() and hashCode()

Еще одно быстрое решение, сохраняющее шаблонный код, - Аннотация Lombok EqualsAndHashCode. Это просто, элегантно и легко настраивается. И не зависит от IDE. Например;

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;

См. Доступный опции, чтобы настроить, какие поля использовать в равных. Ломбок доступен в знаток. Просто добавьте его с областью действия при условии:

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

оператор instanceOf часто используется при реализации равенства.

Это популярная ловушка!

Проблема в том, что использование instanceOf нарушает правило симметрии:

(object1.equals(object2) == true)если и только если(object2.equals(object1))

если первое равенство истинно, а объект2 является экземпляром подкласса класс, к которому принадлежит obj1, то второе равенство вернет false!

если рассматриваемый класс, к которому принадлежит ob1, объявлен как final, то этот Проблема не может возникнуть, но в целом вам следует провести тестирование следующим образом:

this.getClass() != otherObject.getClass();, если нет, вернуть false, в противном случае проверить поля для сравнения на равенство!

в Android Studio есть alt + insert ---> равно и hashCode

Пример:

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

recordId - свойство объекта

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

Рассматривать:

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