PHP类方法中$this->token为null的问题排查及解决

嘿，我来帮你拆解下这个问题——你遇到的$this->token在checkToken()里显示为null，但构造函数里能正常输出的情况，核心问题其实出在条件判断的语法错误上，咱们一步步理清楚：

首先看你代码里的checkToken()方法中的条件判断：

if (current_method() == "POST" || "PUT" || "PATCH") {
    // POST分支逻辑
} elseif (current_method() == "GET" || "DELETE") {
    // GET/DELETE分支逻辑
}

这个写法完全不符合PHP的逻辑运算规则！PHP里||逻辑或的两侧必须是完整的布尔表达式，你现在的写法等价于：

if ( (current_method() == "POST") || true || true ) {

因为非空字符串（比如"PUT"）在布尔上下文里会被解析为true，所以不管当前请求方法是什么，第一个if条件永远为true，代码永远只会走POST/PUT/PATCH的分支，根本不会进入elseif的GET/DELETE分支。

这就解释了为什么你在elseif分支里看到$this->token是null——因为代码从来没走到过那里！而你在构造函数里var_dump能看到正确的token，是因为构造函数里的$this->token = $token;赋值是完全没问题的，只是checkToken()里的分支逻辑完全跑偏了。

解决方法 #

修复条件判断 #

有两种简洁的方式修正这个逻辑：

1. 逐个比较请求方法：

protected function checkToken() {
    $method = current_method();
    if ($method == "POST" || $method == "PUT" || $method == "PATCH") {
        $postToken = isset($_POST['token']) ? $_POST['token'] : '';
        if (CSRF::verifyToken($postToken)) {
            $this->checkPrefix();
        }
    } elseif ($method == "GET" || $method == "DELETE") {
        if (CSRF::verifyToken($this->token)) {
            $this->checkPrefix();
        }
    }
}

2. 用in_array()更简洁地判断：

protected function checkToken() {
    $method = current_method();
    if (in_array($method, ["POST", "PUT", "PATCH"], true)) {
        $postToken = isset($_POST['token']) ? $_POST['token'] : '';
        if (CSRF::verifyToken($postToken)) {
            $this->checkPrefix();
        }
    } elseif (in_array($method, ["GET", "DELETE"], true)) {
        if (CSRF::verifyToken($this->token)) {
            $this->checkPrefix();
        }
    }
}

这里第三个参数true开启了严格模式，确保请求方法的字符串类型完全匹配，避免意外的类型转换问题。

额外优化建议 #

• 去掉@错误抑制符：原来的@$_POST['token']会隐藏$_POST['token']不存在时的警告，建议先判断isset($_POST['token'])再取值，就像上面代码里那样，这样能避免潜在错误被掩盖，方便后续调试。
• 可以在checkToken()开头加一句var_dump($this->token);，确认token确实已经被正确赋值，彻底排除赋值环节的问题。

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