CodeIgniter控制器传递多上传字段变量至模型时某变量未定义

Hey there, let’s tackle this undefined variable issue you’re hitting with your CodeIgniter file uploads. I’ve run into similar snags before, so here are the most likely culprits and actionable fixes to check:

First off, CodeIgniter is case-sensitive when it comes to form input names. Make sure the name attribute on your sic_pic file input matches exactly what you’re passing to do_upload().

For example, if your form has:

<input type="file" name="sic_pic" />

Your controller must explicitly pass that name to the upload library—don’t rely on the default userfile name:

// Correct way for sic_pic
$this->upload->do_upload('sic_pic');
$sic_pic_name = $this->upload->data('file_name');

If you forget to pass 'sic_pic' to do_upload(), the library will look for a field named userfile instead, and $sic_pic_name will never get assigned.

It’s easy to skip error checking for one of the uploads, which leads to undefined variables if that upload fails (e.g., invalid file type, too large). Always wrap your upload logic in a conditional to set a fallback value:

$config['upload_path'] = './uploads/';
$config['allowed_types'] = 'jpg|png|jpeg';
$this->load->library('upload', $config);

// Process sic_pic with error handling
if (!$this->upload->do_upload('sic_pic')) {
    // Optional: log or display the error with $this->upload->display_errors()
    $sic_pic_name = null; // Or an empty string—something to avoid "undefined"
} else {
    $sic_pic_name = $this->upload->data('file_name');
}

Without this, if the upload fails, $sic_pic_name is never created, triggering the error when you pass it to the model.

If you’re defining the problematic variable inside a conditional block (like an if or foreach), it might not exist if the condition isn’t met. Always initialize variables outside the block first:

// Initialize first to avoid undefined errors
$sic_pic_name = null;

if ($this->input->post('has_sic_pic')) {
    $this->upload->do_upload('sic_pic');
    $sic_pic_name = $this->upload->data('file_name');
}

This way, even if the condition fails, $sic_pic_name exists (with a null/empty value) when you pass it to the model.

Double-check that your model’s save function accepts all three parameters, and that you’re passing them in the correct order. For example:

If your model has this function:

public function save_user_images($self_pic, $passport_pic, $sic_pic) {
    // Database save logic here
}

Make sure your controller calls it with all three variables:

// Correct: all three variables passed
$this->user_model->save_user_images($self_pic_name, $passport_pic_name, $sic_pic_name);

// Wrong: missing the third variable, causing $sic_pic to be undefined in the model
$this->user_model->save_user_images($self_pic_name, $passport_pic_name);

A mix-up in parameter order can also cause this—if you pass $sic_pic_name as the second argument, the model’s third parameter will be undefined.

When all else fails, add a quick debug check right before passing variables to the model:

var_dump($self_pic_name, $passport_pic_name, $sic_pic_name);
exit;

This will show you exactly which variable is undefined. If $sic_pic_name doesn’t show up at all, go back to the upload logic for that field—you probably missed assigning it. If it’s null or empty, check the upload errors with echo $this->upload->display_errors();.

内容的提问来源于stack exchange，提问作者Mulqan Junaidi