И
Поподробнее, пожалуйста
Size: a a a
2020 February 02
E
да что вы к моему русскому придерайтесь
И
да что вы к моему русскому придерайтесь
ладно, проехали. Мне критика кода нужна
E
Invoice::query()->with(['user', 'payment'])->where('user_id', $user->id)->latest()->paginate(10)E
это не красиво
E
во первых зачем ты подгружаешь пользователя?
И
это не красиво
я выдрал с очень примитивного сайта, поэтому вот)
E
->where('user_id', $user->id)И
а что красиво?
E
у тебя он уже есть
И
->where('user_id', $user->id)ну можно whereId($user->id)
И
но лишнюю магию никто не любит
E
у тебя запрос по пользователю так что where user_id is redundant
E
дальше смотря из кода у тебя уже пользователь известен, и запрос должен быть таким:
$user->invoices()->with('payment')->latest()->paginate()
$user->invoices()->with('payment')->latest()->paginate()
E
payment has one, or belongs to, has many?
И
у тебя запрос по пользователю так что where user_id is redundant
значит там так неудобно было почему-то. Согласен, в общем и целом можно и так
E
не можно а нужно
И
значит там так неудобно было почему-то. Согласен, в общем и целом можно и так
у пользователя много счетов
E
блин как офигенно стикеры тут можно