Laravel по-русски
Laravel по-русски
Вступил в наши ряды: 18 апреля 2016
Замечен в последний раз: 27 мая 2021
Оставил на форуме: 346 сообщений
Последнее сообщение: 5 августа 2018
Вы сможете отправить письмо, если войдёте
MVC - это паттерн, а не архитектура, вот пара слов:
http://ahrameev.ru/article/why-do-php-developers-think-mvc-is-an-application-architecture.html
Сейчас в тренде модульность, спёрли от yii2 (у кого что раньше появилось - не знаю, факт в том, что в yii это из коробки и в документации, для ларавела есть куча пакетов на гите) - http://www.yiiframework.com/doc-2.0/guide-structure-modules.html .
Да и модульность - это часть архитектуры, каждый модуль можешь делать как душа ляжет.
Рефактор получился не плохим.
Добавлю свое представление: метод handleUploadedImage следует видоизменить, передавать ему не Request, а непосредственно File (иначе как можно переиспользовать этот метод?).
А при config('app.uploading_enabled') == false, метод handleUploadedImage должен выкидывать Exception (ибо метод был вызван, но выполниться он не может, это как-то странно, что он молча ничего не сделает).