Node.js Web API价格小于0时未返回有效错误信息问题排查

近期开发了一个连接MySQL数据库的简易Node.js Web API，数据库中的price参数要求大于等于0。我在product.service中添加了校验逻辑，但当价格小于0时，请求返回的错误响应为空：

{
    "error": {}
}

服务层校验代码：

async function createProduct(params){
    if(await db.Product.findOne({where: {name: params.name}})){
        throw 'Such a product already exists, please try again';
    }

    if(params.price < 0){
        throw 'The price of the product cannot be null, please try again';
    }

    const product = new db.Product(params);

    await product.save();
}

Controller代码：

router.post('/', createSchema,createProduct);

function createProduct(req, res, next){
    productService.createProduct(req.body)
    .then(() => res.json({message: '{Product was succesfuly created!'}))
    .catch(next);
}

function createSchema(req, res, next){
    const schema = Joi.object({
        name: Joi.string().required(),
        price: Joi.number().min(0).max(99999999.99).precision(2).required()
    });

    validateRequest(req, next, schema);
}

请问为何价格小于0时未返回有效错误信息？

核心原因 #

1. Joi校验已提前拦截非法请求：你在createSchema里用Joi定义了price: Joi.number().min(0)，当传入价格小于0时，请求会被Joi校验直接拦截，根本不会进入服务层的createProduct方法执行。
2. 错误处理链路缺失：你的validateRequest方法（未贴出代码）大概率没有正确将Joi的校验错误转化为可识别的错误对象；同时全局缺少统一的错误处理中间件，无法把捕获到的错误内容填充到响应的error字段中，最终返回空的错误对象。

分步解决 #

1. 修复Joi校验的错误传递逻辑

调整validateRequest方法，确保它能正确捕获Joi校验错误并传递给后续错误处理流程：

function validateRequest(req, next, schema) {
    const { error } = schema.validate(req.body);
    if (error) {
        // 把Joi的错误信息拼接成可读字符串，包装成Error对象传递
        return next(new Error(error.details.map(item => item.message).join('; ')));
    }
    next();
}

2. 添加全局错误处理中间件

在Express应用的最末尾（所有路由定义之后）添加全局错误处理中间件，统一格式化错误响应：

app.use((err, req, res, next) => {
    res.status(400).json({
        error: err.message || '请求参数错误'
    });
});

3. 修正服务层错误文案（可选）

服务层里的价格错误提示文案有误，当前写的是“价格不能为null”，和实际校验逻辑不符，建议修正：

if(params.price < 0){
    throw 'The price of the product cannot be less than 0, please try again';
}

4. 清理冗余校验（可选）

既然Joi已经完成了price >=0的校验，服务层的同逻辑校验可以删除，减少重复代码。如果需要数据库层面的兜底校验，可以在数据库表的price字段设置CHECK约束。

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