Node.js+Express+Mongoose开发REST服务传入错误ID时崩溃如何解决？

• 混用 await 与 Mongoose 查询回调：回调函数内抛出的错误不属于 async 函数的上下文，无法被外层的 catch 捕获，最终成为未处理异常导致进程崩溃
• 未提前校验 ID 格式：MongoDB 的 ObjectId 为固定24位十六进制字符串，传入非法格式的 ID 会触发 Mongoose 内部错误
• 非法的 ID 截断逻辑：直接将任意 ID 截断为24位的处理方式，得到的字符串大概率不是合法的 ObjectId，依然会触发查询错误

第一步：统一使用 async/await 处理查询，移除回调 #

Mongoose 支持直接 await 查询语句，错误会正常抛出，可被外层捕获，避免未处理异常。

第二步：提前校验 ObjectId 合法性 #

使用 Mongoose 内置的 mongoose.Types.ObjectId.isValid() 方法校验传入的 ID 格式，格式不合法时直接返回400错误，无需进入查询流程。

第三步：调整服务层代码示例 #

const mongoose = require('mongoose');
// 其他原有导入保留

async function get(req) {
    const findParams = {};
    if (req.query.id) {
        const inputId = req.query.id.trim();
        // 校验ID格式合法性
        if (!mongoose.Types.ObjectId.isValid(inputId)) {
            throw { status: 400, error: '传入的ID格式不合法' };
        }
        findParams['_id'] = inputId;
    }

    try {
        // 移除回调，纯用await执行查询
        const post = await Posts.find(findParams).sort({ createdDate: -1 });
        
        if (req.query.id) {
            if (!post.length) {
                throw { status: 404, error: 'Post not found' };
            }
            return { ...post[0]._doc };
        }
        return post;
    } catch (err) {
        // 统一捕获查询过程的所有错误
        // 若已经是自定义抛出的错误则直接抛出，否则包装为500错误
        throw err.status ? err : { status: 500, error: '服务器内部错误' };
    }
}

如果是按ID查询单个文档，建议直接使用 findById 方法，无需处理数组返回结果：

const post = await Posts.findById(inputId);

无需拼接查询条件，代码更简洁，性能也略优。

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