Можно ли смешивать логику валидации с методами валидации в одном классе?

Question

[thorii]

Есть класс Validation. В конструктор передаются правила валидации

$validation = new Validation([
    'FieldName' => [
        'filtername' => ['filterArg1' => 'value'] //Example ['filterLen' => ['max' => 10, 'min' => 5]]
    ]
])

if($validation->isValid(['FieldName' => 'Value']) {
    // is valid, continue work;
} else echo $validation->getMessage();

В валидаторе пока один метод isValid(array $data) который пробегается по фильтрам (данные передаются в формате 'ИмяПоля' => 'Значение');

Так вот можно ли в этом же классе определить методы для валидации (а пока я их вынес в отдельный класс Filters). Будет ли это считаться смешиванием логики?

Answer 1

[xmoonlight]

По логике:
1. Фильтрация (ожидаемый набор данных)
2. Валидация (проверка значения)
Я бы фильтрацию вставил бы приватным методом в класс Validation и прописал бы внутри-классовые зависимости в зависимости от параметров в методе валидации.

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

Comments

[thorii]

Я как всегда оговорился в вопросе. Под "filters" Я имею в виду методы валидации. Или проще: может ли класс validation содержать методы validateRequired, ValidateLen, ValidateEmail, Validate*

[thorii]

То есть в yii например, создается отдельный класс ValidateEmail.php например и так далее (не помню как они точно называются). Я же не хочу плодить сотню классов с единственным методом внутри

[thorii]

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

Я пока не использую такие прелести как DI, Singleton или Factory. Поэтому валидацию вызывает моя модель, тогда когда надо и сколько надо (1 форма один экземпляр)