Node.js模块错误处理优化咨询：嵌套回调代码冗余问题

你遇到的这个**回调地狱（Callback Hell）**问题在早期Node.js代码里很常见——虽然异步操作确实需要做错误检查，但我们完全可以通过Promise结合async/await来让代码更整洁，同时统一处理错误，不用再写层层嵌套的if-else。

第一步：把回调风格的方法转为Promise风格 #

Mongoose的方法本身就支持Promise，bcrypt也提供了Promise版本的API（如果是旧版本，也可以用Node.js内置的util.promisify包装回调方法）。我们先改写addUser模块：

// user.js 模块
const bcrypt = require('bcrypt');

module.exports.addUser = async function(newUser) {
  // 检查用户是否已存在
  const existingUser = await User.findOne({ email: newUser.email });
  if (existingUser) {
    // 抛出自定义错误，方便后续区分处理
    throw new Error('User already exists');
  }

  // 生成盐并哈希密码
  const salt = await bcrypt.genSalt(10);
  newUser.password = await bcrypt.hash(newUser.password, salt);
  
  // 保存新用户
  await newUser.save();
  return true; // 返回创建成功的标记
}

第二步：路由中用async/await + try/catch统一处理错误 #

把路由处理函数改成async函数，用try/catch捕获所有异步操作的错误，代码结构会变得非常扁平：

// user.js 路由
router.post('/register', async function(req, res, next) {
  try {
    const newUser = new User({
      firstname: req.body.firstname,
      lastname: req.body.lastname,
      email: req.body.email,
      password: req.body.password,
    });

    await User.addUser(newUser);
    res.sendStatus(200);
  } catch (err) {
    // 区分自定义错误与其他系统错误
    if (err.message === 'User already exists') {
      return res.status(403).send('User already exists');
    }
    // 其他错误交给全局错误处理中间件处理
    next(err);
  }
});

这样优化的好处 #

• 代码扁平化：彻底去掉了嵌套回调，逻辑一目了然，可读性大幅提升
• 错误统一处理：所有异步操作的错误都会被try/catch捕获，不用在每个回调里重复写if(err) cb(err)
• 可维护性更高：后续修改逻辑时，不用在嵌套结构里来回跳转，降低出错概率

补充小提示 #

• 如果你的bcrypt版本没有Promise API，可以用util.promisify包装：

  const { promisify } = require('util');
  const genSalt = promisify(bcrypt.genSalt);
  const hash = promisify(bcrypt.hash);
  

• 更规范的做法是自定义错误类（比如UserExistsError），代替用message判断错误类型，后续扩展错误处理更灵活

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