Что может пойти не так если переопределить только equals без hashCode

Переопределение только equals без hashCode

Это классическая ошибка Java разработчиков, которая приводит к неправильному поведению и трудноуловимым багам. Давайте разберёмся в контракте между этими двумя методами.

Контракт equals() и hashCode()

Java Doc явно указывает три правила:

1. Если объекты равны по equals(), то их hashCode() должен быть одинаковым
2. Если hashCode() одинаковый, то equals() может вернуть false (hash коллизия)
3. Если equals() вернул false, hashCode() может быть любым

Проблема переопределения только equals()

Нарушение контракта. Если ты переопределил equals() без hashCode(), это нарушает контракт. Объекты которые равны по equals(), могут иметь разные hashCode().

Проблемы с HashSet и HashMap:

class User {
    private String email;
    private String name;

    public User(String email, String name) {
        this.email = email;
        this.name = name;
    }

    @Override
    public boolean equals(Object o) {
        if (this == o) return true;
        if (!(o instanceof User)) return false;
        User user = (User) o;
        return email.equals(user.email);
    }
}

public class Main {
    public static void main(String[] args) {
        User user1 = new User("john@email.com", "John");
        User user2 = new User("john@email.com", "John");

        System.out.println(user1.equals(user2)); // true
        System.out.println(user1.hashCode()); // 1234567
        System.out.println(user2.hashCode()); // 7654321

        Set<User> users = new HashSet<>();
        users.add(user1);
        users.add(user2);

        System.out.println(users.size()); // 2 вместо 1 - ОШИБКА!
    }
}

Последствия

Дубликаты в HashSet. Ты можешь добавить одинаковые объекты в HashSet и оба останутся.

Потеря значений в HashMap. Если использовать эти объекты как ключи:

Map<User, String> userData = new HashMap<>();

User key1 = new User("john@email.com", "John");
User key2 = new User("john@email.com", "John");

userData.put(key1, "Some Data");
String value = userData.get(key2); // null вместо "Some Data" - ОШИБКА!

Почему? HashMap сначала проверяет hashCode():

1. HashMap вычисляет hashCode(key2)
2. Ищет bucket с этим hashCode
3. Не находит (там другой hashCode)
4. Возвращает null, не проверяя equals()

Правильное решение

Всегда переопределяй ОБА метода:

class User {
    private String email;
    private String name;

    public User(String email, String name) {
        this.email = email;
        this.name = name;
    }

    @Override
    public boolean equals(Object o) {
        if (this == o) return true;
        if (!(o instanceof User)) return false;
        User user = (User) o;
        return email.equals(user.email);
    }

    @Override
    public int hashCode() {
        return Objects.hash(email);
    }
}

Правило IDE

В IntelliJ IDEA и Eclipse есть инспекция, которая предупреждает об этом. Используй Code -> Generate -> equals() and hashCode() — IDE автоматически согласует оба метода.

Вывод

Никогда не переопределяй equals() без hashCode() — это нарушает Java контракт и приведёт к неправильному поведению с коллекциями. Всегда переопределяй ОБА методов вместе, основываясь на одних и тех же полях.