Согласно принятому ответу « Обоснование предпочтения локальных переменных переменным экземпляра? », Переменные должны жить в наименьшей возможной области видимости.
Упростите проблему в моей интерпретации, это означает, что мы должны реорганизовать такой код:
public class Main {
private A a;
private B b;
public ABResult getResult() {
getA();
getB();
return ABFactory.mix(a, b);
}
private getA() {
a = SomeFactory.getA();
}
private getB() {
b = SomeFactory.getB();
}
}во что-то вроде этого:
public class Main {
public ABResult getResult() {
A a = getA();
B b = getB();
return ABFactory.mix(a, b);
}
private getA() {
return SomeFactory.getA();
}
private getB() {
return SomeFactory.getB();
}
}но согласно «духу» «переменные должны жить в наименьшей возможной области видимости», разве «никогда не иметь переменные» имеют меньшую область действия, чем «иметь переменные»? Поэтому я думаю, что приведенная выше версия должна быть реорганизована:
public class Main {
public ABResult getResult() {
return ABFactory.mix(getA(), getB());
}
private getA() {
return SomeFactory.getA();
}
private getB() {
return SomeFactory.getB();
}
}так что getResult()вообще не имеет локальных переменных. Это правда?
72
Создание явных переменных имеет то преимущество, что им приходится называть их имена. Введение нескольких переменных может быстро превратить непрозрачный метод в читаемый.
—
Джаред Гогуэн
Честно говоря, пример с точки зрения имен переменных a и b слишком придуман, чтобы продемонстрировать значение локальных переменных. К счастью, вы все равно получили хорошие ответы.
—
Док Браун
Во втором фрагменте ваши переменные на самом деле не являются переменными . Они фактически являются локальными константами, потому что вы не изменяете их. Java-компилятор будет обрабатывать их одинаково, если вы определили их окончательно, потому что они находятся в локальной области видимости, и компилятор знает, что с ними произойдет. Это вопрос стиля и мнения, нужно ли вам использовать
—
Хайд
finalключевое слово или нет.
@JaredGoguen Создание явных переменных сопряжено с бременем их именования и возможностью их именования.
—
Берги
Абсолютно ноль правил стиля кода, таких как «переменные должны находиться в наименьшей возможной области видимости», универсально применимы без размышлений. Таким образом, вы никогда не захотите основывать решение о стиле кода на аргументации формы в этом вопросе, где вы берете простое руководство и расширяете его в менее очевидную область («переменные должны иметь небольшие области действия, если это возможно» -> «переменные не должны существовать, если это возможно»). Либо есть веские причины, конкретно подтверждающие ваше заключение, либо ваше заключение является неуместным продолжением; в любом случае вам не нужно оригинальное руководство.
—
Бен