nemirova-e / web-practikum

Проект
0 stars 0 forks source link

Review #4

Open robotateme opened 2 years ago

robotateme commented 2 years ago

https://github.com/nemirova-e/web-practikum/blob/d8ddb778c0cb9c65940f90e584804dfcd5777de6/app/Http/Controllers/ProductController.php#L35

Наверное сложно, но я бы обернул это еще в сервис один, FilterService. В прочем ладно, и так симпатично. Надо мне попробовать самому обернуть это в более удобные классы. Форкну или пушну отдельной веткой. Просто как пример.

https://github.com/nemirova-e/web-practikum/blob/d8ddb778c0cb9c65940f90e584804dfcd5777de6/app/Http/Controllers/ProductController.php#L46

Посмотри здесь Model::all(). Моделей может быть много. Я бы подумал в сторону джойна. Можно переменную как-то передавать из подложки в шаблон вообще. Отрабатывать ее фасадом в подложке. Ну вообще, если это метод подгрузки первой страницы то не страшно.