主键USERID重复时更新失败，重复插入生成重复行问题求助

嘿，我来帮你搞定这个问题！你的核心问题出在两个地方：条件判断的低级错误和非原子操作导致的竞态/主键冲突，下面一步步拆解解决：

1. 先揪出最明显的语法错误 #

你代码里的这个判断：

if ((int)$row['USERID'] = (int)$USERID){

这里用了赋值运算符=，而不是比较运算符==（松散比较）或者===（严格比较）！这会把$USERID的值直接赋给$row['USERID']，导致这个条件永远为真，但逻辑完全不对，这也是你更新逻辑没生效的直接原因之一。

修正的话应该改成：

if ((int)$row['USERID'] === (int)$USERID){

但即使改了这个，你的方案还是有隐患，继续往下看。

2. 根治问题：用原子操作代替先查再更 #

你现在“先查询是否存在→再决定插入/更新”的逻辑，在高并发场景下会出现竞态条件：比如两个请求同时查询到某个USERID不存在，然后都执行插入，就会触发主键重复报错（也就是你看到的“fail updating data with same primary key”）。

MySQL提供了原生的原子语法INSERT ... ON DUPLICATE KEY UPDATE，能完美解决这个问题——数据库层面保证要么插入新数据，要么更新已存在的主键数据，全程原子性，不会出现冲突。

3. 安全又高效的正确实现 #

另外还要注意：你直接把变量拼进SQL的写法存在严重的SQL注入风险，必须用预处理语句来避免。下面是用mysqli预处理实现的完整示例：

// 假设$conn是你的mysqli数据库连接对象（已经成功连接）
$sql = "INSERT INTO DATA (USERID, SANDI, Photo, Name) 
        VALUES (?, ?, ?, ?)
        ON DUPLICATE KEY UPDATE 
            SANDI = VALUES(SANDI),
            Photo = VALUES(Photo),
            Name = VALUES(Name)";

// 预处理SQL语句
$stmt = $conn->prepare($sql);
// 绑定参数：第一个参数是类型占位符（i=int，s=string），后面是对应变量
$stmt->bind_param("isss", $USERID, $SANDI, $Photo, $Name);
// 执行语句
$stmt->execute();

// 检查执行结果
if ($stmt->affected_rows > 0) {
    echo "数据插入或更新成功";
} else {
    echo "操作失败：" . $conn->error;
}

// 关闭语句和连接（按需）
$stmt->close();
$conn->close();

为什么这个方案更好？ #

• 原子性：数据库直接处理插入/更新逻辑，完全避免并发场景下的主键冲突问题
• 简洁高效：省去了先查询再判断的冗余步骤，减少数据库交互次数
• 安全：预处理语句彻底杜绝SQL注入风险，原写法的安全漏洞非常致命

补充：原代码的其他潜在问题 #

你贴的代码片段里只定义了$sql的查询语句，但没看到执行查询的代码（比如$result = $conn->query($sql);），如果这部分缺失，$result会是null，调用$result->num_rows会直接报错，这也是需要检查的点。

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