L
сделал бы лучше? :D
Не, я слишком стар, чтобы спорить в чатах
Size: a a a
2019 November 24
AP
Ну, если на Ютубе полно роликов где бизнес логика в контроллерах то о чем вообще можно говорить)
Ну отсюда можно сделать вывод, что в Ютубе полно говна)
A
Ну отсюда можно сделать вывод, что в Ютубе полно говна)
Если это не Несмеянов и Елисеев то можно вообще удалять этот мусор с ютуба...
S
В модели этого мусора быть точно не должно.
В контроллер, а если его засрёшь — в сервисы
В контроллер, а если его засрёшь — в сервисы
а как в сервисах сделать это?
M
а как в сервисах сделать это?
везде, где делаешь $order->price надо делать $orderService->calculatePriceByUser($order, $user)
Но в качестве простого грязного решения сойдёт и твоё. У него один минус - у него неявная зависимость от $user . В сервисе явная, туда можно подать любого пользователя, не только залогиненного. Профит - оно тестируется. Но если ты не тестируешь, то и твой вариант норм.
Но в качестве простого грязного решения сойдёт и твоё. У него один минус - у него неявная зависимость от $user . В сервисе явная, туда можно подать любого пользователя, не только залогиненного. Профит - оно тестируется. Но если ты не тестируешь, то и твой вариант норм.
M
С сервисами, если ты передаёшь данные на фронт в виде json, надо ещё заморачиваться с гидрацией его вот этой вот ценой. Много головняка.
ЕП
Michael
везде, где делаешь $order->price надо делать $orderService->calculatePriceByUser($order, $user)
Но в качестве простого грязного решения сойдёт и твоё. У него один минус - у него неявная зависимость от $user . В сервисе явная, туда можно подать любого пользователя, не только залогиненного. Профит - оно тестируется. Но если ты не тестируешь, то и твой вариант норм.
Но в качестве простого грязного решения сойдёт и твоё. У него один минус - у него неявная зависимость от $user . В сервисе явная, туда можно подать любого пользователя, не только залогиненного. Профит - оно тестируется. Но если ты не тестируешь, то и твой вариант норм.
Ещё Профит что любой админ может узнать цену для любого юзера, например
S
Michael
везде, где делаешь $order->price надо делать $orderService->calculatePriceByUser($order, $user)
Но в качестве простого грязного решения сойдёт и твоё. У него один минус - у него неявная зависимость от $user . В сервисе явная, туда можно подать любого пользователя, не только залогиненного. Профит - оно тестируется. Но если ты не тестируешь, то и твой вариант норм.
Но в качестве простого грязного решения сойдёт и твоё. У него один минус - у него неявная зависимость от $user . В сервисе явная, туда можно подать любого пользователя, не только залогиненного. Профит - оно тестируется. Но если ты не тестируешь, то и твой вариант норм.
а есть пример как это сделать в сервисе?
🎃
а есть пример как это сделать в сервисе?
с помощью php
🎃
чем тебе мало ордера и юзера
M
Ещё Профит что любой админ может узнать цену для любого юзера, например
да, ты прав, поэтому в простом решении лучше переименовать геттер в getPriceForLoggedUserAttribute и сделать метод в модели calculatePrice($user)
M
а есть пример как это сделать в сервисе?
Так я же написал.
M
Сервис - это обычный php-класс
Создаёшь его где угодно, вызываешь в контроллере например или где там нужно цену считать
Создаёшь его где угодно, вызываешь в контроллере например или где там нужно цену считать
S
Michael
Сервис - это обычный php-класс
Создаёшь его где угодно, вызываешь в контроллере например или где там нужно цену считать
Создаёшь его где угодно, вызываешь в контроллере например или где там нужно цену считать
в контроллере не подойдет, нужно в модели
V
Так во второй и 3 строке фигня
AB
Michael
Call to a member function comments() on null - тут про таблицу не слова. Если перевести, тут написано что вызывается метод ->comments() у null . Т.е. в $post у тебя null , а не пост.да сорри, забыл что минут пять назад ресетнул миграции, разобрался спс
AP
Michael
С сервисами, если ты передаёшь данные на фронт в виде json, надо ещё заморачиваться с гидрацией его вот этой вот ценой. Много головняка.
Гидрацию можно делать в другом месте
AP
Тут уже вкусовщина
M
А почему именно в модели ? Смысл как раз отвязать модель от того, чтобы она расчитывала цену для залогиненного юзера. Сделай как я предлагал выше, метод в модели calculatePrice($user), если хочешь всё в модели делать.