ИС
в некоторых кейсах, конечно, использую
Size: a a a
2019 March 05
РИ
Только не говори что в контроллере
ИС
но зачастую у меня сервисы выполняли операции с моделями
и по факту это был лишний слой
и по факту это был лишний слой
РИ
Я блин только Дзен с сервисами постиг)))
ИС
поэтому я такие сервисы убрал и перенес логику в модели
РИ
поэтому я такие сервисы убрал и перенес логику в модели
Т.е теперь у тебя операции с моделями делаются в моделях?
ИС
Я блин только Дзен с сервисами постиг)))
нет единого правильного подхода)
ИС
нет единого правильного подхода)
это не значит, что после моего высказывания надо сразу все переносить в модели
ИС
Т.е теперь у тебя операции с моделями делаются в моделях?
да
РИ
это не значит, что после моего высказывания надо сразу все переносить в модели
Ну вроде ты очень понимающий в теме. Поэтому я к тебе прислушаюсь.
РИ
Но это конечно прикол. Я всегда так делал, потом умные люди мне сказали что так не нужно делать и сейчас мне опять говорят что так лучше делать
KB
Но это конечно прикол. Я всегда так делал, потом умные люди мне сказали что так не нужно делать и сейчас мне опять говорят что так лучше делать
А потом твой опыт дорастёт до момента, когда тебе всё равно кто там что говорит, и ты будешь писать код, ориентируясь в первую очередь на требования бизнеса, и удобство поддержки, а не мнения других людей)
AK
Погоди. И где у тебя теперь все происхоит?
сейчас я юзаю Scopes, Observers
РИ
А потом твой опыт дорастёт до момента, когда тебе всё равно кто там что говорит, и ты будешь писать код, ориентируясь в первую очередь на требования бизнеса, и удобство поддержки, а не мнения других людей)
Похоже не дорастет если буду плавать))
KB
Похоже не дорастет если буду плавать))
Всё будет со временем + количеством написанных проектов)
РИ
Всё будет со временем + количеством написанных проектов)
Спасибо =). Просто смотрю что с каждым новым проектом что-то все меняется)))
YP
#Мимопроходил
Если есть возможность (а она есть, скорее всего), то
Если есть возможность (а она есть, скорее всего), то
$products = Product::query()можно упростить до
->when($request->is('asc'), function ($query) {
$query->orderBy('price', 'asc');
})
->when($request->is('desc'), function ($query) {
$query->orderBy('price', 'desc');
})
->get();
$products = Product::orderBy('price', $request->price_sorting_order)->get();YP
предварительно отвалидировав поле price_sorting_order
ОТ
Это будет самый правильный путь.
ИС
Это будет самый правильный путь.
Ещё круче было бы воспользоваться Laravel Query Builder, вкусная штучка