CodeIgniter多图上传异常排查：仅单张图片上传成功问题求助

嘿，我一眼就看出你多图上传失败的核心问题了——你没有针对循环里的每个文件单独处理，CodeIgniter的上传类默认只认单个userfile字段，而你直接循环却没给每个文件重新构造上传上下文。具体问题和修复方案如下：

你代码里的两个关键问题 #

1. 未针对循环内的单个文件重构上传数据
   多文件上传时，$_FILES["userfile"]是一个多维数组（每个属性如name/tmp_name都是数组），但CI的do_upload()默认只处理单个文件的结构。你循环时直接调用do_upload()，每次都只会处理数组里的第一个文件，后面的文件根本没被上传类识别到。

2. 文件名赋值错误
   上传成功时你写的$image = $_FILES['userfile']['name'];会直接拿到整个文件名数组，而不是当前循环的单个文件名，这会导致数据库里存的是数组字符串（比如Array）而不是正确的文件名。

修正后的完整代码 #

控制器部分 #

if(count($_FILES["userfile"]["name"]) == 0) { 
    $this->session->set_flashdata('success', '?? ????? ?????? ?????'); 
    redirect('accidents/index'); 
} else { 
    // 上传库配置
    $config['upload_path'] = './uploads/images'; 
    $config['allowed_types'] = 'gif|jpg|png|jpeg'; 
    $config['max_size'] = '2048000'; // 最大文件大小（KB）
    $config['max_width'] = '20000'; // 最大分辨率宽度
    $config['max_height'] = '20000'; // 最大分辨率高度

    // 提前创建上传目录（避免目录不存在导致上传失败）
    if (!is_dir($config['upload_path'])) {
        mkdir($config['upload_path'], 0755, true);
    }

    // 加载CI的上传类库
    $this->load->library('upload'); 
    
    for($count = 0; $count < count($_FILES["userfile"]["name"]); $count++) { 
        // 跳过上传有错误的文件（比如用户选中但未成功上传到服务器的文件）
        if ($_FILES["userfile"]["error"][$count] !== UPLOAD_ERR_OK) {
            $image = 'no_image.png';
            $this->accidents_model->addphoto($image,$last_id);
            continue;
        }

        // 构造当前循环的单个文件数据，适配CI上传类的要求
        $_FILES['userfile'] = array(
            'name'     => $_FILES["userfile"]["name"][$count],
            'type'     => $_FILES["userfile"]["type"][$count],
            'tmp_name' => $_FILES["userfile"]["tmp_name"][$count],
            'error'    => $_FILES["userfile"]["error"][$count],
            'size'     => $_FILES["userfile"]["size"][$count]
        );
        
        // 每次循环重置上传类配置，避免前一次上传的状态干扰
        $this->upload->initialize($config);
        
        // 尝试上传当前文件
        if(!$this->upload->do_upload()) { 
            // 可以把错误信息存起来用于调试，比如$this->session->set_flashdata('upload_errors', $this->upload->display_errors());
            $image = 'no_image.png'; 
        } else { 
            $upload_data = $this->upload->data(); 
            // 用CI上传后的实际文件名（如果有重名，CI会自动重命名，避免文件覆盖）
            $image = $upload_data['file_name']; 
        } 
        $this->accidents_model->addphoto($image,$last_id); 
    } 
} 

模型部分（无需修改，但若要增强可靠性可优化） #

public function addphoto($photo,$last_id) { 
    $data = array( 
        'cp_photo' => $photo, 
        'ac_id' => $last_id 
    ); 
    // 可选：返回插入结果，方便控制器判断是否插入成功
    return $this->db->insert('cars_photos', $data); 
}

额外优化建议 #

• 错误日志记录：可以把上传失败的错误信息收集起来，最后统一提示给用户或者存入日志，方便排查问题。
• 文件重命名配置：可以在$config里添加$config['encrypt_name'] = true;，让CI自动生成唯一的文件名，彻底避免重名覆盖问题。
• 前端验证：在前端先验证文件类型、大小，减少后端的无效处理。

内容的提问来源于stack exchange，提问作者Wesam Beshir