1

Есть пример кода:

//объявляем ссылку
SomeClass someClass;
//в зависимости от условий инициализируем ссылку тем или иным образом
if (condition1) {
  someClass = value1;
} else {
  someClass = value2;
}

Я столкнулся с мнением, что это "грязный код", и вместо этого надо стараться объявлять иммутабельные переменные везде, где это возможно:

final SomeClass someClass = someValue;

Чем плох первый пример и какие аргументы в пользу второго?

5
  • Вы можете сделать в первом примере кода final SomeClass someClass; и будет красиво. 20 июл 2022 в 12:44
  • Не уверен, что "иммутабельные переменные везде, где это возможно:" - это нормально, но код можно явно сократить, просто задав переменной какойе-то дефолтное значение, например value2 и тогда достатчно будет потом написать только if без всяких els'ов 20 июл 2022 в 12:56
  • 1
    final SomeClass someClass = condition ? value1 : value2; 20 июл 2022 в 16:39
  • Так и приходится писать с тернарником, но это не всегда подходит
    – EsReich
    20 июл 2022 в 23:58
  • 1
    Можно всего вынести в отдельный метод, вместо присваивания делать return value1; и return value2;
    – insolor
    28 июл 2022 в 13:52

1 ответ 1

1

Просто добавьте модификатор final в вашем примере:

final SomeClass someClass;
if (condition) {
  someClass = value1;
} else {
  someClass = value2;
}

Так сложнее в коде допустить ошибку и присвоить переменной что-нибудь не то. Считаю, что писать final — это хорошо.

В простых случаях, как уже подсказал @NowhereMan, пользуйтесь тернарным оператором:

final SomeClass someClass = condition ? value1 : value2;

Если кода много, то, как предлагает @insolor, можно объявить новый метод:

final SomeClass someClass = getSomeClass();
private SomeClass getSomeClass() {
    if (condition) {
        return value1;
    } else {
        return value2;
    }
}
2

Ваш ответ

By clicking “Отправить ответ”, you agree to our terms of service and acknowledge that you have read and understand our privacy policy and code of conduct.

Всё ещё ищете ответ? Посмотрите другие вопросы с метками или задайте свой вопрос.