Express.js try/catch块catch分支响应未触发 注册接口始终返回成功

• db.query是异步回调API，回调函数的执行晚于外层同步代码，你在回调内部抛出的throw err无法被外层的同步try/catch捕获，错误会直接泄漏到Node进程，不会触发catch块的错误响应逻辑。
• 成功响应的res.json写在了异步逻辑外部，代码执行到db.query时不会等待回调执行完成，会直接向下执行返回success: true，所以无论数据库操作最终结果如何，成功响应都会先被发送到客户端。

回调写法修正 #

把所有响应逻辑移到db.query的回调内部，不要在外层写返回逻辑：

register(app, db) {
  app.post('/register', (req, res) => {
    const email = req.body.email
    const password = req.body.password
    
    try {
      // 同步的加密操作错误可以被外层try/catch捕获
      const hashPwd = bcrypt.hashSync(password, 9)
      db.query("INSERT INTO users VALUES (?, ?)", [email, hashPwd], (err) => {
        if (err) {
          // 数据库操作错误直接在回调内处理，不要抛出
          return res.json({
            success: false,
            msg: err.message
          })
        }
        // 成功逻辑放在回调内部，确保数据库操作完成后再返回
        res.json({
          success: true,
          email: email
        })
      })
    } catch (err) {
      res.json({
        success: false,
        msg: err.message
      })
    }
  });
}

推荐async/await写法（更易维护） #

改用支持Promise的MySQL驱动（比如mysql2/promise），同时把bcrypt的同步加密改为异步加密避免阻塞事件循环，所有错误都可以被统一的try/catch捕获：

register(app, db) {
  app.post('/register', async (req, res) => {
    try {
      const { email, password } = req.body
      // 可在此处增加参数校验逻辑，比如邮箱格式、密码长度校验
      const hashPwd = await bcrypt.hash(password, 9)
      await db.query("INSERT INTO users VALUES (?, ?)", [email, hashPwd])
      res.json({
        success: true,
        email
      })
    } catch (err) {
      res.json({
        success: false,
        msg: err.message
      })
    }
  });
}

额外优化建议 #

• 给users表的email字段增加唯一索引，避免重复注册同邮箱账号
• 返回给前端的错误信息不要直接返回原始err对象，避免泄露数据库表结构、字段等敏感信息，可根据错误类型返回友好提示
• 生产环境不要直接返回错误的详细堆栈信息

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