Надеюсь, что тухлыми помидорками новичка не закидают. Только осваиваю симфонию,

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

12 ответов

14 просмотров

контроллер контролирует. Грубо говоря смысл контроллера в том что бы сконвертить http запрос в действие приложения. Контроллеры не должны прнимать решений сами (делигируют кому-то, например какой denyAccessUnlessGranted). Есть так же argument value resolvers которые позволяют какие-то общие вещи по работе с данными запросов выносить выше. Удобно для мэппинга структур каких и т.д. В случае кто должен непосредственно загрузку делать - сервисы. Контроллер опять же ленивый менеджер и все делегировать должен.

я тоже только осваиваю, для понимания работы можно и в контроллере оставить, но потом это вырастет в толстый контроллер вот другой подход ImageUploader https://github.com/adelf/acwa_book_ru/blob/master/manuscript/1-bad-habits.md

Artem-Shubin 🇷🇺 Автор вопроса
Светлана О
я тоже только осваиваю, для понимания работы можно...

Ну, вот Сергей, ИМХО, пишет верно. Контроллер для другого. Пойду читать про сервисы и работу с ними. Тут еще непочатый край работы. А за ссылочку спасибо, поизучаю тоже.

Sergey P
контроллер контролирует. Грубо говоря смысл контро...

Контроллеры, которые просто перекладывают запросы в сервисы в симфе нафик не нужны. Тут достаточно мощный фронтконтроллер, который позволяет обращаться напрямик к сервисам. Роутинг + аргумент резолвер и вуаля, половина гетовых контроллеров превращается в метод репы. Остальное в другие сервисы. А контроллеры как точки входа становятся просто бесполезными

Андрей Ява
Контроллеры, которые просто перекладывают запросы ...

Я именно это и говорю. Но логики в контроллерах быть не должно. Не потому что "контроллер" а потому что много зависимостей. Логика должна быть там где зависимости минимизированы (whole value это ещё называется)

Sergey P
Я именно это и говорю. Но логики в контроллерах бы...

Ну я веду к тому, что мы называем "контроллерами" должнф быть как раз конечные сервисы (даже не сервисы, а функции - метод сервиса это прото частный случай). Которые могут быть вызваны по запросу, по событию, из консольной команды. Да вообще пофик откуда - это самостоятельная штука. А вот разруливание всяких внешних условий, таких как проверка прав, маппинг реквеста на ДТОшку, валидация и другие общие для всех реквестов и не относящиеся напрямик к выполняемому действию штуки, должны разруливатся уровнем выше в одном единственном на весь проект фронтконтроллере, который в симфе находится в HttpKernel е Ну это в идеале конечно.

Андрей Ява
Ну я веду к тому, что мы называем "контроллерами" ...

Это просто миграция названий... Вы делаете одно и другое и суммарно получается то, что и делаете... с др названиями Теперь хэндлер называется контроллером, и из cli команды вы его дёргать собираетесь... что сейчас люди и делают с хэндлером. Контроллер теперь что-то другое, по факту тоже самое, с др названием... Не совсем понятно зачем минрировать термины и все делать как было, ну видимо вам виднее

Максим Федоров
Это просто миграция названий... Вы делаете одно и ...

Это отказ от "контроллеров" как просто передастов запросов в хендлеры.

Андрей Ява
Это отказ от "контроллеров" как просто передастов ...

Откажетесь, но на фронт-контроллере все равно абстракцию наворачивать... как не назвать, но реквест для роута надо переварить

Андрей Ява
Ну я веду к тому, что мы называем "контроллерами" ...

Чет фигня тогда получится. Один большой класс с кучей аннотаций

Андрей Ява
Ну я веду к тому, что мы называем "контроллерами" ...

Только это не отменяет того факта что контроллер эти проверки должен делегировать кому-то.

Похожие вопросы

Обсуждают сегодня

Добрый вечер, Пока не совсем понимаю как наладить общение между телеграм ботом и ПО для работы с сим боксом. По самому боту так понял: - Нужен некий баланс, который можно поп...
Magic
6
сделал сайт, прикрутил в боте сайт, и виджет логина. как автоматически логинить пользователя в аккаунт(телеграм), при входе с бота?
Александра Чернивецкая
5
Объясните, пожалуйста, почему компилятор ругается на использование в условии неинициализированной переменной: int x; Task.Run(async () => { x = await somefunc(); }).Wait...
Александр
5
Ребят, подскажите, пожалуйста, почему в префиксе к ассетам, которые генерируются через фильтр | theme в шаблоне, стал вдруг появляться index.php? Вот так выглядит ссылка на а...
Виталий
1
Всем привет. Ребята, подскажите, пожалуйста. у ботов есть ограничение на отправку сообщений - 30 сообщений в секунду, эти ограничения накладываются на все сообщения? или на со...
Artem Stormageddon
4
Блин, ребята, сори за тупые вопросы. А можно ли как-то открыть вебапку по нажатию на кнопку в меню(которое появляется слева, команды)?
Artem Stormageddon
3
а плаксы из-под питона умеют только в комфортных условиях что-то выдавить из себя?)
Lencore
9
Но, может, есть уже проверенная? Наши требования такие: 1. Сообщения должны приходить из Инста в CRM оду 2. Должна быть возможность подключить несколько экаунтов Инстаграм. Р...
Alexander Sharoiko MSE / Александр Шаройко
13
Это может быть все-таки не флудвейт? у меня ботфазер принимает изменения и отображает даже что они изменились, на видео видно что он прислал якобы уже измененное описание, н...
OVERLINK
13
Коллеги, может знает кто, можно ли цвет бейджа счётчика в BackendMenu менять без бубнов?
Alex Blaze
3
Карта сайта