ткнул пальцем и пошел вниз alexeymezenin пишет:По своей сути, это лишь один из частных случаев принципа единой ответственности. Выносите работу с данными в модели при работе с Eloquent насколько помню оно AR? AR как бэ нарушает SRP. Модель не обязана знать откуда она берется итд итп.
alexeymezenin пишет:выносите валидацию из контроллера в Request классы превращать Request в DTO...ну хз..тоесть если мы из екселя будем импортировать нам повторять валидацию еще где-то?
Нарушение SRP для Request.
alexeymezenin пишет:Бизнес логика в сервис-классах Как-раз надо ее в моделях держать(по сути модель и есть бизнес-логика, в слой персистентности она прератилась в вебе)... особенно если выбросить персистентность станет удобно
В сервисы лучше переносить логику с большим количеством зависимостей или с пересечением ответственности моделей
alexeymezenin пишет:Комментируйте код
alexeymezenin пишет:Determine if there are any joins.
Надо писать самодокументируемый код, а не лепить на строчку по коменту. И это не очень хороший пример комента. Если комент и нужен, то не чтобы описать что и как мы делаем, это и так видно, а для чего мы это делаем. То есть не лучше имхо, а то же самое.
Дальше не смотрел.
И вообще там наверное половина про программирование, а не про Laravel.
Но в целом alexeymezenin пишет:точно будет пролезна новичкам, мидлам и некоторым сениорам.
именно пролезная)) (Отредактировано автором: 08 Ноября, 2017 - 19:30:06)
|