PHP多帖子图片上传问题：同名图片覆盖旧文件的解决需求

问题根源 #

当前代码直接使用用户上传的原始文件名保存图片，当有用户上传同名文件时，新文件会覆盖旧文件，同时旧帖子关联的图片路径指向被覆盖的新文件，导致旧帖子显示错误。

解决方案 #

通过生成唯一文件名避免覆盖，同时将唯一文件名存入数据库，确保每个帖子关联的图片路径唯一有效。常用生成方式是结合uniqid()（生成唯一ID）或时间戳+随机字符串，再拼接原始文件扩展名。

修改后的完整代码 #

<?php 
    session_start(); // 移至最前面，避免headers already sent错误
    include "config.php"; // Connection with server

    // File Upload
    $unique_filename = '';
    if (isset($_FILES['fileToUpload'])) {
        $Errors = array();

        $Original_File_Name = $_FILES['fileToUpload']['name'];
        $File_Size= $_FILES['fileToUpload']['size'];
        $File_Tmp = $_FILES['fileToUpload']['tmp_name'];
        $File_Type = $_FILES['fileToUpload']['type'];
        $File_ext = strtolower(end(explode('.',$Original_File_Name))); // 转为小写，统一扩展名格式
        $Extensions = array('jpeg','jpg','png');
        
        if (!in_array($File_ext,$Extensions)) {
            $Errors[] = "仅允许上传jpeg、jpg、png格式的图片";
        }
        if ($File_Size > 2097152) {
            $Errors[] = "图片大小不能超过2MB";
        }
        
        if (empty($Errors)) {
            // 生成唯一文件名：前缀+唯一ID+扩展名，确保唯一性
            $unique_filename = uniqid('post_', true) . '.' . $File_ext;
            $upload_path = "upload/" . $unique_filename;
            
            // 检查文件上传是否成功
            if (!move_uploaded_file($File_Tmp, $upload_path)) {
                $Errors[] = "文件上传失败，请检查upload目录权限";
            }
        }
        
        if (!empty($Errors)) {
            print_r($Errors);
            die();
        }
    }

    $Title = mysqli_real_escape_string($conn,$_POST["post_title"]);
    $Description = mysqli_real_escape_string($conn,$_POST["postdesc"]);
    $Category = mysqli_real_escape_string($conn,$_POST["category"]);
    $date = date("D M,Y");
    $Auther = $_SESSION['name'];

    // 使用生成的唯一文件名存入数据库
    $sql = "insert into post (title,description,category,post_date,author,post_img) values ('{$Title}', '{$Description}', {$Category}, '{$date}', '{$Auther}','{$unique_filename}');";
    $sql .= "Update category set post = post+1 where category_id = $Category;";
    
    if (mysqli_multi_query($conn, $sql)) {
        mysqli_close($conn);
        header('Location: http://'.$hostname.'/cms/admin/post.php');
        exit(); // 跳转后终止代码执行，避免冗余操作
    } else {
        echo "Query Failed";
    }
?>

关键修改点说明 #

1. 唯一文件名生成：用uniqid('post_', true)生成带前缀的高熵唯一ID，再拼接原始扩展名，几乎完全避免文件名重复。
2. 扩展名统一处理：将扩展名转为小写，避免因用户上传.JPG/.jpg等大小写差异导致的格式判断错误。
3. session位置调整：把session_start()移到代码最开头，防止因提前输出内容导致的header跳转错误。
4. 上传结果校验：增加move_uploaded_file的结果判断，及时捕获目录权限不足等上传失败场景。
5. 变量名区分：用$Original_File_Name存原始文件名，$unique_filename存生成的唯一文件名，避免逻辑混淆。

额外建议 #

• 提前确保upload目录存在且拥有写入权限，可通过mkdir("upload", 0755, true)自动创建多级目录。
• 若需极端严谨的唯一性，可在生成文件名后检查目录中是否已存在该文件，若存在则重新生成（概率极低）。

内容的提问来源于stack exchange，提问作者Mohsin Ullah Shah