mongoose调用findOne时报Query was already executed错误如何解决

报错原因 #

• 混合使用回调和异步查询触发重复执行：Mongoose的findOne方法如果传入回调函数会立刻触发查询执行，你将返回的查询实例return到外部后，外层代码大概率又对该实例做了await/.exec()调用，导致同一个查询被执行两次，触发Query was already executed报错。
• 模型重复注册：model('guilds', GuildSchema)是Mongoose的模型注册操作，只需要全局执行一次，你把它放在查询函数内部，每次调用函数都会重复注册模型，也会引发未知异常。

修复方案 #

第一步：抽离模型注册逻辑 #

不要在查询函数内定义模型，单独在Schema文件里导出注册好的模型即可，修改你的Schema文件：

const { model, Schema } = require('mongoose')

const GuildSchema = new Schema(
    {},
    {
        strict: false,
        versionKey: false,
    }
)
// 直接在这里注册模型并导出
const GuildModel = model('guilds', GuildSchema)

module.exports = { GuildSchema, GuildModel }

第二步：改写通用查询函数 #

统一用async/await语法，不要混写回调，既可以满足通用调用需求，也符合异步代码规范：

// 先导入提前注册好的模型，路径替换为你自己的文件路径
const { GuildModel } = require('./schema文件所在路径')

async function lookUpDB(guildName, lookFor) {
    try {
        const query = { name: guildName }
        // 如果需要指定返回字段可以把lookFor传进findOne的第二个参数，比如findOne(query, lookFor)
        const result = await GuildModel.findOne(query)
        console.log(result)
        return result
    } catch (err) {
        console.error('查询出错：', err)
        throw err // 外层调用处可以捕获错误做自定义处理
    }
}

第三步：调用方式 #

外部需要查询的地方直接await这个函数即可：

// 示例调用
const guildData = await lookUpDB('Stream Squire')
console.log('查询到的公会数据：', guildData)

额外优化建议 #

• 如果需要实现更通用的多集合查询函数，可以把集合名、查询条件、返回字段都作为参数传入，不用限定死只查guilds集合。
• 你现有的数据库连接逻辑是全局初始化的，不需要在查询函数里做额外处理，只要保证服务启动时先执行连接代码即可。

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