PHP用户资料更新表单：未传新图时旧图被删问题求助

我帮你找出问题所在了！你的代码里判断文件是否上传的逻辑有误，导致没传新头像时旧头像被清空，下面是具体分析和修复方案：

问题根源 #

你原来的判断条件 isset($_FILES['profile_image']['name']) 并不准确——即使用户没有选择任何文件上传，这个键依然存在，只是它的值是空字符串。这就导致当用户没上传新头像时，代码会进入第一个if分支，把$profile_image赋值为空字符串，随后执行UPDATE语句时就会把数据库里的头像字段清空。

修复方案 #

我们需要正确判断用户是否真的上传了文件，并且确保没上传时保留旧头像的值。

修改后的PHP代码 #

if(isset($_POST['update_profile'])){ 
    // 第一步：先获取数据库中当前用户的旧头像，作为默认值
    $query = "SELECT profile_image FROM profiles WHERE profile_id = $profile_id"; 
    $result = mysqli_query($connection, $query); 
    $row = mysqli_fetch_array($result); 
    $profile_image = $row['profile_image']; 

    // 第二步：正确判断是否有文件上传且上传成功
    // 使用UPLOAD_ERR_OK常量检查上传是否无错误
    if(isset($_FILES['profile_image']) && $_FILES['profile_image']['error'] == UPLOAD_ERR_OK){ 
        $profile_image_temp = $_FILES['profile_image']['tmp_name']; 
        $profile_image = $_FILES['profile_image']['name']; 
        // 执行文件上传（建议这里可以添加文件名去重逻辑，避免覆盖同名文件）
        move_uploaded_file($profile_image_temp, "../includes/profile_images/$profile_image"); 
    } 

    // 第三步：执行更新操作（这里建议使用预处理语句防止SQL注入）
    // 安全写法：使用mysqli预处理
    $update_profile = "UPDATE profiles SET profile_image = ? WHERE profile_id = ?";
    $stmt = mysqli_prepare($connection, $update_profile);
    mysqli_stmt_bind_param($stmt, "si", $profile_image, $profile_id);
    $update_result = mysqli_stmt_execute($stmt);

    if(!$update_result){
        die("更新失败: " . mysqli_error($connection));
    }
}

修改后的HTML代码 #

另外，HTML中的文件输入框无法通过value属性回显已上传的文件名（浏览器出于安全限制禁止这么做），我们可以在输入框上方显示当前用户的头像，让用户更直观看到现有头像：

<div class="contact-clean col-md-5" style="margin:0 auto; padding-top: 50px;"> 
    <h2 class="text-center">Update Profile</h2> 
    <p class="text-center"><small>Fields with an asterisk are required </small></p>
    
    <!-- 显示当前用户的头像 -->
    <div class="text-center mb-3">
        <img src="../includes/profile_images/<?php echo htmlspecialchars($profile_image); ?>" 
             alt="Your Current Profile Image" 
             style="width: 150px; height: 150px; object-fit: cover; border-radius: 50%;">
    </div>

    <form method="post" action="#" enctype="multipart/form-data"> 
        <div class="form-group"> 
            <label for="profile_image">Profile Image (leave blank to keep current)</label>
            <!-- 移除无效的value属性 -->
            <input class="form-control" type="file" name="profile_image" id="profile_image"> 
        </div> 
        <button style="margin-top: 20px; margin-bottom: 300px;" class="btn btn-primary btn-lg" type="submit" name="update_profile" id="update_profile">Update Profile</button> 
    </form> 
</div>

额外建议 #

• 防止SQL注入：我在PHP代码里替换成了预处理语句，避免直接把变量拼接到SQL中，这能有效防止SQL注入攻击。
• 文件名去重：如果多个用户上传同名头像，会覆盖之前的文件，建议给文件名添加唯一标识（比如用户ID+时间戳），比如：$profile_image = $profile_id . '_' . time() . '_' . $_FILES['profile_image']['name'];

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