Организовать структуру хранения контроллеров?

Question

[Виктор Капралов]

Здравствуйте!
Возникла следующая проблема. были 2 крупных контроллера и соответственно 2 крупные модели. Сейчас хочу распилить контроллеры и модели на более мелкие. С этим проблем нет никаких-они удобно пилятся. Но в итоге вместо 2 контроллеров будет 8. А там дальше планируется добавлять еще и еще. Не получится ли помойка? Может надо как-то изменить архитектуру папок?

Comments

[Inviz Custos]

Зачем здесь теги "ООП" и "MVC" ?

[Виктор Капралов]

Inviz Custos, модель и контроллер разве не относятся к MVC?

[Inviz Custos]

Виктор Капралов, "ООП" и "MVC" - это не о файликах и папочках...

[Виктор Капралов]

Inviz Custos, может вы уже по делу ответить соизволите?

[Дмитрий]

Виктор Капралов, во-первых не хамите.
Во-вторых, помойка у Вас сейчас.
Разделить класс на более мелкие будет лучше.
Но сначала изучите ООП, иначе у Вас будет помойка из нескольких мелких классов.

[Виктор Капралов]

Дмитрий, а где я хамлю?
По остальному понял.

[Михаил Братеньков]

Inviz Custos, Но ведь ООП и MVC - это вполне себе о классах и их архитектуре (о чем и задает вопрос ТС).

[Дмитрий]

Виктор Капралов, да, хамите.

может вы уже по делу ответить соизволите

[Виктор Капралов]

Дмитрий, у вас чсв.

[FanatPHP]

Чсв как раз у тебя. Ты считаешь свой тупой вопрос гениально сформулированным и хамишь людям, которые тебе объясняют обратное. Отличный способ получить консультацию специалистов

[FanatPHP]

Михаил Братеньков, если назвать свой говнокод "контроллером" и "моделью", это ещё не значит, что структура приложения соответствует паттерну MVC

[Vitsliputsli]

Виктор Капралов,

были 2 крупных контроллера и соответственно 2 крупные модели

Не должно быть такого соответствия, как впрочем и крупных контроллеров. Прочитайте про ТТУК. Это что касается MVC, что касается ООП - посмотрите на ваши модели, это действительно отдельные объекты, а не набор функций собранных в одном месте?

[Виктор Капралов]

FanatPHP, ради прикола погуглил твои статьи на хабре. Умом то ты тоже не блещешь. Поэтому ты такой злобный. И кроме как умничать ты ничего даже не прояснил.
Vitsliputsli, вот как раз для этого и пилю, чтобы не было 2 соответствия этим контроллерам раз и набором функций два.

[FanatPHP]

Какая смешная попытка потявкать в ответ :)

[Михаил Братеньков]

FanatPHP, во первых следите за речью.
Во вторых Виктор нигде не говорил что его код соответсвует MVC, тег говорит лишь о том что он рассматривает вопрос в контексте MVC.
В третьих вопрос как-раз касается этого паттерна т.к. его суть можно переформулировать как "Каким образом лучше поделить классы контроллеров?"

Answer 1

[Игорь]

Храните контроллеры как вам удобно.
Умеренная декомпозиция, контроллерам не навредит.

Конечно, если в контроллере чёрт ногу сломит, то лучше разбить на части.
Ещё один момент.
Зачем инициализировать дополнительные строки кода которые не будут использоваться в данный момент. Это очен важно!

Сами понимаете, если кода меньше то и интерпретатору будет легче работать.

Это как если бы вы взяли 5 мешков цемента по 20 кг, когда вам нужен один, но в будущем понадобятся и другие.
Потом можно сходить за цементом. Пока берём один мешок.

Merci

Answer 2

[Северное Сияние]

Но в итоге вместо 2 контроллеров будет 8. А там дальше планируется добавлять еще и еще. Не получится ли помойка?

помойка - это как реализуются контрооллеры во всех современных фреймворках, когда в одном классе контроллера куча действий.
У меня в самописном фрейме 1 контроллер == 1 класс и один паблик метод run().
Очень удобно и элегантно.

Comments

[Игорь]

Полностью согласен!

[Crash]

помойка - это как реализуются контрооллеры во всех современных фреймворках

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

[Северное Сияние]

Crash,

Если так реально реализуется во всех современных фреймворках, которые используют сотни тысяч, если не миллионы разработчиков - может все-таки так и надо?)

современные фреймворки пишутся смертными людьми, а не богами. А факт того, что их используют миллионы - ни о чем не говорит.

С подходом 1 контроллер - 1 экшен явно будет переусложенная логика распознавания запроса и формирования ответа.

причем тут вообще это?

Вот тебе пример контроллера - удаление пользователя администратором. Какой смысл весь этот код пихать о один общий контроллер? Я могу этот контроллер безболезненно перенести в любой другой проект и все будет фактически без переделок работать.

[Crash]

Василий Берестов, и сколько таких контроллеров придется нагородить в мало-мальски сложном проекте? Это больше риторический вопрос) Удачи вам с таким подходом

[Северное Сияние]

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

зато я ими верчу, как кирпичиками - могу перекидывать из модуля в модуль или из проекта в проект максимально безболезненно.

[Crash]

Василий Берестов, тут действительно есть дилемма, что выбрать:
1. Много файлов (это может затруднить визуальный поиск), но они легкие и содержание каждого из них можно понять за секунды.
2. Немного файлов, в каждом больше кода, но в пределах разумного.

Я понял ваш подход, в целом он тоже имеет право на жизнь. Но популярностью он не пользуется, так уж сложилось. Мне самому ближе 2-й подход. Я не начинал с самописных решений, сразу начал изучать популярные фреймворки. Контроллеры в них действительно могут серьезно разрастаться, но никто не мешает их потом делить на более специализированные.

[Михаил Братеньков]

Василий Берестов, а если делаем CRUD то аж 4 отдельных контроллера получается?
Впринципе если делить их по отдельным папкам то использовать такой подход вполне возможно

[Северное Сияние]

Михаил Братеньков, я совмещаю контроллер редактирования и добавления записи. там отличие лишь в наличии ID из запроса.