Рефакторинг некоторого кода я нашел это. Я предполагаю, что это должно быть внутри конструктора. Есть мысли о том, что это вне класса?

Question

Я занимаюсь рефакторингом кода из приложения, которое я разрабатываю вместе с другими коллегами. У меня есть мысли, должно ли это быть вне класса или внутри его в конструкторе.

Всем спасибо!

-Charl ie

Answer 1

Массив BRANCHES является постоянным полем, и вполне нормально, если он будет определен за пределами класса BranchSelector

Объявление этого способа дает вам гибкость для экспорта массива BRANCHES и импорта он также находится в каком-то другом файле

Вы не получаете никаких преимуществ, определяя его в конструкторе.

Однако, если возникают случаи, когда вам нужно изменить и обновить этот массив, добавив или удалив элементы , тогда имеет смысл указать c компоненту класса и определить его как состояние