PostgreSQL批量更新删除异常：仅删单条数据问题排查

首先，你的删除逻辑存在根本性的错误——当前的循环方式完全不符合“保留指定unit，删除其余记录”的需求，哪怕你只传入单个unit，也可能因为逻辑漏洞出现意外结果，更别说传入多个unit的场景了。

为什么当前代码会出问题？ #

咱们拆解一下：
当你传入仅含{"unit":"abc"}的Details数组时，循环只执行一次，理论上DELETE FROM position WHERE unit != 'abc' AND user_id = $2应该删掉所有非abc的记录（也就是pqr和xyz）。你说只删了pqr，大概率是个别数据的异常（比如xyz的unit值和传入的不匹配、user_id不一致），但更严重的是：如果传入多个要保留的unit（比如abc和xyz），当前代码会彻底搞砸——第一次循环删掉所有不等于abc的记录（pqr没了），第二次循环删掉所有不等于xyz的记录（abc也被删了），最后只剩xyz，完全违背你“保留多个指定unit”的初衷。

另外，循环执行多次数据库操作也会降低性能，还可能因为异步回调的时序问题引发其他意外。

正确的修复方式 #

不要循环删除，而是一次性收集所有要保留的unit，然后用一条SQL删除不在列表中的记录，既高效又逻辑准确：

// 第一步：提取所有需要保留的unit值
const keepUnits = req.body.Details.map(item => item.unit);

// 第二步：执行单次删除操作，删除user_id下不在keepUnits列表中的记录
db.query(
  'DELETE FROM position WHERE user_id = $1 AND unit NOT IN ($2::text[])',
  [companyuser.rows[0]._id, keepUnits],
  function(err, data) {
    if (err) return callback(new Error(err));
    console.log(`成功删除 ${data.rowCount} 条无关记录`);
    callback(null);
  }
);

如果你更习惯用ANY语法，也可以这么写：

const keepUnits = req.body.Details.map(item => item.unit);

db.query(
  'DELETE FROM position WHERE user_id = $1 AND unit != ANY($2)',
  [companyuser.rows[0]._id, keepUnits],
  function(err, data) {
    if (err) return callback(new Error(err));
    console.log(`成功删除 ${data.rowCount} 条无关记录`);
    callback(null);
  }
);

额外提示 #

如果存在Details数组为空的场景（即用户想删除该user_id下所有记录），上面的代码会直接执行全删；如果这不是你想要的行为，可以加个判断：

const keepUnits = req.body.Details.map(item => item.unit);
if (keepUnits.length === 0) {
  // 这里可以返回错误提示，或者执行全删逻辑，根据你的需求来
  return callback(new Error("不能传入空的Details数组"));
}
// 再执行删除操作...

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