PHP通过循环上传多张图片仅成功1张的问题求解

核心故障原因 #

• 文件名生成逻辑错误：使用秒级时间戳round(microtime(true))作为文件名前缀，循环处理多图的耗时不足1秒，所有文件生成同名，后续文件直接覆盖前者，最终仅保留1张图片。
• 附带潜在问题：
  • 重复目录判断逻辑不完善，重新生成目录名后未再次校验、创建目录，可能导致上传失败
  • 原文件名存在性校验无效，实际保存用的是重命名后的文件名，校验逻辑完全不匹配
  • 数据库仅存储目录路径，未存储图片文件名，后续无法正常读取展示上传的多张图片

修复后PHP代码 #

function generateRandomString($length = 25) {
    $characters = '0123456789abcdefghijklmnopqrstuvwxyzABCDEFGHIJKLMNOPQRSTUVWXYZ';
    $charactersLength = strlen($characters);
    $randomString = '';
    for ($i = 0; $i < $length; $i++) {
        $randomString .= $characters[rand(0, $charactersLength - 1)];
    }
    return $randomString;
}
if (isset($_POST['newYoungSubmit'])) {

    $imageDirectory = "images/youngMembers";
    // 循环生成目录直到确认无重复
    do {
        $newDirName = generateRandomString(20);
        $targetDir = $imageDirectory."/".$newDirName."/";
    } while(file_exists($targetDir));
    mkdir($targetDir, 0777, true);

    $postDate = date("Y-m-d");
    // 存储所有上传成功的文件名 后续存入数据库
    $uploadedFiles = [];
    // Count total files
    $fileCount = count($_FILES['new_young']['name']);
    // Iterate through the files
    for($i = 0; $i < $fileCount; $i++){
        $target_file = $targetDir . basename($_FILES["new_young"]["name"][$i]);
        $uploadOk = 1;
        $imageFileType = strtolower(pathinfo($target_file,PATHINFO_EXTENSION));
        // Check if image file is a actual image or fake image
        $check = getimagesize($_FILES["new_young"]["tmp_name"][$i]);
        if($check !== false) {
            $uploadOk = 1;
        } else {
            $errorTxt = "File is not an image.";
            $uploadOk = 0;
        }
        // 修正文件存在校验 校验重命名后的文件
        $temp = explode(".", $_FILES["new_young"]["name"][$i]);
        // 文件名前缀加上循环变量$i 保证同批次文件不会重名
        $newfilename = $i . '_' . round(microtime(true)) . '.' . end($temp);
        $savePath = $targetDir . $newfilename;
        if (file_exists($savePath)) {
            $errorTxt =  "Sorry, file already exists.";
            $uploadOk = 0;
        }
        // Allow certain file formats
        if($imageFileType != "jpg" && $imageFileType != "png" && $imageFileType != "jpeg"
        && $imageFileType != "gif" ) {
            $errorTxt = "Sorry, only JPG, JPEG, PNG & GIF files are allowed.";
            $uploadOk = 0;
        }
        // Check if $uploadOk is set to 0 by an error
        if ($uploadOk == 0) {
            // 可在这里加错误日志记录
        // if everything is ok, try to upload file
        } else {
            if (move_uploaded_file($_FILES["new_young"]["tmp_name"][$i], $savePath)) {
                $uploadedFiles[] = $newfilename;
            } else {
                $errorTxt = "Sorry, there was an error uploading your file.";
            }
        }
    }

    // 把上传成功的文件名转成JSON存入数据库 后续读取解析即可
    $newYoungQuery = $conn->prepare("INSERT INTO youngMembers (youngTitle, youngImgDir, youngDesc, youngImgList) VALUES (?,?,?,?)");
    $newYoungQuery->execute([$_POST['youngTitle'], $targetDir, $_POST['youngDesc'], json_encode($uploadedFiles)]);
    $newYoungQuery = null;
}

调整说明 #

1. 文件名生成逻辑增加了循环变量$i作为前缀，保证同批次上传的多张图片文件名不会重复
2. 优化了目录生成逻辑，循环校验目录唯一性，确认无重复后再创建目录
3. 修正了文件存在性校验逻辑，直接校验最终保存的路径
4. 新增youngImgList字段存储所有上传成功的文件名列表，方便后续读取展示，你需要先在youngMembers表中新增文本类型的youngImgList字段即可使用。

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