Я не увидел слова Validator в классе

Есть мнение, называть классы тем, что они делают. Суффиксы ор, ер типа моветон. Слышал про такое? Теперь посмотри на папочку, в которой это лежит. Стало понятней?

Есть мнение, называть классы тем, что они делают. Суффиксы ор, ер типа моветон. Слышал про такое? Теперь посмотри на папочку, в которой это лежит. Стало понятней?

Т.е. если я увижу твой класс в коде, мне понять что он делает, надо найти его папку? ЛолШто...

Опять таки, в его названии глагол, а хорошее название класса - существительное

Т.е. если я увижу твой класс в коде, мне понять что он делает, надо найти его папку? ЛолШто...

А если он называется просто валидатор понятно что именно он делает?

Допустим, если там 2+ реализации внутри.

А если он называется просто валидатор понятно что именно он делает?

А что мне надо от класса, который просто валидатор) Валидатор и есть валидатор ))

Допустим, если там 2+ реализации внутри.

Врядли твоему коду вообще нужно две реализации, скорее всего должен быть интерйес и 2 реализации

Опять таки : AnnotationValidator OtherFrameWorkValidator AttributeValidator

Если я так сделал, значит нужно. Не?

Если я так сделал, значит нужно. Не?

Возможно, ок, ладно, закончим. Но честно выглядит гавном ) Могу быть и не прав)

Так. я заебался оправдываться. Вот тебе задача. Надо валидировать дто. В некоторых случаях надо ошибки отдавать с группировкой по полям, в некоторых просто список ошибок текстом. Предлагай реализацию

Тут тоже есть валидация)

Тут из FromRequest заполняется Command? Не юзал атрибуты, там можно так дтошки/команды заполнять?

Опять таки, в его названии глагол, а хорошее название класса - существительное

а это когда успело появиться? 🙂 я не то чтобы за игрульками дддшными сильно слежу, но так краем глаза поглядываю иногда. такого вот не помню

Тут из FromRequest заполняется Command? Не юзал атрибуты, там можно так дтошки/команды заполнять?

Да, примерно так. Но это можно и без атрибутов, через интерфейс единый для DTO. Интерфейс чисто для заполнения из http

И юзать типа как кастомные laravel-like FooBarRequest инжектируемые через argumentresolver в контроллер с эксепшеном если реквест не провалидировался?

И юзать типа как кастомные laravel-like FooBarRequest инжектируемые через argumentresolver в контроллер с эксепшеном если реквест не провалидировался?

Примерно, только в Ларе привязка к http (ну раньше была, наверное и сейчас так)

Опять таки, в его названии глагол, а хорошее название класса - существительное

Бла-бла-бла. На эту тему срутся вечно. Похуц. Давай лучше по реализации

Так. я заебался оправдываться. Вот тебе задача. Надо валидировать дто. В некоторых случаях надо ошибки отдавать с группировкой по полям, в некоторых просто список ошибок текстом. Предлагай реализацию

Мое решение было ранее, отображение вообще не касается валидатора

Зачем тебе вообще 2 отображения ошибок?

а это когда успело появиться? 🙂 я не то чтобы за игрульками дддшными сильно слежу, но так краем глаза поглядываю иногда. такого вот не помню

Про существительные тоже слышал) но немного рвет пукан Manager -or/-er окончания. Например UseCase/FooHandler))

Но это мелочь

Мое решение было ранее, отображение вообще не касается валидатора

Таскать ещё одну зависимость в конструктор для отображенияв нужном виде?

а это когда успело появиться? 🙂 я не то чтобы за игрульками дддшными сильно слежу, но так краем глаза поглядываю иногда. такого вот не помню

Это вроде к ДДД не относится, Вроде обычный хороший тон кода, откуда не вспомню

Классы - не глаголы

не глаголы, да, но worker - это не глагол

это вполне себе существительное

не глаголы, да, но worker - это не глагол

А причем тут worker?

Ну да, методы глаголы у идеале. Но фактор методы fromFoo, fromBar тоже ведь не глаголы) хотя тут просто упускается createFromFoo/createFromBar

то есть почему -or/-er плохо-то?

то есть почему -or/-er плохо-то?

Это не ко мне, я такого не говорил

Классы - не глаголы

Не истина в последней инстанции. Когда навешаешь на 1 ивент штук 20 хендлеров, потом будешь рассказывать как называть

это вполне себе существительное

В этом и конфуз)))

сорян тогда, сплю уже одним глазом

Не истина в последней инстанции. Когда навешаешь на 1 ивент штук 20 хендлеров, потом будешь рассказывать как называть

Эм, не вижу проблемы. П.С, думаю над твоей задачей

Don't Create Objects That End With -ER https://www.yegor256.com/2015/03/09/objects-end-with-er.html

А ля такого плана есть мнения)

ахаха бугаенко нет, пожалуйста

Дада) но в целом дискашенв есть и на других ресурсах вроде типа реддита и стака, но хз) за шо купил, так сказать, за то и продаю))

Эм, не вижу проблемы. П.С, думаю над твоей задачей

Увидишь. Потому что в названии надо описывать то, что делает этот хендлер. Иначе ебанешься понимать что там внутри.

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

А потом и остальное начнёшь называть на понятном языке, а не набором слов

@theantt Если брать конкретно симфони, можно сделать два ErrorResponse (название придумай) и в них класть ошибки и по разному нормализовать. Т.е. это как раз view класс.

Но и если без симфони, надо разделять валидацию и отоюражение. Да и через di тащить не обязательно

А потом и остальное начнёшь называть на понятном языке, а не набором слов

У меня и так все норм)))

да нормального у него названо, это два хелпера, которые один формат массива кладут в другой, как блин их ещё называть-то

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

Понял) ну в целом именовать все как UserManager, UserQueryManager, как и UserService, UserCompanySrrvice, мб обезличенно и излишне

да нормального у него названо, это два хелпера, которые один формат массива кладут в другой, как блин их ещё называть-то

Это два валидатора с разным выводом

Но и если без симфони, надо разделять валидацию и отоюражение. Да и через di тащить не обязательно

Дай ка угадаю, хелпер на статических методах?

Понял) ну в целом именовать все как UserManager, UserQueryManager, как и UserService, UserCompanySrrvice, мб обезличенно и излишне

Но иногда можно))

Это два валидатора с разным выводом

Нет, это два декоратора с разным выводом, внутри симфонический валидатор ))

Дай ка угадаю, хелпер на статических методах?

Нет, $this->json(new OneTypeValidationErrorResponse($validationErrors)): $this->json(new SecondTypeValidationErrorResponse($validationErrors)):

Нет, это два декоратора с разным выводом, внутри симфонический валидатор ))

Ну тогда и назови их по человечески)) они даже не похожи на декраторы или на view некий

Нет, $this->json(new OneTypeValidationErrorResponse($validationErrors)): $this->json(new SecondTypeValidationErrorResponse($validationErrors)):

Как мне в тот онетипвалидайшен прокинуть валидатор?

Ну тогда и назови их по человечески)) они даже не похожи на декраторы или на view некий

Как?

Как мне в тот онетипвалидайшен прокинуть валидатор?

Никак, из вадитора возвращаеются ошибки, вот эти ошибки.

Как?

Что они должны возвратить? Без внутренней реализации

Никак, из вадитора возвращаеются ошибки, вот эти ошибки.

Т.е. мне типов респонсов наложить?

Т.е. мне типов респонсов наложить?

Ну это типо view, template. Только в виде класса

Что они должны возвратить? Без внутренней реализации

Ошибки конечно же, в разных вариантах

Ошибки конечно же, в разных вариантах

Точнее

Выше уже написано же

Раз пять написал, а ты не читаешь )

В некоторых случаях надо ошибки отдавать с группировкой по полям, в некоторых просто список ошибок текстом. Ок ща

Тут тоже есть валидация)

Где у тебя тут валидация и ответ с ошибками? В аргумент ресолвер поди засунул?

Ну смотри, это не может быть декоратор, так как ты возвращаешь разные типы. Например валидатор возвращает некий результат (массив или класс) и его надо преобразовать в разные виды. Это ближе к паттеру мост, что как раз в MVC, есть данные, а есть View. Т.е. я или бы предложил как выше, сделать несколько классов ErrorResponse, Например StringValidateResponse, ArrayValidateResponse, (Можешь назвать по другому) и их создавать в контроллерах. Они оба принимают ошибки валидатора, а не сам валидтор. Или же Сделать какие то Трансформеры, как ты это и делал. Но как ты и сказал, 2 класса в DI для валидатора - не очень. Короче главное - это название нихера не понятное. Ну и не декоратор.

Где у тебя тут валидация и ответ с ошибками? В аргумент ресолвер поди засунул?

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

Тут тоже есть валидация)

А можно вопрос. В чем суть этого?

автору кажется что это круто выглядит 🙂

А можно вопрос. В чем суть этого?

В том, что можно убрать болерплейт с валидацией и формированием ошибок из контроллера

автору кажется что это круто выглядит 🙂

Меньше кода, меньше работы)

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

Задача ресолвера отресолвить аргумент, а не отвалидировать до кучи и обработать ошибки ещё. Бля, топите за солиды, и вот этой хернёй занимаетесь

Задача ресолвера отресолвить аргумент, а не отвалидировать до кучи и обработать ошибки ещё. Бля, топите за солиды, и вот этой хернёй занимаетесь

Резолвер не обрабатывает ошибки, он обрабатывает первичную валидацию. Вот одна ответственность

В том, что можно убрать болерплейт с валидацией и формированием ошибок из контроллера

Не понимать. Что ты возвращаешь хотя бы?

Не понимать. Что ты возвращаешь хотя бы?

Ничего, данный метод не требует ответа с данными

Резолвер не обрабатывает ошибки, он обрабатывает первичную валидацию. Вот одна ответственность

У него ответственност - подсунуть в контроллер то, что просят

У него ответственност - подсунуть в контроллер то, что просят

Да, тут ты прав. Или не подсунуть)

Не понимать. Что ты возвращаешь хотя бы?

Он command заполняет как я понял

По аналогии как устроено в ларавел

Правда там нет заполнения команд)

Правда там нет заполнения команд)

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