返回总是返回false

Question

问题在于此函数总是返回0。为什么？

public function valid_token ()
{
    if (!isset($_SESSION['token']) || $this->token != $_SESSION['token'])
    {
        $this->errors[] = "Formulario incorrecto";
    }

return count($this->errors)? 0 : 1;
}

Answer 1

编辑：

忽略我以前的答案。 愚蠢的虚假价值观。 您总是得到0的原因仅仅是因为...您在数组中有一个值。 正如@Orangepill在评论中指出的，转储$this->token和$_SESSION['token]以查看发生了什么。

旧：

count()返回数组中元素的数量。 现在，您只是在运行count() 。 您需要将其与整数值进行比较，即：

count($this->errors)>0 ? 0 : 1;

Answer 2

令牌是一个mt_rand函数，这就是为什么我认为问题出在return语句上。 同样是因为即使我调用另一个名为valid_data（）的函数时也会出错，该函数具有相同的return语句。 $ token = $ _SESSION ['token'] = md5（uniqid（mt_rand（），true））;

Answer 3

在valid_token（）中使用它之前，您是否在类中初始化了$ this-> errors？ 如果没有，您可能正在计算未设置的内容，这将返回false。 确保像这样初始化成员：

受保护的$ error = array（）;

要么

公共$错误= array（）;

另外，您使用了三元表达式，但没有将条件包装在括号中。 因此，该语句可能没有正确评估。 尝试这个：

 $isValidForm = (count($this->errors) > 0) ? false : true;
 return $isValidForm;

我同意您应该“转储”变量，以查看返回的内容。 我可以通过将其字符串值记录到Apache error.log中来做到这一点：

error_log（“ this token：”。print_r（$ this-> token，true））;

error_log（“会话令牌：”。print_r（$ _ SESSION ['token']，true））;

error_log（“此错误：”。print_r（$ this-> error，true））;

在GNU / Linux或OSX中，您可以像这样从控制台拖尾日志：

尾巴-f /var/log/apache2/error.log

这样，您不必中断程序的流程即可对其进行调试。

最后，这只是一个建议-valid_token（）不是一个好的方法名称，它听起来像一个变量名称。 validate_token（）或validateToken（）是更好的名称，因为它们使用动词来表示它们是动作。