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

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

12 ответов

5 просмотров

контроллер контролирует. Грубо говоря смысл контроллера в том что бы сконвертить 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 команды вы его дёргать собираетесь... что сейчас люди и делают с хэндлером. Контроллер теперь что-то другое, по факту тоже самое, с др названием... Не совсем понятно зачем минрировать термины и все делать как было, ну видимо вам виднее

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

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

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

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

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

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

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

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

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

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

Всем привет. Подскажите, почему не меняется значение поля при переключении сайта?
Alexander Peterikov
11
Можно ли загрузить скрипт py в бота чтобы он работал по нему? как это сделать?
huskadam #RCC Фанат? @hitlerpvp
13
'frakturBold' => ['𝖆', '𝖇', '𝖈', '𝖉', '𝖊', '𝖋', '𝖌', '𝖍', '𝖎', '𝖏', '𝖐', '𝖑', '𝖒', '𝖓', '𝖔', '𝖕', '𝖖', '𝖗', '𝖘', '𝖙', '𝖚', '𝖛', '𝖜', '𝖝', '𝖞', '𝖟', '𝕬', '𝕭', '𝕮', '𝕯'...
Roma
4
Добрый день, не подскажите, если в OC-V3 поменять страндартную директорию /storage/ на /storage2/ - не будет сильно много проблем ?
Max Dubovsky
32
Ребят, а за скок можно впарить анон чат с апишкой и веб админкой ?
Eugene Неелов
15
Цвет аватарки с буквой же зависит от айди? Как может быть такое, что этот цвет поменялся?
Lencore
2
Приветствую друзья, подскажите сколько в среднем стоит на данный момент создать тг бота который будет как магазин? Показывать ассортименты доставлять заказы и тд? Все по станд...
Eugene
3
@dblackCat Привет. Это же твой плагин? https://octobercms.com/plugin/catdesign-productbundle
Alexey Yakimov
5
Добрый день! Кто-нибудь знает как подключить твиг в контроллеры плагина?
Николай Афанасенко
5
Ты просто гитлеровскую эстетику плохо понимаешь. Он же всё под Цезаря делал. А это как бы запрещённый приём в политике. Пиджаки они зачем все носят? Чтобы показать что они тип...
Ivan Kropotkin
4
Карта сайта