PHP登录逻辑问题：空值判断分支阻断SQL执行请求协助

Hey Hannah, let's break down why your login flow isn't making it past the empty check into the SQL query branch, and fix those issues step by step.

1. 致命的SQL语法错误 #

这是导致代码无法正常进入查询分支的核心问题：你的SELECT语句中，$email是字符串类型，但你没有给它包裹单引号，直接会触发SQL语法错误，让mysqli_query执行失败，后续逻辑自然无法推进。

原错误代码：

$sql = "SELECT * FROM users WHERE email = $email";

修正后：

$sql = "SELECT * FROM users WHERE email = '$email'";

（虽然你已经用mysqli_real_escape_string做了转义，但SQL中字符串必须用单引号包裹才能被正确识别）

2. 缺失查询失败的错误检测 #

你现在的代码没有检查mysqli_query的执行结果，就算SQL报错，你也看不到具体问题。建议添加错误判断，方便调试：

$row = mysqli_query($conn, $sql);
if (!$row) {
    // 开发阶段输出错误信息，上线后可替换为友好提示
    die("SQL查询失败: " . mysqli_error($conn));
}

这样能直接拿到SQL执行的具体错误，快速定位问题。

3. 空值判断的跳转参数优化 #

你的跳转参数empty field包含空格，可能导致URL解析异常，建议规范参数格式：

header('Location: project.php?error=empty_fields');

这种格式更标准，后续前端也更容易获取错误信息。

补充：登录功能的关键遗漏——密码验证 #

你原来的代码只检查了邮箱是否存在，完全没有验证密码是否匹配，这是登录功能的核心缺失。修正后的完整查询分支代码如下：

//if user has an account
else{
    $sql = "SELECT * FROM users WHERE email = '$email'";
    $row = mysqli_query($conn, $sql);
    
    // 新增查询失败检测
    if (!$row) {
        die("SQL查询错误: " . mysqli_error($conn));
    }
    
    $result = mysqli_num_rows($row);
    if ($result < 1) {
        header('Location: project.php?error=invalid_credentials');
        exit();
    } else {
        // 获取用户数据并验证密码
        $user_data = mysqli_fetch_assoc($row);
        if (password_verify($pwd, $user_data['pwd'])) {
            // 密码匹配，跳转到后台
            header('Location: admin.php');
            exit();
        } else {
            // 密码错误
            header('Location: project.php?error=invalid_credentials');
            exit();
        }
    }
}

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