Лучше хотя бы перечислять, что записываешь в модель. Сейчас же у тебя нет контроля этого
ну лень так делать, быстрый способ передать массив валидированный и create массив(
А потом через месяц зайдёшь, и "ой, а что сюда записывается, я забыл"
как понял так имели ввиду это? public function run(Dto $dto){ data= [ 'name'=>$dto->name ] Category::create($data) }
С DTO отдельная тема, для начала, нужно ли оно тебе? если всё же юзаешь, то главное чтобы были одинаковые имена атрибутов, в дто и в модели
если без Дто, то передаете весь обьект реквеста или все 10 параметров передаем отдельно вместо массива?)?)
И касательно именно этого кода - можно не создавать отдельный массив data, после передавать его в create. Можно внутри сразу: create(['key' => $value, ..])
Да, передавать все 10 параметров отдельно, меня здесь так учили) Кода хоть и больше, но ты видишь что записываешь
Смотря где это написано. Похоже на билдер репозитория. Но выглядит норм.
Category::create($request->validated()) Так тоже норм. Лишние поля $fillable в модели отсеит.
Бред. Так делают те, кто либо не умеет в массовое заполнение, либо убили его переопределив свойство модели $guarded
https://laravel.com/docs/10.x/eloquent#inserts
Обсуждают сегодня