Node.js post请求中如何在if判断前完成MongoDB collection.find查询

问题根源 #

你当前的代码将MongoDB的查询操作当做同步逻辑处理，但无论你用的是原生MongoDB Node.js驱动还是Mongoose ODM，searchLog.find()都是异步操作，不会直接返回查询结果：

• 原生驱动下该方法返回查询游标对象，需要调用toArray()等方法才能拿到实际文档数据
• Mongoose下该方法返回Query对象，需要等待异步执行完成才能获取结果
  你直接对返回的游标/Query对象做非空判断，永远会得到真值，自然无法得到预期的查询结果。

解决方法 #

方案1：async/await写法（推荐） #

这是当前最常用的异步处理写法，语法简洁易维护：

// 给路由处理函数添加async关键字标识为异步函数
app.post('/', async (req, res) => {
    try {
        const bookName = req.body.book;
        // 原生MongoDB驱动写法，调用toArray获取结果数组，await等待异步执行完成
        const queriedResult = await searchLog.find({'name': bookName }).toArray();
        // 如果是Mongoose环境，可以直接写成：
        // const queriedResult = await searchLog.find({'name': bookName });

        if (queriedResult.length > 0) {
            // 存在匹配结果，执行对应逻辑
        } else {
            // 无匹配结果，执行对应逻辑
        }
    } catch (err) {
        // 捕获查询异常，做错误处理
        console.error('查询失败：', err);
        res.status(500).send('服务器内部错误');
    }
});

方案2：回调函数写法 #

适合不支持async/await的低版本Node.js环境：

app.post('/', (req, res) => {
    const bookName = req.body.book;
    searchLog.find({'name': bookName }).toArray((err, queriedResult) => {
        if (err) {
            console.error('查询失败：', err);
            return res.status(500).send('服务器内部错误');
        }
        if (queriedResult.length > 0) {
            // 存在匹配结果，执行对应逻辑
        } else {
            // 无匹配结果，执行对应逻辑
        }
    })
});

优化建议 #

• 如果你仅需要判断是否存在匹配条目，不需要获取完整文档列表，推荐使用countDocuments()方法，查询性能更高：

  const matchCount = await searchLog.countDocuments({'name': bookName });
  if (matchCount > 0) {
    // 存在匹配记录
  }
  

• 如果你只需要获取单条匹配记录，直接使用findOne()方法，返回值为单个文档对象，无匹配时返回null，无需调用toArray：

  const matchDoc = await searchLog.findOne({'name': bookName });
  if (matchDoc) {
    // 存在匹配记录
  }
  

• 注意你原有代码的判断逻辑写反了：if (!queriedResult)的分支注释描述的是结果不为空的逻辑，实际开发中要注意避免逻辑倒置。

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