[php函数在我期望为假时返回true

Question

我很困惑为什么我的代码在我期望true时返回false，反之，这是我的代码：

public function CheckMac($mac){
    $database = new Database();
    $db = $database->connectDatabase();
    $checkedmac = $db->prepare("SELECT * FROM `displays` WHERE `displayMac` = '$mac'");
    $checkedmac->execute();
    $count = (int)$checkedmac->fetchColumn();
    if ($count > 0) {
        return true;
    } else {
        return false;
    }
}

我拥有正确的查询，当我回显$mac并将其放入查询中时，phpmyadmin给我返回了预期的行，因为它存在于数据库中，但是当我运行此代码时，我得到了false返回。

我在哪里出错了？

Answer 1

[一个不错的答案，但被低估了，If your code is doing something unexpected, there's a good chance you're making an assumption somewhere。

您尚未学习称为debugging的编程艺术的基石概念。这意味着您应该验证您所做的每个假设。

[您在这里假设，当查询返回一行时，(int)$checkedmac->fetchColumn();语句返回一个正数。因此，您必须进行验证。

我可以做出假设，您表中的第一列不包含正数，而是一个字符串。带数字的字符串将返回0。它将解释为什么在找到记录时得到0。但是您也必须进行验证。

而不是*选择一个常数，例如1，它将正常工作。

public function CheckMac($db, $mac){
    $stmt = $db->prepare("SELECT 1 FROM `displays` WHERE `displayMac` = ?");
    $stmt->execute([$mac]);
    return $stmt->fetchColumn();
}

一些注意事项：

• 您应该始终仅连接once。预先在一个地方创建$ db变量，然后在任何地方使用它
• 您使用的是cargo cult prepared statement，它不受任何保护。它应该始终是我的示例中显示的方式。
• [if ($count > 0) return true是重言式 $count > 0已经返回true或false，因此您可以返回结果的精确度，而不会出现多余的情况
• 此外，在PHP中1和true一样好。因此，最后您可以立即返回fetchColumn（）的结果。如果找到该行，则它将为1等于true，否则为false。