Теоретический вопрос о правильном коде. Как в рамках MVC правильно реализовать метод ("не action") в контроллере?

Question

[MIK Ek]

В контроллере есть метод, который реализует большой функционал используемый в экшенах этого контроллера. (обмен данными между некоторыми произвольными моделями).
Ни к какой конкретной модели этот функционал логично не вписывается.
По логике (на мой взгляд) он является частью контроллера.
Правильно ли этот оставлять этот код в контроллере или лучше куда-то вынести?

Суть терзаний:
С одной стороны он вроде находится на своём месте, но с другой стороны он занимает большую часть контроллера, да и в принципе ведь лучше чтобы контроллер содержал только доступные из вне методы(экшены)?

Answer 1

[Михаил Бобков]

Создайте сервис и вынесите туда "обмен данными между некоторыми произвольными моделями".
Назовите его так чтобы было ясно что за обмен происходит и используйте в контроллере.

Answer 2

[Crash]

Все методы контроллера что не экшены - смело переносите в хелперы, там им самое место.
Разве что вспомогательные методы типа findModel можно оставить.

Comments

[MIK Ek]

Вот это плохой вариант т.к. в хелперах находятся исключительно классы вне бизнес логики, а часть контроллера это обязательно бизнес логика

[Crash]

MIKEk8, ну так хелпер можно ассоциировать с контроллером по имени класса, какие проблемы?
если всю бизнес-логику пихать в контроллер, то он разрастается со страшной силой. Проходили, знаем.