ЕК
if ($discount) {
return $price;
} else {
return $price;
}
return $price;
} else {
return $price;
}
Size: a a a
2019 November 24
ЕК
эээ
ЕК
што
AP
Привет, в плане чистого кода и правильной разработки такое можно в модели писать?
public function getPriceAttribute($price)
{
$user = Auth::guard('customer')->user();
$discount = Customer::find($user->id)->discount;
if ($discount) {
return $price + $discount;
} else {
return $price;
}
}
В плане чистого кода не стоит юзать елокуент да и практически весь ларавел.
ЕК
Но это следующий этап, сначала бы понять, зачем ты написал всю эту конструкцию вместо return $price;
S
што
если в $discount есть запись тогда выводить return $price + $discount ;
ЕК
Ладно)
L
В плане чистого кода не стоит юзать елокуент да и практически весь ларавел.
Нифига себе вброс
ЕК
Вот теперь отстал)
A
В плане чистого кода не стоит юзать елокуент да и практически весь ларавел.
И ООП в целом....
AP
И ООП в целом....
А где ты видешь ООП в ларавел?)
ЕК
А где ты видешь ООП в ларавел?)
А где ты видишь не ООП?
ЕК
S
Что посоветуеете, куда логику перенести и сделать модель такой для чего она нужна и чистым код
A
А где ты видешь ООП в ларавел?)
Узнаешь меня?)
ЕК
Что посоветуеете, куда логику перенести и сделать модель такой для чего она нужна и чистым код
В модели этого мусора быть точно не должно.
В контроллер, а если его засрёшь — в сервисы
В контроллер, а если его засрёшь — в сервисы
AP
Нифига себе вброс
Ну а чем вброс. Сущность знает о бд, зависит от статики. Бизнес логика напиха где попало....
A
В модели этого мусора быть точно не должно.
В контроллер, а если его засрёшь — в сервисы
В контроллер, а если его засрёшь — в сервисы
Ну, если на Ютубе полно роликов где бизнес логика в контроллерах то о чем вообще можно говорить)
D
Нифига себе вброс
сделал бы лучше? :D
AP
А где ты видишь не ООП?
То что есть в ларавел, это больше похоже на процедурку, только функции попихали в рандомные классы, то туда, то туда