Собрал хорошие практики Laravel в репозиторий. Если у кого-нибудь есть дополнения, пожалуйста делитесь ими.
Еще написал статью на эту же тему, которая точно будет пролезна новичкам, мидлам и некоторым сениорам.
1. alexeymezenin - 08 Ноября, 2017 - 15:52:14 - перейти к сообщению
2. LIME - 08 Ноября, 2017 - 18:35:53 - перейти к сообщению
ткнул пальцем и пошел вниз
Нарушение SRP для Request.
В сервисы лучше переносить логику с большим количеством зависимостей или с пересечением ответственности моделей
Надо писать самодокументируемый код, а не лепить на строчку по коменту. И это не очень хороший пример комента. Если комент и нужен, то не чтобы описать что и как мы делаем, это и так видно, а для чего мы это делаем. То есть не лучше имхо, а то же самое.
Дальше не смотрел.
И вообще там наверное половина про программирование, а не про Laravel.
Но в целом
именно пролезная))
alexeymezenin пишет:
насколько помню оно AR? AR как бэ нарушает SRP. Модель не обязана знать откуда она берется итд итп. По своей сути, это лишь один из частных случаев принципа единой ответственности. Выносите работу с данными в модели при работе с Eloquent
alexeymezenin пишет:
превращать Request в DTO...ну хз..тоесть если мы из екселя будем импортировать нам повторять валидацию еще где-то?выносите валидацию из контроллера в Request классы
Нарушение SRP для Request.
alexeymezenin пишет:
Как-раз надо ее в моделях держать(по сути модель и есть бизнес-логика, в слой персистентности она прератилась в вебе)... особенно если выбросить персистентность станет удобноБизнес логика в сервис-классах
В сервисы лучше переносить логику с большим количеством зависимостей или с пересечением ответственности моделей
alexeymezenin пишет:
Комментируйте код
alexeymezenin пишет:
Determine if there are any joins.
Надо писать самодокументируемый код, а не лепить на строчку по коменту. И это не очень хороший пример комента. Если комент и нужен, то не чтобы описать что и как мы делаем, это и так видно, а для чего мы это делаем. То есть не лучше имхо, а то же самое.
Дальше не смотрел.
И вообще там наверное половина про программирование, а не про Laravel.
Но в целом
alexeymezenin пишет:
точно будет пролезна новичкам, мидлам и некоторым сениорам.
именно пролезная))
3. alexeymezenin - 08 Ноября, 2017 - 19:30:07 - перейти к сообщению
Цитата:
насколько помню оно AR? AR как бэ нарушает SRP. Модель не обязана знать откуда она берется итд итп.
Каким образом?
Цитата:
превращать Request в DTO...ну хз
В Laravel валидация входных данных (из форм и пр.) делается либо в контроллере (спагетти), либо в Request классе.
Цитата:
Надо писать самодокументируемый код, а не лепить на строчку по коменту.
Там ниже идет речь именно про самодокументируемый код.
4. alexeymezenin - 18 Ноября, 2017 - 10:39:19 - перейти к сообщению
Добавил мини статью о соглашениях об именовании классов, маршрутов и тд
5. LIME - 18 Ноября, 2017 - 11:03:07 - перейти к сообщению
Спорно
Чистый пиар себя любимого
Мое Имхо
(Добавление)
Не ответил на твой ответ потому как ты глупости написал
Не обижайся
Сам просил ревью
Чистый пиар себя любимого
Мое Имхо
(Добавление)
Не ответил на твой ответ потому как ты глупости написал
Не обижайся
Сам просил ревью