Какой код с точки зрения проектирования более правильный
Такой:
public List<String> getStringList(int expression){
List<String> list;
switch(expression){
case 1:
list = getList1();
break;
case 2:
list = getList2();
break;
...
}
return list;
}
Или такой:
public List<String> getStringList(int expression){
switch(expression){
case 1:
return getList1();
case 2:
return getList2();
...
}
}
В данный момент мы никак не изменяем и не предопалагаем, что нам нужно изменять список list
в функции getStringList
.
Хотелось бы прочитать обоснованный ответ в пользу того или иного варианта.
В первом случае у нас одна точка выхода их функции, в switch
мы только присваиваем переменную, которую возвращаем, а во втором случае у нас получается несколько точек выхода.
for
-...; i++
или же...; ++i
. Всё зависит от конкретного случая, как удобнее будет, как вам больше нравится и т.п. В книжках часто пишут про некий идеальный код в вакууме, да, я тоже неоднократно читал, например, что в методе нужно чтобы был 1return
. На практике же, где-то будет костыль, который уводит код от идеальности и придется впихнуть два или триreturn
. Главное чтобы код был читабельным, эффективным и таким, чтобы если понадобилось добавить новую фичу - вы смогли это сделать не перепиливая весь проект.