使用 CodeIgniter + MySql 进行表单验证不起作用


Form Validation with CodeIgniter + MySql Not Working

我似乎无法使用Codeigniter进行表单验证。我尝试通过创建My_Form_validation.php来扩展Form_validation类,但没有成功。我现在正在尝试回调方法。我有一段时间出现错误,但是它们不正确。

这是位于我的控制器中的代码:

 function create_user() {
    $this->load->library('form_validation');
    $validate = array(
        array(
            'field' => 'first_name',
            'label' => 'First Name',
            'rules' => 'trim|required|xss_clean'
        ),
        array(
            'field' => 'last_name',
            'label' => 'Last Name',
            'rules' => 'trim|required|xss_clean'
        ),
        array(
            'field' => 'username',
            'label' => 'Username',
            'rules' => 'trim|required|xss_clean|callback_user_exists'
        ),
        array(
            'field' => 'email_address',
            'label' => 'Email Address',
            'rules' => 'trim|required|valid_email|callback_email_exists'
        ),
        array(
            'field' => 'password',
            'label' => 'Password',
            'rules' => 'trim|required|min_length[5]|max_length[32]'
        ),
        array(
            'field' => 'password2',
            'label' => 'Confirm Password',
            'rules' => 'trim|required|matches[password]'
        )
    );
    $this->form_validation->set_rules($validate);
    if($this->form_validation->run() == FALSE) {
        $this->load->view('user/user-signup');      
    } else {
        $this->load->model('user_model');
        if($query = $this->user_model->create_user()) {
            $this->load->view('user/user-login');
        } else {
            $this->index();
        }
    }
}
function user_exists($username) {
    $this->load->model('user_model');
    $this->user_model->user_exists($username);
    $this->form_validation->set_message('user_exists', 'This username is already taken');
}
function email_exists($email) {
    $this->load->model('user_model');
    $this->user_model->email_exists($email);
    $this->form_validation->set_message('email_exists', 'This email is already in use');
}

这是位于我的模型中的代码:

function create_user() {
    $insert_user = array(
        'first_name'    => $this->input->post('first_name'),
        'last_name'     => $this->input->post('last_name'),
        'username'      => $this->input->post('username'),
        'email_address' => $this->input->post('email_address'),         
        'password'      => md5($this->input->post('password'))                      
    );
    $insert = $this->db->insert('users', $insert_user);
    return $insert;
}

function user_exists($username) {
    $this->db->where('username', $username);
    $query = $this->db->get('users');
    if($query->num_rows > 0) {
        return true;
    } else {
        return false;
    }
}

function email_exists($email) {
    $this->db->where('email_address', $email);
    $query = $this->db->get('users');
    if($query->num_rows > 0) {
        return true;
    } else {
        return false;
    }
}

我想通过检查数据库中是否已存在用户名或电子邮件地址来验证,如果是,用户将需要进行适当的更改。

有什么想法吗?

你的代码很难阅读,所以我会告诉你如何改进它。 :)

在控制器中,您可以使用构造函数来加载模型,而不是以下两行:

$this->load->model('user_model');

喜欢这个:

function __constructor() {
    parent::__constructor();
    $this->load->model('user_model');
}

将user_exists回调更改为:

function user_exists($username) {
    $user_check = $this->user_model->user_exists($username);
    if($user_check > 0) {
        $this->form_validation->set_message('user_exists', 'This username is already taken');
        return FALSE;
    }
    else {
        return TRUE;
    }
}

将email_exists回调更改为以下内容:

function email_exists($email) {
    $check_email = $this->user_model->email_exists($email);
    if($check_email > 0) {
        $this->form_validation->set_message('email_exists', 'This email is already in use');
        return FALSE;
    }
    else {
        return TRUE;
    }
}

现在,返回到您的模型并更改这两个模型方法:

function user_exists($username) {
    $this->db->where('username', $username);
    $query = $this->db->get('users');
    return $query->num_rows();
}
function email_exists($email) {
    $this->db->where('email_address', $email);
    $query = $this->db->get('users');
    return $query->num_rows();
}

现在,你做错了,因为你不明白模型的含义。 在模型方法中,您可以编写数据库查询...因此,如果要创建用户,则应在控制器中获取输入信息,然后将其传递给模型方法create_user,如下所示:

控制器方法create_user:

function create_user() {
    $this->load->library('form_validation');
    $validate = array(
        array(
            'field' => 'first_name',
            'label' => 'First Name',
            'rules' => 'trim|required|xss_clean'
        ),
        array(
            'field' => 'last_name',
            'label' => 'Last Name',
            'rules' => 'trim|required|xss_clean'
        ),
        array(
            'field' => 'username',
            'label' => 'Username',
            'rules' => 'trim|required|xss_clean|callback_user_exists'
        ),
        array(
            'field' => 'email_address',
            'label' => 'Email Address',
            'rules' => 'trim|required|valid_email|callback_email_exists'
        ),
        array(
            'field' => 'password',
            'label' => 'Password',
            'rules' => 'trim|required|min_length[5]|max_length[32]'
        ),
        array(
            'field' => 'password2',
            'label' => 'Confirm Password',
            'rules' => 'trim|required|matches[password]'
        )
    );
    $this->form_validation->set_rules($validate);
    if($this->form_validation->run() == FALSE) {
        $this->load->view('user/user-signup');      
    } else {
        $user_data['first_name'] = $this->input->post("first_name");
        $user_data['last_name'] = $this->input->post("last_name");
        $user_data['username'] = $this->input->post("username");
        $user_data['email_address'] = $this->input->post("email_address");
        $user_data['password'] = $this->input->post("password");
        if($query = $this->user_model->create_user($user_data)) {
            $this->load->view('user/user-login');
        } else {
            $this->index();
        }
    }
}

模型的方法create_user:

function create_user($user_data) {
    return $this->db->insert("users", $user_data);
}

仅此而已,它会起作用。祝你好运。

你试过is_unique[table_name.field_name]规则吗?

例:

$this->form_validation->set_rules('username', 'Username',
 'required|min_length[5]|max_length[12]|is_unique[users.username]');
$this->form_validation->set_rules('email', 'Email',
 'required|valid_email|is_unique[users.email]');

更新

如果要使用回调函数,则user_exists函数应位于控制器中,而不是如您所提到的模型中。正确的定义方法是——

public function username_check($str)
{
    if ($str == 'test')
    {
        $this->form_validation->set_message('username_check', 'The %s field can not be the word "test"');
        return FALSE;
    }
    else
    {
        return TRUE;
    }
}

像这样重写你的函数

function user_exists($username) {
$this->load->model('user_model');
$result = $this->user_model->user_exists($username);
if($result != NULL){
    $this->form_validation->set_message('user_exists', 'This username is already taken');
    return FALSE;
}else{
    return TRUE;
}
}

您没有返回 true 或 false,因此它总是得到 xss_clea 返回的最后一个 true。

我遇到了同样的问题。回调函数的问题之一是它只能接受一个参数。检查表单中记录的唯一性时,需要考虑两种状态。1) 您正在添加新记录2) 您正在编辑现有记录。

如果要添加新记录,则内置is_unique工作正常。如果您正在编辑现有记录is_unique则不起作用,因为它会找到您正在编辑的记录并显示表单数据不是唯一的。

为了解决这个问题,我使用了 Session 类,在运行验证脚本之前将其设置为案例 2,因此您需要知道您是正在编辑现有记录还是添加新记录。 为此,我只需在编辑表单时向表单添加一个隐藏的输入,例如记录唯一 ID。
大概您在 users 表中有一个唯一的用户 ID,例如在运行验证之前设置它。

if($this->input->post('user_id')){$this->session->set_userdata('callback_user_id',$this->input->post('user_id'));}

然后在回调中,使用这种算法:

案例 1) 即 $this->会话>用户数据('callback_user_id') == FALSE如果用户名是唯一的,则验证并返回 true。如果用户名不唯一,则返回 false 并显示验证消息"用户必须是唯一的"。

情况2)即,设置callback_user_id。如果用户名是唯一的,则验证并返回 true如果已设置用户名,并且该记录与user_id具有相同的 ID,则表示您正在更新相同的记录,并且验证起来很好。否则,另一条记录具有用户名,并且验证失败。在模型中,我只有一个返回用户名唯一 ID 的方法。

运行验证后,最好取消设置 callback_user_id 会话变量。很抱歉我没有要粘贴的代码,但我认为此描述应该对您有所帮助。

==== 编辑如今,我认为用新功能覆盖表单验证是要走的路。所以:有一个语言包条目、一个表单验证行和覆盖:这假定发布名为 ID 的字段具有行的 id。

$lang['form_validation_is_unique_not_current'] ='The {field} field must contain a unique value.';
array('field' => 'username', 'label' => 'lang:…username…', 'rules' => 'trim|required|min_length[2]|max_length[40]|is_unique_not_current[users.username]'),

class MY_Form_validation extends CI_Form_validation {
    function __construct($rules = array())
    {
        parent::__construct($rules);
        $this->_error_prefix = '<div class="alert alert-danger"><p>';
        $this->_error_suffix = '</p></div>';
    }
    public function is_unique_not_current($str, $field)
    {
        sscanf($field, '%[^.].%[^.]', $table, $field);
        $id = $this->CI->input->post('id');
        if($this->CI->input->post('field_name'))
        {
            return isset($this->CI->db)
                ? ($this->CI->db->limit(1)->get_where($table, array(
                    $field => $str,
                    'id <> ' => $id))->num_rows() === 0)
                : FALSE;
        }
        return FALSE;
    }
}