Почему enum должен быть immutable?

Question

[nuclear_kote]

Разработчик сделал из енума контейнер. Как объяснить почему так делать не надо? Может есть статья какая-нибудь? Кроме того что это синглтон ничего в голову не приходит

Comments

[Artem Gapchenko]

А пример-то можно того, как это выглядит?

[nuclear_kote]

Artem Gapchenko,

public enum Type {
    A(1), B(2), C(3);

    private int value;
    Type(int value) {...}
    
    public int getValue() { ... }
    public void setValue() { ... }
}

[Mikhail Osher]

Далек от Джавы. А можно вопрос. Зачем вообще создавать enum, кроме как такого рода?

public enum Status {
  REGISTERED,
  ACTIVE,
  SUSPENDED
}

[nuclear_kote]

Mikhail Osher, ну создавать можно такое например чтобы были сразу ключи локализации или дефолтные значения

[Денис Загаевский]

Mikhail Osher, затем, что это удобно.

Answer 1

[Dmtm]

в такой реализации будут проблемы с конкурентным доступом,
как аргумент - нарушение общепринятого стиля кодирования = проблемы конторы при смене разработчика,
знаю ситуацию когда одного кодера уволили за упрямое не соблюдение ява-соглашений

Answer 2

[Денис Загаевский]

Потому что иммутабельность это вообще хорошо. И обычно, видя енам, ожидаешь, что он иммутабельный.
Чем он аргументирует такой подход?

Answer 3

[Дмитрий Александров]

Хорошо или плохо надо смотреть по ситуации. Аналогичная ситуация с интерфейсами в которые суют default реализации.
К примеру приблизительно такую реализацию я не считаю плохой:

public enum Model {
       CYLINDER(new Cylinder(), "Cylinder"),
       CONES(new Cone(), "Cones"),
       BOX(new Box(), "Box"),
       Line(new Line(), "Line");

       private final String label;
       private final Model model;

       Model(Model model, String label) {
           this.model = vis;
           this.label = label;
       }
       public Model getVisualization() {
           return this.model;
       }
       public String getLabel() {
           return this.label;
       }
   }

Ситуация когда туда пихают вообще все что непопадя уже проблема.
Приведите живой пример что он у вас туда напихал.

Comments

[nuclear_kote]

Код под NDA. Выкладывать естественно не буду.

Но смысл там в том что есть параметры некой системы например загрузка CPU, RAM и тп. вот их он в енум и сетил

[Денис Загаевский]

Если все потомки Model - immutable, то ок. Если нет, то не ок.