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

Нужно создавать объект пошагово. Реализовал паттерн Строитель. Однако не всегда

требуется пошаговое создание (например из запроса).

Если не использовать Строителя, а использовать конструктор для создания объекта, то можно было бы сделать метод 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'))
}

9 ответов

17 просмотров
W.-Blazkowicz Автор вопроса

или можно и строителя отдельно, и конструктор в классе объекта использовать? Просто конструктор получается просто гигантский в этом случае. Свойств около 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();

W.-Blazkowicz Автор вопроса
Volodymyr Melko
в объекте не нужен fromRequest но его можно засуну...

То есть все-таки 100 if-ов придется сделать? if ($request->has('title')) if ($request->has('foo')) if ($request->has('bar')) Может все-таки конструктор в объекте вместе со строителем?

W. Blazkowicz
То есть все-таки 100 if-ов придется сделать? if ...

я бы в конструкор только обязательные штуки передавал или подождал именованных аргументов. Потому что совсем не круто, когда у тебя 90 необязательных полей, а тебе надо передать 89-ое из них =)

W. Blazkowicz
То есть все-таки 100 if-ов придется сделать? if ...

DTO? Сделать там необязательные поля сразу дать им дефолт значения и создать дтошку передав ей весь массив пришедших параметров spatie/data-transfer-object

W.-Blazkowicz Автор вопроса
Дмитрий Щербаков
DTO? Сделать там необязательные поля сразу дать им...

Но пакет же тоже самое предлагает: https://pastebin.com/z6yJEw6b

W. Blazkowicz
Но пакет же тоже самое предлагает: https://pastebi...

а можно сделать так return new self($request->all()); или как там у тебя получить все параметры в виде массива, и тогда ДТО наполнится только теми данными что пришли, остальные будут по дефолту а еще можно принести параметры в виде JSON и тогда даже типизация будет

W.-Blazkowicz Автор вопроса
Дмитрий Щербаков
а можно сделать так return new self($request->all(...

Но это свойство же само не заполнится? Все равно нужен какой-то сеттер 'author' => Author::find($request->get('author_id')),

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

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

а через ESC-код ?
Alexey Kulakov
29
30500 за редактор? )
Владимир
47
Чёт не понял, я ж правильной функцией воспользовался чтобы вывести отладочную информацию? но что-то она не ловится
notme
18
У меня есть функция где происходит это: write_bit(buffer, 1); write_bit(buffer, 0); write_bit(buffer, 1); write_bit(buffer, 1); write_bit(buffer, 1); w...
~
13
Недавно Google Project Zero нашёл багу в SQLite с помощью LLM, о чём достаточно было шумно в определённых интернетах, которые сопровождались рассказами, что скоро всех "ибешни...
Alex Sherbakov
5
Как передать управляющий символ в открытую через CreateProcess консоль? Собсна, есть процедура: procedure TRedirectThread.WriteData(Data: OEMString); var Written: Cardinal;...
Serjone
6
в JclConsole объявлено так: function CtrlHandler(CtrlType: DWORD): BOOL; stdcall; - где ваше объявление с stdcall? у вас на картинке нет stdcall
Karagy
8
Ребят в СИ можно реализовать ООП?
Николай
33
program test; {$mode delphi} procedure proc(v: int32); overload; begin end; procedure proc(v: int64); overload; begin end; var x: uint64; begin proc(x); end. Уж не знаю...
notme
6
https://github.com/erlang/otp/blob/OTP-27.1/lib/kernel/src/logger_h_common.erl#L174 https://github.com/erlang/otp/blob/OTP-27.1/lib/kernel/src/logger_olp.erl#L76 15 лет назад...
Maksim Lapshin
20
Карта сайта