Какова best practice для преобразования входящего JSON при создании/обновлении модели через API, где это принято делать?

Question

[Роми]

Суть вопроса:

При создании модели мы получаем на /api/create JSON в котором 100500 переменных, которые затем надо как-то привязать к модели, дополнить другими данными, и сохранить.

Получается такая примерно простыня

$user->name = $input['name'];
$user->address = $input['address'];
...
$user->babushka = $input['babushka'];

$user->save();

то же самое и на /api/update

В общем, получается контроллер состоящий из нескольких жирнейших методов.

В принципе, все работает, но хочется узнать какова Best practice по сборке таких объектов.

Где это общепринято делать - оставить в контроллере? Убрать в модель? Создать какой-то особый класс?

Заранее спасибо.

Comments

[FanatPHP]

Модель должна уметь работать со списками.
Задача контроллера только транслировать входящий жейсон в формат, который понимает модель

[Роми]

FanatPHP, ну, то есть, оставить эту простыню (см. код выше) в контроллере, так?

[Vitsliputsli]

Роми, нет, сделайте модели Request, которые будут знать что там приходит в запросе и будут эти данные валидировать. И именно в этих моделях будет простыня. Т.е. то что написал jazzus. Если таких моделей много, сделайте генератор кода, который будет генерить нужные модели по описанию.

[Роми]

Vitsliputsli,

сделайте генератор кода, который будет генерить нужные модели по описанию.

а что мне загуглить, чтобы посмотреть образец такого генератора?

[Vitsliputsli]

Роми, я даже не знаю, что там смотреть. json с описанием модели, тупо свойства и их типы, и где модель будет лежать. И на основе его создать модели, с геттерами, сеттерами или заполнением на основе массива, и стандартной вашей валидацией этих типов. Можно, конечно, подсмотреть это в фреймворке, в том где есть генераторы моделей, например для работы с БД, но по-моему дольше будете разбираться в избыточном функционале, который для этой задачи не нужен.

[Роми]

Vitsliputsli, ну, я думал там что-то особенное... )) Спасибо!

Answer 1

[jazzus]

$user->update($request->validated())
для создания аналогично с create

Comments

[Роми]

то есть, упаковать всю логику сборки объекта в "запрос формы": https://docs.rularavel.com/docs/8.x/validation#cre... ??

[jazzus]

Роми, валидацию упаковать в валидацию, а потом просто сохранение по валидированеым полям. просто и безопасно

[Роми]

jazzus, бро, смотри: там сборка объекта идет не только по данным с Request - еще с базы может что-то дернуть нужно и сделать еще 1001 мелочей :) И я хочу понять, куда общепринято пихать все эти действия. Да, я знаю что общепринято - в контроллер))) но хочется понять - как оптимально будет. Вот.

[jazzus]

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

[Роми]

jazzus, ну, в принципе, да, это существенная часть решения. Спасибо!

Answer 2

[inFureal]

Выдели отдельный сервис или экшн для манипуляций с моделью. Например

class UserHandler {
    static function handle($user, $data) {} 
}

Или же если комбинировать примеры, которые дали выше и добавить обсервер на события внутри модели

class UserObserver {
    function saving(User $user) {}
}

// И в AppServiceProvider
User::observe(UserObserver::class);

Comments

[inFureal]

Событие saving будет тригериться каждый раз при вызове save у модели.
В методах create и update за кадром именно save и вызывается

[Роми]

спасибо. любопытно :) надо осмыслить.

Answer 3

[Роми]

В конечном итоге просто создал сервисный класс и перетащил всё туда. Ну, хз "правильно" это или нет, но сейчас это так ))