rustem_ck Dec 27 2015 at 13:03

Как мы делали Linux-пингвинов к Новому году

2 min

4.6K

CMS*Laravel*PHP*Website development*

-5

Comments 7

andrewnester Dec 27 2015 at 13:27

есть пару замечаний, которые сходу бросились в глаза
1) папка vendor и composer.lock в репозитории
2) местами разный стиль кодирования, для open-source проектов лучше выбирайте PSR
3) нет тестов
4) местами макаронный код github.com/ZENLIX/LaraShop/blob/master/laravel/app/Http/Controllers/PurchaseController.php

ну и уже как личное предпочтение, в Laravel 5 хороший DI механизм, лично я предпочитаю его фасадам, как по мне статические фасады в Laravel было не самой классной идеей

AotD Dec 27 2015 at 14:32

vendor таки да, нафиг, а что не так с composer.lock? Версии зафиксированы, никаких неожиданностей.

andrewnester Dec 27 2015 at 14:59

ну если рассматривать этот код как код исключительно проекта/приложения, то всё ок, согласен

честно говоря, лучше было бы оформить данный код как Laravel package, а не цельное приложение и вот там лично я бы composer.lock убрал, как советует доки по composer

andrewnester Dec 27 2015 at 13:31

и смотрели ли Вы на готовые решения, например, github.com/amsgames/laravel-shop

Alex10 Dec 27 2015 at 13:58

А фильтр по атрибутам товаров есть?

Pinsky Dec 27 2015 at 22:23

У Вас мобильная вверстка поехала.

thunderspb Dec 28 2015 at 16:31

Только у меня не сходится название и содержание? Или я что-то такое не понимаю? Если второе, то объясните пожалуйста :)