高效检查数据库用户是否存在的SQL查询逻辑优化问题

修复方案 #

你的问题由两个逻辑缺陷共同导致：

1. 原始SQL未处理运算符优先级：SQL中AND优先级高于OR，原查询实际执行逻辑为 (phone=:phone) OR (email=:email AND status=:status)，不符合「手机号或邮箱匹配、同时状态符合要求」的预期
2. 未做空值过滤：当数据库中对应字段为空，或传入的查询参数为空时，会触发空=空的误匹配逻辑，返回不符合要求的用户

直接修复代码

$user_exists_query = "SELECT COUNT(*) FROM users 
WHERE status = :status 
AND (
    (phone = :phone AND phone IS NOT NULL AND phone != '')
    OR 
    (email = :email AND email IS NOT NULL AND email != '')
)";
$check_user_exists = $conn->prepare($user_exists_query);
$check_user_exists->bindValue(':phone', $phone, PDO::PARAM_STR);
$check_user_exists->bindValue(':email', $email, PDO::PARAM_STR);
$check_user_exists->bindValue(':status', $status, PDO::PARAM_STR);
$check_user_exists->execute();

修改说明

• 外层统一加status = :status过滤，确保所有命中用户均为指定状态
• 用括号包裹手机号、邮箱两个匹配逻辑，修正运算符优先级错误
• 每个匹配项新增对应字段的非空、非空字符串判断，只有当数据库中该字段不为空、且和传入参数完全相等时才会命中，彻底解决空值误匹配问题

可选优化（支持单字段查询）

如果业务允许只传手机号/邮箱其中一个参数做查询，可动态拼接SQL避免传入空参数时的无效匹配：

$conditions = [];
$params = [':status' => $status];
// 仅当传入手机号不为空时加手机号匹配条件
if (!empty($phone)) {
    $conditions[] = "(phone = :phone AND phone IS NOT NULL AND phone != '')";
    $params[':phone'] = $phone;
}
// 仅当传入邮箱不为空时加邮箱匹配条件
if (!empty($email)) {
    $conditions[] = "(email = :email AND email IS NOT NULL AND email != '')";
    $params[':email'] = $email;
}
// 拼接最终SQL
$user_exists_query = "SELECT COUNT(*) FROM users WHERE status = :status ";
if (!empty($conditions)) {
    $user_exists_query .= "AND (" . implode(" OR ", $conditions) . ")";
}
$check_user_exists = $conn->prepare($user_exists_query);
$check_user_exists->execute($params);

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