169 похожих чатов

Это плохая практика ?) Лучше будет $category= new Category; $category->$name $category->save()?)))

12 ответов

5 просмотров

Лучше хотя бы перечислять, что записываешь в модель. Сейчас же у тебя нет контроля этого

Edige-Kozykhan Автор вопроса
Василий 🇺🇦❤️
Лучше хотя бы перечислять, что записываешь в модел...

ну лень так делать, быстрый способ передать массив валидированный и create массив(

Edige Kozykhan
ну лень так делать, быстрый способ передать массив...

А потом через месяц зайдёшь, и "ой, а что сюда записывается, я забыл"

Edige-Kozykhan Автор вопроса
Василий 🇺🇦❤️
Лучше хотя бы перечислять, что записываешь в модел...

как понял так имели ввиду это? public function run(Dto $dto){ data= [ 'name'=>$dto->name ] Category::create($data) }

Edige Kozykhan
как понял так имели ввиду это? public function run...

С DTO отдельная тема, для начала, нужно ли оно тебе? если всё же юзаешь, то главное чтобы были одинаковые имена атрибутов, в дто и в модели

Edige-Kozykhan Автор вопроса
Василий 🇺🇦❤️
С DTO отдельная тема, для начала, нужно ли оно теб...

если без Дто, то передаете весь обьект реквеста или все 10 параметров передаем отдельно вместо массива?)?)

Edige Kozykhan
как понял так имели ввиду это? public function run...

И касательно именно этого кода - можно не создавать отдельный массив data, после передавать его в create. Можно внутри сразу: create(['key' => $value, ..])

Edige Kozykhan
если без Дто, то передаете весь обьект реквеста ил...

Да, передавать все 10 параметров отдельно, меня здесь так учили) Кода хоть и больше, но ты видишь что записываешь

Смотря где это написано. Похоже на билдер репозитория. Но выглядит норм.

Edige Kozykhan
если без Дто, то передаете весь обьект реквеста ил...

Category::create($request->validated()) Так тоже норм. Лишние поля $fillable в модели отсеит.

Василий 🇺🇦❤️
Да, передавать все 10 параметров отдельно, меня зд...

Бред. Так делают те, кто либо не умеет в массовое заполнение, либо убили его переопределив свойство модели $guarded

Похожие вопросы

Обсуждают сегодня

а зачем этот вопрос для удаления из чата?
Mёdkinson Medvezhkin
63
Эх кто-то пришел и весь праздник испортил :( You need complex FBX scene importing setup to change things on import? good luck with that. You need navigation and pathfinding? g...
Serg Gini
5
Какого хера? /Sources/App/Modules/User/Models/UserLinkApple.swift:21:20: warning: stored property '_id' of 'Sendable'-conforming class 'UserLinkApple' is mutable @ID(...
Alexander Sherbakov
14
Всем привет! Подскажите. Я написал приложение на Delphi 10.2 Tokyo под Windows 10. И передо мной стал вопрос о том чтобы сделать это приложение кроссплатформенным (под Linux и...
Дмитрий Завгородний
24
Привет всем. Подскажите где можно посмотреть, какая версия электрон, поддерживает версии windows? Некий changelog. Мне бы желательно, поддержку 7,8,10... latest, как понимаю и...
Anonym Squad
21
Почему стало ломаться на D11? "739002.86400000' is not a valid timestamp" function IncDateTime(aStamp:TTimeStamp;aKind:TTriggerKind;aInterval:Integer):TDateTime; //aStamp = 2...
Катерина Свиридова
8
У тебя в конфиге нигде нет deny all; или вообще любого deny?
Alexander Sherbakov
10
Портфолио: Зовут меня Александр, мне 36 лет. Город Пушкино. Общий рабочий стаж: ~14 лет Уровень квалификации: Senior Full-stack developer Где прочесть мой код? https://github....
Magic
10
Ребят, чет я уже не догоняю... Крч в коде на асм там происходит нечто вроде a+число (a+1, a+2 и т.д.). Но почему строка lea ecx, [edx+1] работает как a+1?? В edx берется адрес...
Alan 🔝 Бэброу
4
думаешь я не смогу также сделать? мне это просто не удобно
int 💳 𝙖𝙞𝙧 𝙗𝙞𝙜 𝙗𝙤𝙗 🔫 check bio / spam block / AFK / nohello.com / GMT+3
9
Карта сайта