enum

Enum это объект, как и все в Java. Однако это особенный объект. Каждый из объектов енума создаётся только единожды. Давайте на примере:

enum TestEnum {ONE, TWO, THREE}

Если мы создадим 10 переменных TestEnum.ONE, то все они будут ссылаться на один и тот же объект. И поэтому enum можно сравнивать с помощью == и это корректно и будет работать.

Сторонники такого подхода называют следующие преимущества. Давайте их разберем.

Вы никогда не получите NullPointerException. И это правда, но если вы будете придерживаться правила "Сравнение константы слева", то и при использовании .equals() NullPointerException вам не страшен.

Оператор == работает быстрее. Быстрее чего? Видимо метода .equals(). Давайте посмотрим реализацию метода .equals() у enum.

Оператор == более понятный синтаксически. Это еще почему? Для сравнения объектов в Java используется .equals(). Enum это объект. Логичнее и очевиднее использовать .equals() для сравнения, чтобы не нарушать единообразие сравнения объектов.

На мой взгляд, правильнее использовать .equals(), главное не забывать о правиле "Сравнение константы слева".

Представьте, что у вас есть enum, который отвечает за статус пользователя в системе: "онлайн", "офлайн" и "занят".

public enum UserStatus {

    ONLINE, OFFLINE, BUSY
    
}

public class User {

    ...

    @Column(name = "status")
    @Enumerated(EnumType.STRING)
    private UserStatus status;
    
    ...

}

Скорее всего для выполнения бизнес-логики вам потребуется проверять статус пользователя.

if (user.getStatus().equals(UserStatus.ONLINE)) {
    // to do something
}

Вроде бы все отлично, миссия выполнена. Но есть одно НО. Что если getStatus() вернет вам null? Правильно, вы получите NullPointerException.

Чтобы этого избежать следует придерживаться правила "Сравнения константы слева". Оно очень простое. В нашем примере, мы точно уверены, что UserStatus.ONLINE существует, поэтому .equals() стоит вызывать от него.

if (UserStatus.ONLINE.equals(user.getStatus())) {
    // to do something
}

В остальных подобных ситуациях делайте также, например со строками:

if ("Иванов".equals(user.getLastName())) {
    // to do something
}

Это простое правило защитит вас от NullPointerException.