在if条件中使用mysqli预处理语句遇逻辑运算符问题求助

首先要指出你代码里最核心的问题：逻辑运算符用错了，这直接导致邮箱相关的预处理流程没有被正确执行，所以才会出现row_email始终为0的情况。

1. 修复逻辑判断的运算符错误 #

你在绑定参数和执行语句时都用了&&（逻辑与），这个运算符的意思是**只有两个条件同时满足（两个操作都失败）**才会触发错误分支。比如：

• 如果用户名的语句绑定成功了，哪怕邮箱的绑定失败，这个条件也不会触发，邮箱的语句就会处于未正确绑定的状态，执行后自然拿不到正确结果；
• 执行语句时同理，只要其中一个语句执行成功，另一个的失败就会被忽略。

把&&换成||（逻辑或），只要其中一个操作失败就立即退出：

// 绑定参数失败判断
if(!mysqli_stmt_bind_param($stmt_name, "s", $user_name) || !mysqli_stmt_bind_param($stmt_email, "s", $user_email)){
    echo "bind unsuccessfull";
    exit();
}
// 执行失败判断
if( !mysqli_stmt_execute($stmt_name) || !mysqli_stmt_execute($stmt_email)){
    echo "stmt execution failed";
    exit();
}

2. 优化建议：合并查询语句 #

其实你完全不需要创建两个预处理语句，把两个查询合并成一个可以减少数据库交互，代码更简洁，性能也更好：

// 合并查询：一次检查用户名或邮箱是否存在
$query = "SELECT user_name, user_email FROM users WHERE user_name = ? OR user_email = ?";
$stmt = mysqli_prepare($conn, $query);
// 绑定两个参数
mysqli_stmt_bind_param($stmt, "ss", $user_name, $user_email);
// 执行语句
if(!mysqli_stmt_execute($stmt)){
    echo "stmt execution failed";
    exit();
}
// 存储结果集
mysqli_stmt_store_result($stmt);
// 获取匹配行数
$row_count = mysqli_stmt_num_rows($stmt);

// 还可以进一步判断是用户名还是邮箱重复
mysqli_stmt_bind_result($stmt, $existing_name, $existing_email);
mysqli_stmt_fetch($stmt);
if($row_count > 0){
    $error = [];
    if($existing_name === $user_name) $error[] = "用户名已存在";
    if($existing_email === $user_email) $error[] = "邮箱已存在";
    echo implode("，", $error);
    exit();
}

3. 额外注意事项 #

• 处理用户密码时，一定要用password_hash()加密后再存入数据库，绝对不能明文存储；
• 建议给数据库的user_name和user_email字段添加唯一约束，这样即使代码层面有疏漏，数据库也会阻止重复数据插入，增加一层安全保障。

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