模拟社交网站用户名查重报错致应用崩溃，请求技术协助

你的代码出现崩溃的核心原因有两个：一是Promise回调里的错误无法被外层try-catch捕获，二是用户名检查和用户插入的异步逻辑没有同步，导致检查还没完成就执行了插入，同时未捕获的Promise拒绝会直接导致应用崩溃。另外遍历全表检查用户名的方式效率极低，还存在并发竞态风险。

修正后的代码 #

try {
  // 直接用MongoDB查询检查用户名是否存在，高效且原子性更强
  const existingUser = await mongodb.getDb().db().collection("users").findOne({ userName: req.body.userName });
  
  if (existingUser) {
    throw new Error("username is already in use");
  }

  // 用户名无重复，执行用户创建
  const user = {
    userName: req.body.userName,
    password: req.body.password,
    posts: [],
  };

  const response = await mongodb.getDb().db().collection("users").insertOne(user);
  if (response.acknowledged) {
    res.status(201).json(response);
  }
} catch (error) {
  // 统一捕获所有异常，区分错误类型返回对应状态码
  const statusCode = error.message.includes("username is already in use") ? 400 : 500;
  res.status(statusCode).json({ message: error.message });
  console.log(error);
}
};

关键改进说明 #

• 替换遍历为数据库查询：用findOne直接查询目标用户名，效率远高于全表遍历，同时能避免并发场景下的竞态问题（两个请求同时检查时，数据库层面的查询更可靠）。
• 统一异步流程：全程使用await处理异步操作，确保用户名检查完成后再执行插入，所有错误都能被外层try-catch捕获，不会出现未处理的Promise拒绝。
• 规范HTTP状态码：用户名重复属于客户端输入错误，返回400 Bad Request而非500，更符合HTTP语义。
• 简化代码结构：移除嵌套的try-catch，让逻辑更清晰，便于维护。

额外保障措施 #

为了彻底杜绝用户名重复，建议给userName字段创建唯一索引（只需在数据库初始化时执行一次）：

mongodb.getDb().db().collection("users").createIndex({ userName: 1 }, { unique: true });

这样即使代码出现疏漏，MongoDB也会直接拒绝重复插入请求，抛出唯一键冲突错误，进一步保障数据一致性。

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