Node.js：indexOf返回-1但元素在数组中的问题排查与优化

你遇到的这个问题，核心在于**indexOf方法的比较逻辑**。indexOf使用严格相等（===）来匹配元素，而你传入的player对象和foundGame.players数组里的对象，虽然属性（比如id、username）可能完全一致，但它们是不同的对象引用——一个是你新创建的Player实例，另一个是从数据库查询出来的文档对象，内存地址不一样，所以indexOf永远找不到匹配项，返回-1。

举个直观的例子：

const obj1 = { id: 1, name: "Alice" };
const obj2 = { id: 1, name: "Alice" };
console.log([obj1].indexOf(obj2)); // 输出-1，因为是不同引用

另外，你的代码里还有个冗余操作：Player.create(req.user)其实没必要——你的Game模型的players数组已经存储了id和username，单独创建Player文档反而增加了不必要的数据库开销（除非你有其他业务需求）。

方案1：修改内存中的存在性判断逻辑 #

把indexOf替换成Array.some()或Array.find()，通过对象的唯一标识（比如id）来判断用户是否已在数组中：

更新你的isInGroup函数：

var isInGroup = function(player, arr){
  // 转成字符串比较ObjectId，避免类型不匹配
  const userExists = arr.some(item => item.id.toString() === player.id.toString());
  if(!userExists){
    arr.push(player);
    console.log("added user");
  } else {
    console.log("Already in group");
  }
};

方案2：使用MongoDB原子操作（更优） #

直接利用MongoDB的$addToSet操作符，它会自动将元素添加到数组中，仅当元素不存在时才执行添加。这个方案在数据库层面完成去重和更新，比先查询再修改保存更高效，还能避免并发场景下的重复添加问题。

修改你的post路由代码：

router.post("/", middleware.isLoggedIn, function(req, res){
  Game.findByIdAndUpdate(
    req.body.game,
    // $addToSet自动去重，仅添加不存在的元素
    { 
      $addToSet: { 
        players: {
          id: req.user._id,
          username: req.user.username
        }
      }
    },
    { new: true }, // 返回更新后的文档
    function(err, updatedGame){
      if(err){
        req.flash("error", "Something went wrong.");
        res.redirect("back");
      } else {
        req.flash("success", "Successfully joined the game!");
        res.redirect("back");
      }
    }
  );
});

如果没有额外业务需求，你甚至可以删除Player模型，因为Game的players数组已经存储了必要的用户信息。

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