Кратко — какие проблемы и как исправить.
Проблемы в вашем фрагменте
- Метод имеет сигнатуру `equals(A o)`, а не `equals(Object)`. Это не переопределяет `Object.equals` — вы перегружаете метод. HashSet вызывает `equals(Object)`, поэтому для двух разных объектов с одинаковым `id` будут:
- одинаковый `hashCode()` (если он возвращает `id`),
- но `equals` останется от `Object` (сравнение по ссылке) — в результате в HashSet могут появиться дубликаты.
- Не обрабатывается `null` и проверка типа (равенство с `null` должно возвращать `false`).
- Если поле `id` изменяемо после помещения в HashSet, поведение коллекции нарушается (элемент «потеряется» в множестве).
- Потенциально плохое распределение хэшей (если `id` даёт слабое распределение), но это вторично.
Правильные принципы
- Переопределять `equals(Object)` и `hashCode()` вместе, на одних и тех же полях.
- В `equals` сначала проверять `this == o`, потом тип (`instanceof` или `getClass()` в зависимости от семантики), потом сравнивать поля; `equals(null)` должно вернуть `false`.
- Сделать поля, участвующие в equals/hashCode, неизменяемыми (final) или не менять их после вставки в хэш-структуры.
- Использовать `@Override` — компилятор поймает ошибочную сигнатуру.
Правильная реализация (пример для примитивного `int id`)
public class A {
private final int id;
public A(int id) { this.id = id; }
@Override
public boolean equals(Object o) {
if (this == o) return true;
if (!(o instanceof A)) return false;
A a = (A) o;
return this.id == a.id;
}
@Override
public int hashCode() {
return Integer.hashCode(id);
}
}
Если `id` — объект `Integer` (может быть `null`):
public class A {
private final Integer id;
public A(Integer id) { this.id = id; }
@Override
public boolean equals(Object o) {
if (this == o) return true;
if (!(o instanceof A)) return false;
A a = (A) o;
return java.util.Objects.equals(this.id, a.id);
}
@Override
public int hashCode() {
return java.util.Objects.hashCode(id);
}
}
Дополнительно
- Если класс должен быть расширяемым и равенство не должно учитывать подклассы — используйте `getClass()` вместо `instanceof`.
- В современных версиях Java можно использовать `record` для автоматической корректной реализации `equals`/`hashCode`.
- Никогда не менять поля, участвующие в equals/hashCode, пока объект находится в HashSet/HashMap — либо делайте поля final, либо удаляйте и заново добавляйте объект после изменения.