PHP面向对象登录系统：IF ELSE语句异常问题

我太懂这种卡壳的感觉了——明明逻辑上应该触发“密码不正确”的提示，结果却返回个NULL，完全摸不着头脑对吧？咱们从代码逻辑和常见坑点入手，一步步把这个问题解决掉。

首先，先梳理下你遇到的核心问题：当邮箱匹配到数据库中的记录，但密码错误时，你的IF ELSE分支没有正确设置$paError，导致最终返回NULL。大概率是以下几个常见问题中的一个：

1. 密码比对逻辑错误（最常见！） #

如果你的数据库里存的是用password_hash()加密后的密码，却用普通的==或===去和用户输入的明文密码比对，那永远都不会匹配，而且如果你的错误变量没初始化，就会返回NULL。

正确的做法是用PHP官方的password_verify()函数，它专门用来比对明文密码和password_hash()生成的哈希值：

// 错误示例：明文比对
if($userPassword == $this->dbPassword) { ... }

// 正确示例：用password_verify
if(password_verify($userPassword, $this->dbPassword)) {
    // 登录成功逻辑
} else {
    $this->paError = "密码不正确";
    return false;
}

2. 错误变量未初始化 #

你在类里声明了public $paError;但没给初始值，当这个变量没被赋值时，它的默认值就是NULL。所以不管有没有触发错误分支，只要没给它赋值，返回的就是NULL。

修正方法：初始化错误变量为空字符串：

class Login extends Connection {
    // ...其他属性
    public $emError = "";
    public $paError = ""; // 初始化空字符串，避免返回NULL
    // ...
}

3. 数据库查询未正确获取密码 #

可能你的查询语句有问题，比如字段名拼写错误（比如把password写成pass），或者查询后没正确取出结果，导致$this->dbPassword是NULL，比对时进入else分支但没设置错误提示？

排查建议：

• 检查SQL语句的字段名、表名是否正确
• 查询后一定要判断是否有结果返回，再去取密码：

$result = $this->query->get_result();
if($result->num_rows === 1) {
    $row = $result->fetch_assoc();
    $this->dbPassword = $row['password']; // 确认这里的字段名和数据库一致
    // 然后进行密码比对
} else {
    $this->emError = "邮箱不存在";
}

4. 数据库连接或查询出错 #

如果父类Connection的数据库连接有问题，或者查询执行失败，那后续的逻辑根本走不到密码比对的分支，自然也不会设置$paError。

排查建议：

• 在查询前后添加错误检测，比如：

$this->query = $this->conn->prepare("SELECT password FROM users WHERE email = ?");
if(!$this->query) {
    die("查询准备失败: " . $this->conn->error); // 临时调试用，上线后换成友好提示
}

给你整理的完整修正代码片段 #

<?php
include('conn.php');

class Login extends Connection {
    private $dbPassword;
    private $stmt;
    public $emError = "";
    public $paError = "";

    // 确保调用父类构造函数初始化数据库连接
    public function __construct() {
        parent::__construct();
    }

    public function checkLogin($userEmail, $userPassword) {
        // 预处理查询，防止SQL注入
        $this->stmt = $this->conn->prepare("SELECT password FROM users WHERE email = ? LIMIT 1");
        if(!$this->stmt) {
            $this->emError = "数据库异常，请稍后再试";
            return false;
        }

        $this->stmt->bind_param("s", $userEmail);
        $this->stmt->execute();
        $result = $this->stmt->get_result();

        if($result->num_rows === 1) {
            $userRow = $result->fetch_assoc();
            // 正确比对密码
            if(password_verify($userPassword, $userRow['password'])) {
                // 登录成功，这里可以设置Session等操作
                return true;
            } else {
                $this->paError = "密码不正确";
                return false;
            }
        } else {
            $this->emError = "该邮箱未注册";
            return false;
        }

        $this->stmt->close();
    }
}

调用示例 #

$loginObj = new Login();
if($_SERVER['REQUEST_METHOD'] === 'POST') {
    $inputEmail = trim($_POST['email']);
    $inputPassword = trim($_POST['password']);

    if($loginObj->checkLogin($inputEmail, $inputPassword)) {
        echo "登录成功！";
        // header("Location: dashboard.php");
        // exit;
    } else {
        // 显示错误提示
        if(!empty($loginObj->emError)) {
            echo "<p>{$loginObj->emError}</p>";
        }
        if(!empty($loginObj->paError)) {
            echo "<p>{$loginObj->paError}</p>";
        }
    }
}

最后再提个安全小建议：不要区分“邮箱不存在”和“密码错误”的提示（比如统一提示“邮箱或密码错误”），这样能防止恶意用户枚举你的邮箱列表，提升系统安全性。

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