PHP实现用户头像上传更新数据库无响应问题求助

兄弟，我扫了一遍你的代码，发现几个关键问题直接导致上传按钮点击后毫无反应，数据库也没更新。咱们一步步来修复：

1. 最致命的触发逻辑错误 #

你PHP代码里的核心判断逻辑有问题！
在else if ($_POST['action'] == 'Upload')分支里，你又加了一层if (isset($_POST['upload']))，但你的表单里根本没有name="upload"的控件啊！你的提交按钮name是action，值是Upload，所以这个内部判断永远不会成立，上传逻辑完全没执行到！

修复方法： #

直接删掉if (isset($_POST['upload']))这层判断，或者换成检查文件是否正常上传的判断：

} else if ($_POST['action'] == 'Upload') { 
    // 关掉错误屏蔽，调试必须开！
    error_reporting(E_ALL);
    ini_set('display_errors', 1);
    $msg = "";
    // 先确保session已启动！
    if(!isset($_SESSION)) session_start();
    $name = $_SESSION["user"];
    
    // 检查文件是否成功上传
    if (isset($_FILES["uploadfile"]) && $_FILES["uploadfile"]["error"] == UPLOAD_ERR_OK) {
        // 后续上传逻辑...
    }

2. 其他必须修复的问题 #

① 错误报告被禁用 #

你写了error_reporting(0)，这会把所有错误都藏起来，调试的时候完全看不到问题！赶紧改成上面代码里的开启错误报告，这样能直接看到比如session没启动、数据库连接失败之类的报错。

② 严重的SQL注入风险 #

直接把$name和$filename拼进SQL语句里，这是送分题级别的注入漏洞！必须用预处理语句：

// 替换原来的UPDATE语句
$sql = "UPDATE users SET avatar=? WHERE user=?";
$stmt = mysqli_prepare($link, $sql);
// 绑定参数："ss"表示两个字符串类型的参数
mysqli_stmt_bind_param($stmt, "ss", $uniqueFilename, $name);
if(mysqli_stmt_execute($stmt)){
    $msg = "数据库更新成功";
} else {
    $msg = "数据库更新失败：" . mysqli_error($link);
}

③ Session未初始化 #

你用了$_SESSION["user"]，但如果代码开头没调用session_start()，这个变量就是空的，UPDATE语句根本找不到对应用户！必须在所有输出之前加上session_start();，比如PHP文件最顶部。

④ 文件上传的错误处理 #

你没检查文件上传是否有错误（比如文件过大、临时文件丢失），加上这步能快速定位上传失败的原因：

if ($_FILES["uploadfile"]["error"] !== UPLOAD_ERR_OK) {
    switch ($_FILES["uploadfile"]["error"]) {
        case UPLOAD_ERR_INI_SIZE:
            $msg = "上传文件超过php.ini限制的大小";
            break;
        case UPLOAD_ERR_FORM_SIZE:
            $msg = "上传文件超过表单设置的大小";
            break;
        case UPLOAD_ERR_NO_FILE:
            $msg = "请选择要上传的文件";
            break;
        default:
            $msg = "文件上传出错，错误码：" . $_FILES["uploadfile"]["error"];
    }
    // 终止后续逻辑
    echo $msg;
    exit;
}

⑤ 文件名重名问题 #

直接用原文件名存到image/文件夹，很容易被同名文件覆盖，建议生成唯一文件名：

$filename = $_FILES["uploadfile"]["name"];
$tempname = $_FILES["uploadfile"]["tmp_name"];
// 获取文件扩展名
$ext = pathinfo($filename, PATHINFO_EXTENSION);
// 生成唯一文件名
$uniqueFilename = uniqid() . "." . $ext;
$folder = "image/" . $uniqueFilename;

⑥ 表单Action问题 #

表单的action=""可能会导致提交到错误的页面，建议明确指定当前页面：

<form method="POST" action="<?php echo $_SERVER['PHP_SELF']; ?>" enctype="multipart/form-data">

调试步骤建议 #

1. 先开启错误报告，刷新页面看有没有报错；
2. 修复触发逻辑的问题，让上传代码能执行到；
3. 用var_dump($_SESSION["user"])检查当前用户是否正确获取；
4. 用var_dump($_FILES["uploadfile"])看文件上传的信息是否正常；
5. 执行UPDATE语句后，检查数据库是否有对应记录，或者输出mysqli的错误信息排查问题。

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