требуется пошаговое создание (например из запроса).
Если не использовать Строителя, а использовать конструктор для создания объекта, то можно было бы сделать метод fromRequest для создания объекта из запроса.
public static function fromRequest(Request $request): self
{
return new self([
'title' => $request->get('title'),
'body' => $request->get('body'),
'author' => Author::find($request->get('author_id')),
]);
}
Однако со строителем как это будет выглядеть? Запрос может включать, а может не включать все свойства для создания объекта. Неужели кучу if-ов делать? Reflection Api не хотелось бы
if ($request->get('title')) {
$object->setTitle($request->get('title'))
}
или можно и строителя отдельно, и конструктор в классе объекта использовать? Просто конструктор получается просто гигантский в этом случае. Свойств около 100
в объекте не нужен fromRequest но его можно засунуть в строителя и там будет что-то типа public static function fromRequest(Request $request): self { $builder = (new self) ->withRequiredField1($request->get('field1')) ->withRequiredField2($request->get('field2')) ; if ($request->has('title')) { $builder = $builder->withTitle($request->get('title')) } return $builder; } и использовать $object = ObjectBuilder::fromRequest($request)->build();
То есть все-таки 100 if-ов придется сделать? if ($request->has('title')) if ($request->has('foo')) if ($request->has('bar')) Может все-таки конструктор в объекте вместе со строителем?
я бы в конструкор только обязательные штуки передавал или подождал именованных аргументов. Потому что совсем не круто, когда у тебя 90 необязательных полей, а тебе надо передать 89-ое из них =)
DTO? Сделать там необязательные поля сразу дать им дефолт значения и создать дтошку передав ей весь массив пришедших параметров spatie/data-transfer-object
Но пакет же тоже самое предлагает: https://pastebin.com/z6yJEw6b
а можно сделать так return new self($request->all()); или как там у тебя получить все параметры в виде массива, и тогда ДТО наполнится только теми данными что пришли, остальные будут по дефолту а еще можно принести параметры в виде JSON и тогда даже типизация будет
Но это свойство же само не заполнится? Все равно нужен какой-то сеттер 'author' => Author::find($request->get('author_id')),
это да, это самое конечно не заполнится
Обсуждают сегодня