Переопределение метода 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 может помочь множеству глупых ошибок.

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

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

Источник -> Создать equals () и hashCode ()

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

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

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

Еще одно быстрое решение, позволяющее сохранить шаблонный код, - это аннотация 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;

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

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

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

Рассмотреть возможность:

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

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

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

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

(object1.equals(object2) == true) тогда и только тогда, когда (object2.equals(object1))

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

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

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

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