MongoDB判断ObjectId是否在用户history数组的代码报错排查

错误原因 #

你遇到的$in takes exactly 2 arguments错误，核心是$in操作符的使用完全不符合MongoDB语法：

• $in要求接收数组作为参数，格式应为{ 字段名: { $in: [值1, 值2, ...] } }，但你直接传了单个的req.body._id值；
• 更关键的是，你把$in放在了find的投影参数（第二个参数）中，而$in是查询条件的操作符，必须放在第一个参数的查询逻辑里。

代码其他问题梳理 #

除了上述错误，你的代码还有几个明显问题：

1. condition是find返回的数组，用condition>0判断元素是否存在是错误的，正确写法是condition.length > 0；
2. 定义了async函数alreadyPresentOrNot但没有调用，且参数顺序是(res, req)，和常规的(req, res)顺序相反，容易导致后续调用逻辑混乱；
3. 完全没必要手动判断元素是否存在——MongoDB的$addToSet操作符本身就会仅当元素不存在于数组时才执行添加，直接用它就能实现需求，既简化代码又避免多请求并发时的竞态问题。

修正后的代码 #

最优解法（直接用$addToSet） #

利用$addToSet的去重特性，一步完成需求，这是最简洁高效的写法：

const userId = req.user._id;
const cardId = req.body._id;

try {
  await User.findOneAndUpdate(
    { _id: userId },
    { $addToSet: { history: cardId } },
    { new: true } // 可选：返回更新后的用户文档
  );
  res.sendStatus(201);
} catch (err) {
  console.error(err);
  res.sendStatus(500);
}

手动判断写法（仅作演示，不推荐） #

如果一定要先查询判断再操作，正确的查询逻辑如下：

const userId = req.user._id;
const cardId = req.body._id;

// 查询用户是否已包含该卡片ID
const existingUser = await User.findOne({
  _id: userId,
  history: cardId // 直接匹配数组元素，等价于{ $in: [cardId] }
});

if (existingUser) {
  console.log("Already present !");
  res.sendStatus(401);
} else {
  console.log("Card not found, adding to array now !");
  await User.findOneAndUpdate(
    { _id: userId },
    { $push: { history: cardId } }, // 这里用$push也可以，但$addToSet更安全
    { new: true }
  );
  res.sendStatus(201);
}

关键说明 #

• $addToSet是MongoDB专为数组去重添加设计的操作符，比手动查询判断更高效，还能避免并发请求下的重复添加问题；
• 如果history数组存储的是MongoDB的ObjectId类型，需要确保req.body._id被正确转换为ObjectId（比如用mongoose.Types.ObjectId(req.body._id)），否则会因类型不匹配导致判断错误。

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