Мы обсуждаем не вопрос производительности.
Код пишется один раз, а читают его потом всю оставшуюся жизнь.
Логика в контроллерах это лапша, которую невозможно читать.
Так что тут претензии скорее эстетические.
При этом вытащить весь код из экшена и убрать его в SomethingService это вопрос 30 секунд драгоценного времени.