Комментарии огонь)
Так я ж сегодня помню, завтра забуду все))
1. с учетом, что ты написал геттеры, то твои поля должны быть private. я не пишу обычно гетеры, я делаю свойства public, но обязательно readonly, но это не какое-то правило. у тебя как минимум поля должны быть private 2. у тебя валидация массива отсутствует, т.е. ты туда пихаешь любой массив и не проверяешь ключи. 3. конструктор DTO может принимать не массив, а поля, перечислить их, тем самым минимально ты можешь обозначить обзятаельные поля, дефолтные значения и их типы. а создавать из реквеста ты можешь в статическом методе, например make(), который вернет new self() и передаст правильно поля
У тебя какая версия PHP?
1 Понял 2 А разве того, что я в контроллере пробрасываю $request->validated() (при том, что реквест я свой писал, точно зная, какие ключи он вернет) не достаточно? 3. Поищу примеры ещё, так не очень понятно. Я видел, как отдельно приписываются поля. Тогда я вместо $request–>validated() должен по отдельности прокидывать элементы, а не массивом ...$request–>validated()
2. а речь не только про контроллер, смысл в том, что ты можешь свой сервис создать где угодно, в том числе там где и нет request-а, и следовательно когда ты по своему коду пробрасываешь эту дто, ты не можешь гарантировать, что на каком-то этапе у тебя данные не изменятся.
Тогда заметка про readonly поля для тебя. В 8.2 вообще красота, там readonly классы добавили
Получается, нужно создать отдельный метод для обработки данных, если они попали из реквеста?
$dto = new Dto($request->validated()); // какой-то код $dto->setDate('blablabla'); // какой-то код $service->run($dto);
https://github.com/adelf/acwa_book_ru https://github.com/adelf/acwa_book_ru/blob/master/manuscript/4-application-layer.md https://github.com/adelf/acwa_book_ru/blob/master/manuscript/6-validation.md
подразумевается, что валидация в dto происходит, независимо от того, откуда данные пришли
Обсуждают сегодня