即使存在错误,PHP表单验证也会通过

时间:2015-02-10 20:13:38

标签: php forms

我正在尝试开发一个表单验证系统。但问题是即使用户没有根据需要提供数据验证通过。无法找出问题所在。

这是我的表单验证类

 class Validation
 {

private $_passed = false,
        $_errors = array(),
        $_db = null;

public function __construct() {
        $this->_db = DB::getInstance();
    }
public function check($source, $items= array()) {
        foreach ($items as $item => $rules) {
            foreach ($rules as $rule => $rule_value) {
                $value = $source[$item];

                if ($rule === 'required' && empty($value)) {
                    $this->addError("{$item} is required");
                } else {

                }
            }
        }

        if (empty($this->_errors)) {
            $this->_passed = true;
        }
        return $this;

    }

private function addError($error) {
        $this->errors[] = $error;
    }
public function errors() {
        return $this->_errors;
    }
public function passed() {
        return $this->_passed;
    }
 }

这是包含Html表单的表单页面。

require_once 'core/init.php';

if (Input::exists()) {
    $validate = new Validation();
    $validation = $validate->check($_POST, array(
        'username' => array(
            'required' => true,
            'min' => 2,
            'max' => 20,
            'unique' => 'users'
            ),
        'password' => array(
            'required' => true,
            'matches' => 'password'
            ),
        'password_again' => array(
            'required' => true,
            'min' => 6
            ),
        'name' => array(
            'required' => true,
            'min' => 2,
            'max' => 60
            ),
        ));
    if ($validation->passed()) {
        //register new user
        echo "passed"; //this passes even though users provides no data
    } else {
        print_r($validation->errors());
    }
}

所以,即使用户根本没有提供任何数据,我得到的只是在屏幕上回显传递。它应该抛出错误。请帮忙。感谢

1 个答案:

答案 0 :(得分:1)

addError$this->errors中写入,而其他方法则使用$this->_errors。 (带下划线)。 _errors数组将保持为空,因此在此语句中_passed将设置为true:

if (empty($this->_errors)) {
    $this->_passed = true;
}