Как практичнее выводить сообщения о невалидных данных?

Question

[thorii]

Есть класс Validation. В конструктор передаются правила валидации. Я склоняюсь к такому варианту, но получается что ключ msg передается в сам метод валидации и приходится в каждом методе искать этот ключ и городить такие конструкции return (isset($options['msg']) ) ? $options['msg'] : false;

$validation = new Validation([
    'FieldName' => [
        'validateAs' => ['ValidateArg' => 'value', 'msg' => 'FieldsName in invalid.'] 
    ]
])

Нормальна ли такая реализация, или есть способы получше?

Answer 1

[xmoonlight]

Возвращайте в переменную (через return) ассоциативный массив (вместо объекта или значения):

'ValidationError'=>array(
  [0]=>array(
            'msg'=>'Переменная должна быть в диапазоне: 0-100',
            'FieldName'=>'percent',
            'FiledValue'=>150
          )
);

Comments

[thorii]

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

[xmoonlight]

thorii: в значение у вас попадёт ассоциативный массив с ошибкой! Я же написал! Читайте! Просто читайте!

[thorii]

xmoonlight: Хорошо. С ассоциативным массивом мне понятно, а где его формировать? В валидаторах (validateEmail...validate*) или в методе старта валидации isValid в случае false?

[xmoonlight]

thorii: массив ошибки - это внутриклассовая переменная private $error;. Добавлять туда - по мере необходимости внутри разных методов. Перед возвратом значения, проверяем: если массив не пустой - return [этот массив].

[thorii]

xmoonlight: Мне нужно знать другое (простите если не понимаю вас сразу). Какой метод должен "формировать массив ошибки и отправлять в $error"? Это делить при проходе по валидаторам в цикле?

[xmoonlight]

thorii: если я правильно понял, Вы перебиаете из isValid все методы и соотв. возвращать Вы должны из него же.

[thorii]

xmoonlight: Ясно, спасибо. В дополнение: "не люблю работать с целым массивом ошибок (их по вьюхе придется возвращать же), а если полей в форме 10-15-*(утрирую), блок ошибок очень растянется, а костыли в виде обрезания массива во вьюхе я писать не желаю(да и вовсе, невалидно одно поле = не валидна вся форма). Пусть уж в js будет доп. валидация, а php вернет первую найденую ошибку, и так пока все не будет исправлено"

[xmoonlight]

thorii: после обнаружения (в isValid) или вывода во вьюхе 1-ой ошибки, Вас никто не заставляет проверять/выводить остальные. Это уж сами как хотите решайте.

[xmoonlight]

thorii: помните про break в цикле)

[thorii]

xmoonlight: В общем, спасибо за наставление!

Answer 2

[Оптимус Пьян]

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

Comments

[thorii]

Мой метод isValid($data) пробежит данными из $data по методам валидации (validateEmail() например) и вернет в случае false остановится и выкинет false а сообщение запишется в переменную $error которое я могу получить из $validation->getError(). Проблема в том, что с сообщением работает не сам валидатор а его методы валидации (validateEmail) получается дублирование кода в одну строку во всех методах, нормально ли так? Если неясно объясняю, то ниже есть код

[thorii]

В общем код в комментах с подсветкой

Answer 3

[thorii]

<?php
namespace System\Validation;

use System\Core;

class Validator {
	
	private $config;
	private $rules;
        private $error;




	public function __construct($rules) { // Для каждой формы, свой экземпляр (new Validator($rules))
		$this->config = Core::libraryConfig(__CLASS__);
                $this->rules = $rules;
	}
	
	public function isValid($data) {
        while(list($field, $value) = each($data)) {
            $rules = $this->rules[$field];
            foreach($rules as $rule => $options) {
                $method = 'validate'.$rule; //тут приписывается префикс
                $valid = $this->$method($value, $options); //@return true|false|string Error Message
                if($valid) {
                  return TRUE;
               } else return $valid; //Это ужасный кусок, это я тоже хочу в порядок привести. А может внутри метода записывать $this->error и вернуть false?
            }
        }
        return TRUE;
    }

    public function validateEmail($value, $options = []) {
        if(filter_var($value, FILTER_VALIDATE_EMAIL)) {
            return TRUE;
        } else return (isset($options['msg'])) ? $options['msg'] : FALSE; //Вот эти строки вечно дублируются
//Может просто вырезать ключ msg в методе валидации и работать уже там с сообщением??
    }

    public function validateRequired($value, $options = []) {
        if(!is_null($value) || !empty($value)) {
            return TRUE;
        } else return (isset($options['msg'])) ? $options['msg'] : FALSE; //Вот тут метод вернет сообщение если оно передано, иначе false соответственно
//Идет дублирование, разве он должен оперировать еще и сообщениями ошибок? Нет. А ума не хватает переделать
    }

}
?>

$validateRules = [
    'firstname' => [
     'required' => ['msg' => 'First name can\'t be empty']
    //Прочие валидаторы...
    ],
      'lastname' => [
      'required' => ['msg' => 'Lastname can\'t be empty']
    ]
];

$validator = new Validator($validateRules);
var_dump($validator->isValid(['firstname' => NULL]));

Comments

[thorii]

Прошу прощение за отступы