Использовать паттерн Builder для полей модели в Laravel?

Question

[siRius32]

1. Необходимо создавать модель с определенными полями и выполнять проверки, перед сохранением, в условиях которых запросы других данных и т.п., использование паттерна builder в таком случае хорошая идея? т.е. устанавливаем поля в методах builder->setЧтото() и в последнем методе builder->save() выполнять проверку перед сохранением или выбрасывать исключение, если проверка не прошла.
2. Изменять существующую модель, передавая в конструктор ид модели
Builder($id = null) {
$this->model = $id ? Model::findOrFail($id) : new Model();
}

Делается так вообще?

Answer 1

[the5x]

Builder используете, когда вам надо несколько условий для определенного запроса. Например GET запрос с несколькими параметрами. Например выборка продукта из БД по цвету, размеру, наличии и тд. Проверяете каждый входящий request. Вот пример использования:

https://timacdonald.me/dedicated-eloquent-model-qu...

Answer 2

[jazzus]

Вначале лучше изучить паттерн "примеры из документации", а потом уже бест-практики из Интернета, пакеты и др. велосипеды)

$this->model = $id ? Model::findOrFail($id) : new Model();

Для какой логики сия схема непонятно. Похоже на firstOrCreate и тогда в Ларавел есть одноименный метод. Если это типа метод show, то нужно юзать Model Binding
И там не нужно проверять никакие id и findOrFail идет вкомплекте. Примеры хороших роутов/методов/наименований есть в ресурсных контроллерах/маршрутах. Советую посмотреть.

Comments

[siRius32]

Есть таблица рейсов, в зависимости от того, какие поля/значения заполнены/установлены получаются разные типы рейсов. Запрос на создание/изменение рейса, приходит на один graphql эндпоинт. Если в запросе присутствует id, это изменение (редактирование), если нет - создание нового. Остальные пришедшие значения устанавливать через паттерн builder, в котором так же проверять, прямо в методах setX().

[jazzus]

siRius32, и? нравится делать по своему - никто не запрещает. Тогда в чем смысл вопроса. Как делать правильно в Ларавел я написал.

[jazzus]

siRius32,

Запрос на создание/изменение рейса

т.е. нужен ларавелевский метод updateOrCreate, только и он не нужен, т.к. юзается id, т.к. если есть ид то нужен отдельно метод update и отдельно store, когда ид нет, а получать модель не через findOrFail, а через model binding, как написал в ответе. updateOrCreate используется когда до запроса неизвестно есть модель или нет. Еще раз - см как устроены ресурсные роуты/контроллеры и др. примеры из документации.

Остальные пришедшие значения устанавливать через паттерн builder

хз что имеется ввиду, без кода сложно понять. Если сохранение то причем тут билдер. Юзай метод save/create/update. Если фильтрация есть метод when, в котором можно проверить реквест. В общем рекомендую обратить внимание на документацию фреймворка, который используется.