MySQL仅可插入新记录无法更新已有记录，求排查思路

这种「插入正常但更新失败」的问题我之前做日志系统时也踩过坑，给你整理几个实用的排查方向，结合你的代码片段来分析：

1. 先确认「存在则更新」的核心逻辑是否正确 #

MySQL里实现“不存在插入、存在更新”最可靠的方式是INSERT ... ON DUPLICATE KEY UPDATE语法，但它有个前提：你的表必须有主键或唯一索引来判断记录是否重复。

• 看你的代码里用到了$date = date('m-d-Y');和用户标识（cusersup/cuseropm），大概率应该把date+用户字段设为组合唯一键，比如UNIQUE KEY idx_date_user (date, u_manager)，这样MySQL才能识别到重复记录并触发更新。
• 如果你的代码是先查询记录是否存在，再分别执行INSERT/UPDATE，那很可能是查询的WHERE条件写错了（比如漏了用户标识，导致找不到对应记录），或者出现了并发竞态（两个请求同时查，都发现不存在，然后都执行插入）。

2. 验证「读取当前值」的逻辑是否生效 #

你怀疑程序读不到当前值，可以直接加调试代码验证：

• 在查询现有记录的代码后面，打印查询结果，比如：

  $existing_log = $database->get_row("SELECT * FROM your_table WHERE date = '$date' AND u_manager = '$cusersup'");
  var_dump($existing_log); // 看看能不能拿到预期的记录
  

  如果打印结果是NULL，要么是这条记录确实不存在，要么是WHERE条件里的字段值不对（比如cusersup没拿到正确的用户ID，或者date的格式和数据库里的不一致）。

3. 检查变量处理是否有问题 #

你的代码里用了safety_filter过滤数值字段，还有get_the_current_user获取用户信息，这些环节很容易出问题：

• 确认safety_filter会不会把合法数值改成空或0？比如原本要更新的l_sales是10，过滤后变成了空，那更新后字段值就不对了，可以用var_dump($l_sales, $l_enroll, $l_offers)看看过滤后的值是否符合预期。
• 检查get_the_current_user返回的用户标识是否正确，比如cusersup是不是当前登录用户的ID，这直接影响WHERE条件能否匹配到现有记录。

4. 开启错误捕获，看MySQL到底报了什么错 #

很多时候更新失败是因为SQL语句有语法错误、字段不存在或者权限问题，但你没捕获到错误信息。可以在执行更新语句后加错误检查：

// 假设你的$database类有错误处理方法
$database->query($update_sql);
if ($database->has_error()) {
    echo "MySQL错误：" . $database->get_error_message();
    exit;
}

直接看MySQL的报错信息，比瞎猜高效多了。

5. 考虑并发场景的影响 #

如果多个请求同时操作同一条记录，可能会出现读不到最新值的情况（比如A请求读了记录，B请求已经更新了，A还拿着旧值去更新）。这种情况可以：

• 改用INSERT ... ON DUPLICATE KEY UPDATE，这是原子操作，MySQL会自动处理重复情况，避免竞态问题。
• 或者用SELECT ... FOR UPDATE锁定记录后再更新，确保同一时间只有一个请求能修改这条记录。

给你的代码优化建议 #

从你的代码片段来看，不如直接用原子的INSERT ... ON DUPLICATE KEY UPDATE来替代先查后改的逻辑，示例代码如下（注意用预处理语句防SQL注入，不要直接拼接变量）：

function add_log($input_output, $l_sales, $l_enroll, $l_offers) {
    global $database;
    $date = date('m-d-Y');
    $l_sales = safety_filter($l_sales);
    $l_enroll = safety_filter($l_enroll);
    $l_offers = safety_filter($l_offers);
    $cusersup = get_the_current_user('u_manager');
    $cuseropm = get_the_current_user('u_opm'); // 假设你要用到这个字段

    // 预处理语句示例（避免SQL注入）
    $sql = "INSERT INTO your_log_table (date, u_manager, u_opm, sales, enroll, offers)
            VALUES (?, ?, ?, ?, ?, ?)
            ON DUPLICATE KEY UPDATE
            sales = VALUES(sales),
            enroll = VALUES(enroll),
            offers = VALUES(offers)";
    
    $stmt = $database->prepare($sql);
    $stmt->bind_param("sssiii", $date, $cusersup, $cuseropm, $l_sales, $l_enroll, $l_offers);
    $stmt->execute();
    $stmt->close();
}

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