使用Express+Mongoose注册用户时遇ERR_HTTP_HEADERS_SENT错误求助

嘿，这个错误我在做Express用户注册功能时也踩过坑，其实核心原因很好理解——你在同一个请求里给客户端发了不止一次响应！

为啥会触发这个错误？ #

看你的代码逻辑：

1. 先查邮箱是否存在，如果存在，就给客户端返回一个400的错误响应
2. 但这时候代码并没有停下来，会继续往下执行，去查用户名是否存在
3. 如果用户名也存在，又会再发一次400响应

HTTP协议要求一个请求只能对应一个响应，你发两次自然就触发这个报错了。哪怕邮箱和用户名只有一个重复，比如邮箱重复了，你发完响应后代码还是会继续执行查用户名的逻辑，虽然这时候不会再发响应，但其实已经埋下了隐患（比如如果后续还有其他响应逻辑，还是会出问题）。

两种解决办法，按需选 #

办法一：发完响应就「刹车」

在每个res.json()前面加上return，这样一旦发送了响应，函数就直接终止，不会再执行后面的代码：

try {
  let user = await User.findOne({ email })
  if (user) {
    return res
      .status(400)
      .json({ status: 'error', body: [{ msg: 'email already exist' }] })
  }
  
  user = await User.findOne({ name })
  if (user) {
    return res
      .status(400)
      .json({ status: 'error', body: [{ msg: 'name already exist' }] })
  }
  
  // ... 这里写后续的注册逻辑（比如创建新用户）
} catch(err) {
  // 别忘了处理异常，同样要加return避免重复响应
  return res.status(500).json({ status: 'error', body: [{ msg: err.message }] })
}

办法二：合并查询，性能更优（推荐）

其实你没必要查两次数据库，用MongoDB的$or操作符一次就能同时检查邮箱和用户名是否存在，既减少了数据库请求，也从根源上避免了重复响应的问题：

try {
  const user = await User.findOne({
    $or: [
      { email: email },
      { name: name }
    ]
  })
  
  if (user) {
    // 判断到底是邮箱还是用户名重复
    const errorMsg = user.email === email ? 'email already exist' : 'name already exist'
    return res.status(400).json({ status: 'error', body: [{ msg: errorMsg }] })
  }
  
  // ... 后续注册逻辑
} catch(err) {
  return res.status(500).json({ status: 'error', body: [{ msg: err.message }] })
}

小提醒 #

以后再遇到这类错误，第一反应要检查：是不是在同一个请求处理函数里，多次调用了res.send()、res.json()、res.redirect()这些发送响应的方法？尤其是异步逻辑里，一定要确保流程控制正确，别让代码“乱跑”。

内容的提问来源于stack exchange，提问作者David L. Rajcher