PHP CRUD项目中实现按用户ID创建独立文件夹存储上传文件的问题求助

我来帮你一步步排查并修复这些错误，咱们先拆解每个问题的原因，再给出修正后的完整代码：

错误原因分析 #

1. Undefined variable $id
   你代码里误用了$id变量，但实际存储用户ID的变量是$loginId，这是典型的变量名拼写错误。

2. mkdir(): File exists
   mkdir()的用法有误：它的第一个参数是要创建的目标文件夹路径，第二个参数是权限值（可选）。你把用户ID作为第二个参数传入，导致每次都尝试创建已经存在的主images文件夹，自然触发“文件已存在”的警告。

3. move_uploaded_file权限&路径错误
   mkdir()的返回值是布尔值（成功返回true，失败返回false），你把这个布尔值当成文件夹路径去拼接，直接导致文件保存路径变成了无效的根路径（比如/8037-bird1.jpg），这也是权限报错的核心原因——PHP尝试往系统根目录写文件，肯定会被拒绝。

修正后的完整代码 #

if(isset($_POST['Submit'])) { 
    $name = $_POST['name']; 
    $qty = $_POST['qty']; 
    $price = $_POST['price']; 
    $loginId = $_SESSION['id']; 

    // 生成带随机前缀的唯一文件名，避免重名覆盖
    $pname = rand(1000,10000)."-".$_FILES["file"]["name"]; 
    $tname = $_FILES["file"]["tmp_name"]; 

    // 用相对路径更灵活（和add.php同目录的images文件夹）
    $uploads_dir = './images'; 
    // 拼接用户专属文件夹的完整路径
    $user_dir = $uploads_dir . '/' . $loginId; 

    // 检查用户文件夹是否存在，不存在则创建（带递归权限）
    if (!is_dir($user_dir)) {
        // 0755权限：所有者读写执行，其他用户读执行，兼顾安全和可用性
        mkdir($user_dir, 0755, true); 
    }

    // 拼接最终的文件保存路径
    $save_path = $user_dir . '/' . $pname; 

    // 执行文件移动，并做结果判断
    if(move_uploaded_file($tname, $save_path)){
        echo "文件上传成功！";
        // 这里可以添加将文件路径存入数据库等后续逻辑
    } else {
        echo "文件上传失败，请检查文件夹权限！";
    }
}

额外注意事项 #

• 优先使用相对路径：./images比硬编码的绝对路径更适配不同环境，避免换服务器后路径失效。
• 权限设置：确保images主文件夹和自动创建的用户文件夹权限为0755或0775，不要用0777（会带来安全风险）。
• 安全校验：建议添加文件类型、大小的验证逻辑，防止恶意文件上传。
• Session判断：在使用$_SESSION['id']之前，最好先判断用户是否已登录（比如if(!isset($_SESSION['id'])) { exit("请先登录！"); }），避免创建空名称的无效文件夹。

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