PHP PDO 中 OR 与 AND 混用的SQL优化技巧

本文将围绕一个常见的 PHP PDO 和 MySQL 查询问题展开，该问题涉及 OR 和 AND 运算符的混合使用，导致查询结果不符合预期。问题的核心在于理解 SQL 运算符的优先级，以及如何通过调整查询语句来获得正确的结果。此外，还会讨论用户身份验证流程中的安全实践，以避免泄露敏感信息。

问题分析

在提供的代码示例中，用户尝试使用用户名或邮箱地址以及密码进行登录验证。原始的 SQL 查询语句如下：

SELECT * FROM db_cms_users WHERE username = ? OR email = ?  AND password = ?

这段 SQL 语句的本意是查询 username 或 email 与输入匹配，并且 password 也匹配的用户。然而，由于 AND 运算符的优先级高于 OR 运算符，实际的执行顺序是先执行 email = ? AND password = ?，然后再将结果与 username = ? 进行 OR 运算。这导致了查询逻辑的错误，使得只有当用户名匹配时，查询才能正确返回结果。

解决方案

要解决这个问题，可以使用括号来明确指定运算符的优先级。将 username = ? OR email = ? 用括号括起来，确保这部分表达式先被计算。修改后的 SQL 查询语句如下：

SELECT * FROM db_cms_users WHERE (username = ? OR email = ?)  AND password = ?

此外，更佳的解决方案是简化查询逻辑。在验证密码之前，先通过用户名或邮箱查询到用户的信息，然后使用 password_verify() 函数来验证密码的正确性。这样可以避免在 WHERE 子句中使用密码，提高安全性。

代码优化

以下是优化后的 loginUser() 函数代码示例：

protected function loginUser($userID, $password) {
  $sql = "SELECT username, id, password FROM db_cms_users WHERE username = ? OR email = ?";
  $stmt = $this->connect()->prepare($sql);

  if(!$stmt->execute([$userID, $userID])) {
    $stmt = null;
    header("location: index.php?error=failstmt");
    exit();
  }

  if($stmt->rowCount() == 0) {
    $stmt = null;
    header("location: login.php?error=loginerror");
    exit();
  }

  $user = $stmt->fetchAll();
  $checkPwd = password_verify($password, $user[0]['password']);

  if($checkPwd == false) {
    header("location: index.php?error=wrongpwd");
    exit();
  }
  elseif($checkPwd == true) {
    session_start();
    $_SESSION['username'] = $user[0]['username'];
    $_SESSION['uid'] = $user[0]['id'];
    return true;
  }
}

代码解释：

1. 简化查询： 只查询 username、id 和 password 字段，避免查询不必要的字段。
2. 密码验证： 使用 password_verify() 函数验证密码，确保密码的安全性。
3. 会话管理： 验证成功后，将 username 和 id 存储到会话中。

安全注意事项

在用户身份验证流程中，安全性至关重要。以下是一些需要注意的安全事项：

• 避免泄露信息： 当用户输入的凭据无效时，不要明确指出是用户名或密码错误。统一返回 "无效的凭据" 消息，避免给恶意攻击者提供线索。
• 密码哈希： 始终使用 password_hash() 函数对密码进行哈希处理，并将哈希后的密码存储在数据库中。
• 防止 SQL 注入： 使用预处理语句（Prepared Statements）来防止 SQL 注入攻击。
• 会话安全： 使用安全的会话管理机制，例如设置 session.cookie_secure 和 session.cookie_httponly 选项。

总结

在使用 PHP PDO 操作 MySQL 数据库时，理解 SQL 运算符的优先级非常重要。通过使用括号来明确指定运算符的优先级，可以避免查询逻辑错误。此外，在用户身份验证流程中，安全性至关重要。通过采取适当的安全措施，可以保护用户的账户安全，防止恶意攻击。