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

Я, получается, просто могу из сервиса вернуть массив с нужными

мне полями?
Не могли бы вы, еще посмотреть, адекватно ли я сделал и применил DTO?

15 ответов

5 просмотров
Egor-Gogh Автор вопроса
Egor Gogh
screenshot

Комментарии огонь)

Egor-Gogh Автор вопроса
Смышляев Даниил
Комментарии огонь)

Так я ж сегодня помню, завтра забуду все))

1. с учетом, что ты написал геттеры, то твои поля должны быть private. я не пишу обычно гетеры, я делаю свойства public, но обязательно readonly, но это не какое-то правило. у тебя как минимум поля должны быть private 2. у тебя валидация массива отсутствует, т.е. ты туда пихаешь любой массив и не проверяешь ключи. 3. конструктор DTO может принимать не массив, а поля, перечислить их, тем самым минимально ты можешь обозначить обзятаельные поля, дефолтные значения и их типы. а создавать из реквеста ты можешь в статическом методе, например make(), который вернет new self() и передаст правильно поля

У тебя какая версия PHP?

Egor-Gogh Автор вопроса
PSYTRGLES
1. с учетом, что ты написал геттеры, то твои поля ...

1 Понял 2 А разве того, что я в контроллере пробрасываю $request->validated() (при том, что реквест я свой писал, точно зная, какие ключи он вернет) не достаточно? 3. Поищу примеры ещё, так не очень понятно. Я видел, как отдельно приписываются поля. Тогда я вместо $request–>validated() должен по отдельности прокидывать элементы, а не массивом ...$request–>validated()

Egor-Gogh Автор вопроса
Egor Gogh
1 Понял 2 А разве того, что я в контроллере пробра...

2. а речь не только про контроллер, смысл в том, что ты можешь свой сервис создать где угодно, в том числе там где и нет request-а, и следовательно когда ты по своему коду пробрасываешь эту дто, ты не можешь гарантировать, что на каком-то этапе у тебя данные не изменятся.

Egor Gogh
8.1

Тогда заметка про readonly поля для тебя. В 8.2 вообще красота, там readonly классы добавили

Egor-Gogh Автор вопроса
PSYTRGLES
2. а речь не только про контроллер, смысл в том, ч...

Получается, нужно создать отдельный метод для обработки данных, если они попали из реквеста?

Egor Gogh
1 Понял 2 А разве того, что я в контроллере пробра...

$dto = new Dto($request->validated()); // какой-то код $dto->setDate('blablabla'); // какой-то код $service->run($dto);

Egor Gogh
Получается, нужно создать отдельный метод для обра...

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

Egor Gogh
Получается, нужно создать отдельный метод для обра...

подразумевается, что валидация в dto происходит, независимо от того, откуда данные пришли

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

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

Коллеги, здравствуйте! А можно узнать ваше мнение относительно Wolfram Mathematica vs Julia? Просто у меня стоит выбор между тем, чтобы продолжить преподавать Wolfram Mathemat...
Илья Гаража
10
Обновленный chat тестили уже господа? Готовимся на заводы ? Простой проект на ларавель собирает за 1 ответ..
Jacov Borisov
14
кому не сложно распишите пожалуйста для какой цели тут вот эти скобки и в них пихается указатель?
Михаил Helper
15
я не магистр хаскеля, но разве не может лейзи тип конвертнуться в не-лейзи запросив вычисление содержимого прям при инициализации?
deadgnom32 λ madao
100
Или мне дублировать структуру с информацией о следующем потоке в адресное пространство текущего потока?
One
9
Серьезно? Вот прям так на глагне на первом экране?
Sasha Beep
8
Такс, блин, таки кто-то знает, каким образом работают макросы stdin/stdout/stderr? Я влез в stdio.h, там определения нет, отладил через асмокод - вызывается функция со странны...
The Bird of Hermes
18
Комрады, посоветуйте, куда копать? Стал прикручивать кастомизацию тем. В OnShow главной главной формы пытаюсь загрузить из файла настроек и применить тему (на скрине, как долж...
Ed Doc
13
в "главной форме" создаются "дочерние формы / копии главной" new_form := TForm1.Create(Application); new_form.blah_blah_blah обработчик, срабатывающий при изменении положени...
livontiy
6
OnShow один раз вызывается? или возможен Hide?
Iluha Companets
14
Карта сайта