Mongoose findOneAndUpdate重复添加ObjectID至数组问题求助

看起来你遇到的问题主要有两个潜在原因，咱们一个个拆解：

1. $push 操作符允许重复元素 #

默认情况下MongoDB的$push不会校验数组中是否已存在目标元素，只要调用就会直接往数组里追加，这正是同一个guest ID被重复插入的核心原因。

解决方案是改用$addToSet操作符——它会自动检查元素是否已存在于数组中，只有不存在时才会执行添加操作：

await Table.findOneAndUpdate(
  { name: req.body.tablename },
  { $addToSet: { guests: req.body.guestId } }, // 将$push替换为$addToSet
  { safe: true, upsert: true }
);

2. 混用async/await和回调函数 #

你的代码同时使用了await和回调函数，这在Mongoose里会引发不可预期的行为：当你给findOneAndUpdate传递回调时，它不会返回Promise，此时await完全无效，但回调函数仍会执行。虽然你确认函数没被调用两次，但这种混用可能导致逻辑混乱，甚至触发额外的更新操作。

修复方式是统一使用async/await风格，去掉冗余的回调：

exports.updateTableGuests = async (req, res) => {
  console.log(req.body.guestId);
  try {
    await Table.findOneAndUpdate(
      { name: req.body.tablename },
      { $addToSet: { guests: req.body.guestId } },
      { safe: true, upsert: true }
    );
    // 在这里编写原回调中的业务逻辑
    res.send('back');
  } catch (err) {
    console.log(err);
    // 建议给前端返回错误状态码，提升调试体验
    res.status(500).send('Failed to update table guests');
  }
};

额外建议：你可以在前端拖拽完成后，同步更新本地状态（比如将已分配的Guest从可拖拽列表中移除），既提升用户体验，也能从源头避免用户重复拖拽同一Guest到同一餐桌的操作。

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