Контроллер одного действия в MVC?

Question

[Alex Walsh]

Не будет ли плохим тоном делать все контроллеры "контроллерами одного действия" (invokable)?

По типу:
- FetchManyPostsController
- StorePostController
- ShowPostController
- UpdatePostController
- DestroyPostController

За ранее благодарю всех ответивших =)

UPD:

Дело в том что сейчас у нас крупный проект где почти каждый эндпоинт имеет на много больше действий чем стандартный CRUD, миксовать стили контроллеров не хочется и большое кол-во методов в одном контроллере тоже такое себе

Comments

[Aleksey]

Звучит как полный бред, значит тон плохой) Нахрена такие заморочки?

P.S. не думали пойти дальше, и под каждый параметр контроллер создать? Ну типа если обновляем пост 123 то и контроллер за него отвечает UpdatePost123Controller?)

[Василий Банников]

Aleksey, если бы пост 123 был каким-то особенным, то я бы выделил )

А так у описанного подхода есть много плюсов:
1. Из файловой структуры сразу видно, какие операции реализованы
2. git сразу показывает в истории изменений, какие операции были затронуты
3. Даже если какая-то одна операция разрослась на кучу строк - это не влияет на читабельность остальных операций, тк они описаны в отдельных файлах.

[jazzus]

Василий Банников,
1. Загаженная файловая структура в которой ничего не видно.
2. И толку? Тебе все равно нужно смотреть изменения.
3. Есть сервис классы

Answer 1

[Василий Банников]

Если тебе так удобнее будет поддерживать - почему нет?
Такой подход много где применяется и он точно лучше, чем божественные контроллеры

Comments

[Alex Walsh]

Спасибо)

[Aleksey]

Какая божественность, здесь тупо CRUD. 3 строки на каждый метод. Ни малейшего смысла на 5 контроллеров разносить.

[Alex Walsh]

Aleksey, Дело в том что сейчас у нас крупный проект где почти каждый эндпоинт имеет на много больше действий чем стандартный CRUD, миксовать стили контроллеров не хочется и большое кол-во методов в одном контроллере тоже такое себе

Answer 2

[jazzus]

Смотря для каких целей - посмеяться, ок. В реальном проекте говнокод и неудобно поддерживать. Не надо путать метод и класс. Для crud в Ларавел используют ресурсные контроллеры которые тянут за собой плюхи типа политик, подключаемых одной строчкой кода через конструктор и роутов для каждого метода. Одобрено тысячами разработчиков. Другие методы можно вынести в новый контроллер.