AM
Да, на завтра сделаю
Замечательно 👍
Size: a a a
2020 September 06
T
and those who do not have api and I want to use the validator with client validator ?
D
TradersVE
and those who do not have api and I want to use the validator with client validator ?
You need implementation of method getName on create your custom rule class only
D
ну да, я же это написал в ПР :)
можно подискусить на эту тему, возможно есть и другие решения
можно подискусить на эту тему, возможно есть и другие решения
Вообщем я пока остаюсь при мнении, что если ты делаешь свой класс правил - будь добр, дай ему имя.
Переменные же именовать никто не переломился. А будешь ли ты этот класс применять на фронте или только для внутренних процессов - дело второе.
Меньше магии, больше строгости.
Опять же IMHO
Переменные же именовать никто не переломился. А будешь ли ты этот класс применять на фронте или только для внутренних процессов - дело второе.
Меньше магии, больше строгости.
Опять же IMHO
AM
Норм именовать строкой.
AM
Имя класса во внешней системе, тем более, это очень плохо.
AM
Рефакторинг аля move to another namespace будет всё ломать.
D
Как вариант - можно сделать переменную под имя, по умолчанию пустую. И в месте применения (при использовании метода asArray) - кидать exception если имя не задано
АН
Ну если делаешь правило валидации, не думаю что составит труда, написать и имя...
Д
А названия классов тогда для чего нужны?
AM
А названия классов тогда для чего нужны?
Чтобы использовать классы?
Д
Ну вы говорите: "если собрался писать валидатор, дай ему имя".
Дам, в название класса заключу это имя.
Дам, в название класса заключу это имя.
D
Ну вы говорите: "если собрался писать валидатор, дай ему имя".
Дам, в название класса заключу это имя.
Дам, в название класса заключу это имя.
Ну тогда реализация в твоём случае будет простая
public function getName(): string
{
return static::class;
}
public function getName(): string
{
return static::class;
}
D
Но я бы всё таки исключил неймспейс из имени
Д
В каждом валидаторе?
AM
Но я бы так не делал.
AM
Это как называть внешние события именами классов в PHP-проекте.
AM
Потом ты не сможешь даже namespace поменять...
AM
Потому что развалится вообще в другом сервисе.
Д
Но я бы всё таки исключил неймспейс из имени
Давай, лишь бы не писать ненужное имя.