使用Mongoose pre save钩子出现ERR_HTTP_HEADERS_SENT错误的解决求助

你的问题出在两个关键地方：Mongoose预保存钩子的逻辑漏洞和路由响应处理的顺序问题，我们一步步来修复：

1. 修复Pre-Save钩子的逻辑错误 #

你的钩子代码中，当条件不满足时调用了next(err)，但没有终止函数执行，导致后续的next()也会被调用。这会让Mongoose同时处理错误流程和正常流程，引发逻辑混乱。

修改后的钩子代码：

mySchema.pre('save', function(next) {
  if (condition_is_not_met) {
    const err = new Error('Condition was not met');
    return next(err); // 添加return，阻止后续代码执行
  }
  next();
});

通过return next(err)，我们确保在抛出错误后不会再执行正常的next()调用，让Mongoose正确处理错误分支。

2. 修复路由的响应处理逻辑 #

你的路由代码中，当save回调返回错误时，你发送了500响应，但没有终止函数执行，导致后续的res.status(200).send(s)也会被执行——这就是ERR_HTTP_HEADERS_SENT错误的直接原因：你试图在已经发送错误响应后，再次发送成功响应的头信息。

我们有两种修复方式：

方式一：在错误处理中添加return #

router.post('/mySchema', returnType, (req, res) => {
  const s = new mySchema(req.body);
  s.save((err) => {
    if (err) {
      const msg = { message: 'Could not add', error: err.message }; // 建议只返回错误信息，避免暴露内部细节
      return res.status(500).send(msg); // return终止函数，避免执行后续代码
    }
    res.status(200).send(s);
  });
});

方式二：使用async/await简化代码（更推荐） #

回调嵌套容易出现这类逻辑错误，改用async/await可以让代码更清晰：

router.post('/mySchema', returnType, async (req, res) => {
  try {
    const s = new mySchema(req.body);
    await s.save();
    res.status(200).send(s);
  } catch (err) {
    const msg = { message: 'Could not add', error: err.message };
    res.status(500).send(msg);
  }
});

这种方式下，一旦save抛出错误，代码会直接进入catch块，不会执行后续的成功响应逻辑，从根源避免了重复发送响应的问题。

为什么会出现这个错误？ #

ERR_HTTP_HEADERS_SENT本质上是因为你尝试在HTTP响应已经发送（头信息已返回给客户端）后，再次修改或发送响应头。你的路由代码中，错误分支没有终止函数，导致成功响应的代码也被执行，触发了这个错误。同时钩子的逻辑问题可能让save的回调被异常触发，加重了这个问题。

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