使用PHP PDO检查数据库重复记录失效问题求助

原代码的核心问题 #

1. 请求参数不统一：查询时取$_GET['useremail']，但插入用$_POST['useremail']——如果表单是POST提交，GET里根本没有这个值，导致查询永远查不到已存在的记录，直接执行插入。
2. 参数类型绑定错误：邮箱是字符串，却用了PDO::PARAM_INT（整数类型），导致数据库查询时类型不匹配，无法匹配到已存在的邮箱记录。
3. 前端脚本语法错误：输出的SweetAlert代码未闭合</script>标签，会导致前端脚本执行失败，用户看不到重复提示。
4. 无数据库层面约束：仅靠代码检查无法应对并发场景（比如用户快速连续点击提交），可能出现重复插入。

修复后的完整代码 #

<?php
// 统一使用POST参数（假设表单为POST提交）
$userEmail = $_POST['useremail'] ?? '';

// 1. 检查邮箱是否已存在
$stmt = $db->prepare('SELECT useremail FROM newsletter WHERE useremail = :useremail');
// 邮箱为字符串类型，绑定正确参数类型
$stmt->bindParam(':useremail', $userEmail, PDO::PARAM_STR);
$stmt->execute();
$row = $stmt->fetch(PDO::FETCH_ASSOC);

if ($row) {
    // 修复JS标签闭合问题
    echo "<script type='text/javascript'>Swal.fire('Info', 'This email is already registered.','info');</script>";
} else {
    // 执行插入操作
    $kaydet = $db->prepare("INSERT INTO newsletter SET useremail = :useremail");
    $insert = $kaydet->execute(['useremail' => $userEmail]);

    if ($insert) {
        header("Location:../index.php?status=success");
        exit;
    } else {
        header("Location:../index.php?status=error");
        exit;
    }
}
?>

额外的安全/可靠性优化 #

• 给数据库字段加唯一约束：在newsletter表的useremail字段上添加唯一索引，从数据库层面强制阻止重复插入，即使代码层面出现疏漏或并发情况：

  ALTER TABLE newsletter ADD UNIQUE INDEX idx_unique_useremail (useremail);
  

• 添加邮箱有效性检查：在开头添加非空和格式验证，避免插入无效数据：

  if (empty($userEmail) || !filter_var($userEmail, FILTER_VALIDATE_EMAIL)) {
      header("Location:../index.php?status=invalid_email");
      exit;
  }
  

内容的提问来源于stack exchange，提问作者Fatih Şahin