PHP/HTML多文件上传问题：未选图片仍生成记录

嘿，我来帮你搞定这个多文件上传的坑！你遇到的问题本质是单文件和多文件场景下$_FILES数组的结构差异导致判断逻辑失效，下面给你详细分析和修复方案：

问题根源 #

你原来的判断if(!empty($_FILES['gallery']['name']))在单文件场景下是有效的——没选文件时$_FILES['gallery']['name']是空字符串，empty()返回true，所以不会进入后续代码。

但在多文件场景（比如表单用了name="gallery[]"），情况就不一样了：

• 即使没选任何文件，浏览器可能会提交一个包含空字符串的数组（比如用户点开文件选择框又取消）
• 此时$_FILES['gallery']['name']是一个非空数组（哪怕元素是空），empty()对非空数组返回false，所以!empty(...)会变成true，导致循环执行，最终创建空记录。

修复后的代码 #

我们需要先过滤掉所有无效的文件项，只保留真正被上传的有效文件，再执行后续逻辑：

// 先收集所有有效的上传文件
$validUploads = [];

// 先确认上传的文件数组存在且是合法结构
if (isset($_FILES['gallery']) && is_array($_FILES['gallery']['tmp_name'])) {
    foreach ($_FILES['gallery']['tmp_name'] as $key => $tmpFile) {
        // 同时检查文件名不为空，且是通过HTTP POST上传的有效文件
        if (!empty($_FILES['gallery']['name'][$key]) && is_uploaded_file($tmpFile)) {
            $validUploads[] = [
                'tmp' => $tmpFile,
                'name' => $_FILES['gallery']['name'][$key]
            ];
        }
    }
}

// 只有当存在有效文件时，才执行上传和数据库操作
if (!empty($validUploads)) {
    foreach ($validUploads as $file) {
        $gallery_tmp = $file['tmp'];
        // 可以给文件名加个唯一前缀避免重复，这里用时间戳+下划线区分
        $gallery = time() . '_' . $file['name'];
        // 执行文件移动
        if (move_uploaded_file($gallery_tmp, "../uploads/$gallery")) {
            // 执行数据库插入逻辑
            $sql = mysqli_query($database->...);
            // 记得这里可以加错误处理，比如判断SQL是否执行成功
        }
    }
}

额外优化建议 #

• 加上文件类型和大小校验，避免上传恶意或不符合要求的文件
• 对文件名进行转义处理（比如替换特殊字符），防止路径遍历攻击
• 数据库操作时使用预处理语句，避免SQL注入风险

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