Express+Mongoose中嵌套Try Catch是否合理？能否用单个Try Catch替代？

原代码的问题 #

你的这段嵌套Try Catch代码存在两个关键问题：

1. 当Hotel.findByIdAndUpdate执行失败时，你调用了next(err)但没有终止后续代码执行，之后会继续执行res.status(200).json(savedRoom)，这会导致Express抛出Cannot set headers after they are sent to the client错误——因为错误中间件已经会发送一次错误响应，你又发了一次成功响应，重复设置了HTTP头。
2. 两个数据库操作（保存房间、更新酒店）没有原子性保障：如果房间保存成功，但酒店更新失败，数据库里会有一条孤立的房间数据（不属于任何酒店），导致数据不一致。

是否需要嵌套Try Catch？ #

完全不需要嵌套。多个异步操作可以放在同一个Try块里统一捕获异常，嵌套反而会增加代码复杂度，还容易出现上面的重复响应问题。

最优解决方案 #

推荐方案：使用Mongoose事务保证原子性 #

因为这两个操作是强关联的，必须同时成功或同时失败，所以要用MongoDB事务（需要MongoDB版本≥4.0，且是副本集部署）：

const session = await mongoose.startSession();
session.startTransaction();

try {
  // 在会话中执行保存房间操作
  const savedRoom = await newRoom.save({ session });
  // 在同一个会话中执行酒店更新操作
  await Hotel.findByIdAndUpdate(hotelId, { $push: { rooms: savedRoom._id } }, { session });

  // 两个操作都成功，提交事务
  await session.commitTransaction();
  res.status(200).json(savedRoom);
} catch (err) {
  // 任何一步失败，回滚事务
  await session.abortTransaction();
  next(err); // 交给错误处理中间件处理
} finally {
  // 结束会话
  session.endSession();
}

简化方案（不推荐，仅允许数据不一致场景） #

如果你的业务可以接受房间保存成功但酒店更新失败的情况（不推荐），可以合并Try Catch并避免重复响应：

try {
  const savedRoom = await newRoom.save();
  // 这里如果更新失败，直接抛出错误进入Catch块
  await Hotel.findByIdAndUpdate(hotelId, { $push: { rooms: savedRoom._id } });
  
  res.status(200).json(savedRoom);
} catch (err) {
  next(err);
}

或者如果想单独处理酒店更新的错误（比如不影响房间保存的响应，但要记录错误）：

try {
  const savedRoom = await newRoom.save();
  
  try {
    await Hotel.findByIdAndUpdate(hotelId, { $push: { rooms: savedRoom._id } });
  } catch (hotelErr) {
    // 可以在这里记录酒店更新失败的日志，而不影响房间创建的响应
    console.error('Failed to update hotel rooms:', hotelErr);
    // 注意：不要调用next(err)，否则会和后面的res.send冲突
  }
  
  res.status(200).json(savedRoom);
} catch (err) {
  next(err);
}

但这种方案会导致数据不一致，除非你的业务逻辑允许，否则强烈推荐使用事务方案。

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