MongoDB报错：Cannot read property 'result' of undefined 插入前置查询异常

咱们直接定位问题根源：你在insertOne的回调函数里没先处理错误参数err，当插入操作出现异常时（比如数据库连接问题、权限不足、字段约束冲突等），insertResult会变成undefined，这时候你直接访问insertResult.result自然就会抛出这个类型错误。另外，你用find().toArray()来检查数据是否存在也有点冗余，用findOne()更高效。

修复后的完整代码示例 #

collection.findOne({sno: req.body.sno}, (err, doc) => {
  // 先处理查询阶段的错误
  if (err) {
    console.error("查询sno时出错:", err);
    callback(false);
    return;
  }

  if (doc) {
    // 已存在该sno，直接返回false
    callback(false);
  } else {
    console.log("未找到匹配的sno，执行插入");
    collection.insertOne(
      {sno: req.body.sno, password: req.body.password},
      (insertErr, insertResult) => {
        // 优先处理插入阶段的错误
        if (insertErr) {
          console.error("插入数据时出错:", insertErr);
          callback(false);
          return;
        }
        // 新版MongoDB驱动推荐用这两个字段判断插入成功
        if (insertResult.acknowledged && insertResult.insertedId) {
          callback(true);
        } else {
          callback(false);
        }
      }
    );
  }
});

关键修复点说明 #

1. 错误优先处理：不管是查询还是插入的回调，都先判断err是否存在，出错时直接终止后续逻辑并通知调用方，从根源避免访问undefined变量。
2. 用findOne替代find().toArray()：findOne找到第一条匹配文档就会停止查询，性能比拉取所有匹配文档更高，代码也更简洁——直接判断返回的doc是否存在即可。
3. 更新插入成功的判断逻辑：新版MongoDB Node.js驱动中，insertResult的结构已经调整，推荐用acknowledged（确认操作被服务器接收）和insertedId（新插入文档的ID）来判断插入是否成功，替代旧版的result.ok。

额外优化：避免并发竞态问题 #

如果你的业务场景存在并发请求（比如两个请求同时插入同一个sno），上面的“先查后插”逻辑可能会出现重复插入的问题。这时候可以用原子操作findOneAndUpdate，把检查和插入合并成一个原子步骤：

collection.findOneAndUpdate(
  {sno: req.body.sno},
  {$setOnInsert: {password: req.body.password}},
  {upsert: true, returnDocument: 'after'},
  (err, result) => {
    if (err) {
      console.error("原子操作出错:", err);
      callback(false);
      return;
    }
    // 通过upserted字段判断是否是新插入的文档
    callback(result.lastErrorObject.upserted !== undefined);
  }
);

这个逻辑是：如果找不到匹配sno的文档，就插入一条新的；如果已经存在，就不做任何修改，完美避免并发场景下的重复插入问题。

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