Логическая ошибка: метод
public boolean equals(Object o) { return this == o; }
сравнивает только ссылки (т.е. использует тождество), а не логическое (по полям) равенство объектов. Это не всегда то, что ожидают при сравнении объектов по содержимому.
Последствия в коллекциях:
- HashSet/HashMap: два разных экземпляра с одинаковыми значениями полей будут считаться разными ключами/элементами — в Set появятся дубликаты по смыслу; поиск/удаление по "эквивалентному" объекту не сработает.
- ArrayList.contains/remove и другие методы, использующие equals, тоже будут работать по ссылке, а не по содержимому.
- Если equals реализовать правильно по полям, но забыть переопределить hashCode, нарушится контракт equals/hashCode и поведение HashMap/HashSet станет некорректным (элементы могут "теряться" при поиске).
(Замечание: текущая реализация equals корректна как ссылка-равенство и синхронна с default hashCode, но это обычно не то, что ожидают при сравнении по содержимому.)
Как правильно реализовать equals и hashCode (рекомендованный шаблон):
- Сравнивать сначала ссылки, затем проверять null и тип, затем сравнивать значимые поля.
- hashCode должен вычисляться на тех же полях, которые используются в equals.
Пример (строгая проверка класса через getClass()):
public class Person {
private final String name;
private final int age;
// конструктор, геттеры...
@Override
public boolean equals(Object o) {
if (this == o) return true;
if (o == null || getClass() != o.getClass()) return false;
Person other = (Person) o;
return age == other.age && java.util.Objects.equals(name, other.name);
}
@Override
public int hashCode() {
return java.util.Objects.hash(name, age);
}
}
Пояснения:
- first line if (this == o) — быстрый путь для одинаковой ссылки;
- getClass() != o.getClass() — строгая проверка типа (альтернатива: instanceof, но с осторожностью — может нарушить симметрию при наследовании);
- java.util.Objects.equals безопасно сравнивает nullable поля;
- java.util.Objects.hash гарантирует согласованность hashCode с equals.
Кратко: если хотите логическое равенство — реализуйте equals по полям и всегда переопределяйте hashCode на тех же полях; если же действительно нужно тождество — документируйте это явно и не переопределяйте equals (или оставьте реализацию через == только осознанно).