注册接口重复创建用户及ERR_HTTP_HEADERS_SENT错误解决

问题说明 #

当前注册接口存在两个核心问题：

1. 检测到用户名/邮箱已存在时，调用res.send('exists')后使用return无法终止整个函数，后续代码仍会执行并保存用户实例，导致数据库出现重复记录。
2. 执行res.send("success")时触发Error [ERR_HTTP_HEADERS_SENT]: Cannot set headers after they are sent to the client错误，原因是同一请求中多次调用响应发送方法。

修改后的完整代码 #

app.post('/register', async function(req, res) {
  // 解构请求体参数，简化代码
  const { username, email, password } = req.body;
  console.log(email, username, password);

  try {
    // 检查用户名是否已存在
    const existingUserByUsername = await userModel.findOne({ username });
    if (existingUserByUsername) {
      // 直接返回响应并终止函数
      return res.send('exists');
    }

    // 检查邮箱是否已存在
    const existingUserByEmail = await userModel.findOne({ email });
    if (existingUserByEmail) {
      return res.send('exists');
    }

    // 创建新用户实例
    const userInstance = new userModel({
      username,
      password,
      email,
      imagelink: "/images/blank",
      links: [],
      plan: 0
    });

    // 保存用户到数据库
    const savedUser = await userInstance.save();
    if (savedUser) {
      res.send('success');
    }
  } catch (error) {
    // 统一处理所有异步操作错误
    console.log(error);
    res.status(400).send("Bad Request");
  }
});

核心修改点解析 #

• 移除.then()回调，改用纯async/await：原代码混用await和.then()，回调内的return只能终止回调函数，无法停止外层接口逻辑。纯async/await写法下，return res.send()会直接终止整个接口函数，避免后续保存用户的代码执行。
• 确保单次请求只发送一次响应：每次检测到重复时，通过return立即终止函数，不会触发后续的响应发送操作，彻底解决重复发送响应导致的头信息错误。
• 统一错误捕获：将所有数据库异步操作放入try/catch块，集中处理查询、保存过程中的异常，避免遗漏错误处理导致的潜在问题。

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