Post请求无法同时校验两个Body参数的问题求助

你的代码无法同时校验两个参数的核心问题是：两个独立的if语句导致逻辑分支混乱，else仅与第二个if绑定，且未在错误响应后终止代码执行，可能引发多次发送响应的问题。

修正方案一：统一校验并返回合并提示 #

app.post("/tshirt/:id", (req, res) => {
    const {id} = req.params;    
    const {logo, size} = req.body;

    // 收集所有缺失的必填参数
    const missingParams = [];
    if (!logo) missingParams.push("logo");
    if (!size) missingParams.push("size");

    if (missingParams.length > 0) {
        res.status(418).send(`Missing required parameters: ${missingParams.join(", ")}`);
        return; // 终止后续代码执行，避免重复发送响应
    }

    // 所有参数合法时返回成功结果
    res.status(200).send({
        tshirt: `👕 with your ${logo} is ready`,
        size: `${size}`
    });
})

修正方案二：分别校验并返回单独提示 #

如果需要针对每个缺失参数返回精准提示，可使用以下写法：

app.post("/tshirt/:id", (req, res) => {
    const {id} = req.params;    
    const {logo, size} = req.body;

    if (!logo) {
        res.status(418).send("It needs a logo");
        return; // 必须终止执行，否则会继续运行后续代码
    }

    if (!size) {
        res.status(418).send("It needs a size");
        return;
    }

    res.status(200).send({
        tshirt: `👕 with your ${logo} is ready`,
        size: `${size}`
    });
})

关键说明 #

• 两种方案都通过return在发送错误响应后终止代码执行，避免Express因重复发送响应报错
• 方案一适合一次性告知用户所有缺失参数的场景，方案二适合需要精准提示单个缺失参数的场景
• 校验条件!logo会覆盖参数为undefined、null、空字符串等空值的情况，若仅需校验undefined，可改为logo === undefined

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