错误：发送后无法设置标头 - 正确处理快速控制器中的错误

Question

我对下面的代码有问题。当遇到错误时，会命中 if 语句并调用 res.send。但是，代码然后继续到下面的 res.json 调用。这会导致标题错误。使用 else 语句是唯一正确的方法吗？我认为调用 res.send 会中断函数。exports.new = function (req, res) {    let team = new Team();    team.name = req.body.name;    team.description = req.body.description;    team.members = [];    // save the team and check for errors    team.save(function (err) {        if (err) {            res.status(400).send(err);        }        res.status(200).json({            message: 'Team successfully created',            team: team        });    });};我也尝试过使用 req、res、next 作为函数参数并使用以下方法：if (err) {    res.status(400);    next(err);}res.status(200).json({    message: 'Team successfully created',    team: team});虽然这不会引发错误，但它仍然继续调用最终的 res.status.json（导致 200，而不是 400 响应）

Answer 1

你得到的错误是因为代码行

  res.status(200).json({

    message: 'Team successfully created',

    team: team

});

无论是否出错，都会被调用。如果出现错误，则 express 已经在此代码中发送了响应标头

if (err) {

    res.status(400).send(err);

}

所以当 express 尝试下面的代码时，它会得到你表达的错误。答案是使用 Rami Jarrar 提供的答案

 // save the team and check for errors

team.save(function (err) {

    if (err) {

        res.status(400).send(err);

    }

    res.status(200).json({

        message: 'Team successfully created',

        team: team

    });

});

Answer 2

您关于res.send()破坏的假设是不正确的，只有当您return. 调用return上res.send()的错误，使其不继续的情况下。如果你得到一个错误，它会将状态设置为 400（在标题中），然后是正文中的错误，但是因为它没有返回它然后尝试将状态设置为 200（在标题中）导致你的错误看到。

// save the team and check for errors

team.save(function (err) {

    if (err) {

        // return here so that it will not continue!

        return res.status(400).send(err);

    }

    // return here if there is no error!

    return res.status(200).json({

        message: 'Team successfully created',

        team: team

    });

});

Express 将中间件链接在一起使用function(req, res, next)，并且可以通过res.send()多个中间件返回结果。链中的下一个中间件在您调用时调用return next()，因此，例如，如果您想在成功时返回另一个中间件中的附加数据，则不会return res.send()而是return next()在res.send().