Node.js中Promise与Callback哪个更优？两种实现方式咨询

问题说明 #

我在编写Node.js（Express+TypeScript）的用户查询函数时，遇到两种实现方式：一种是通过Promise返回查询结果，另一种是把结果挂载到Express的Response对象上。想搞清楚哪种更符合最佳实践，以及两者在性能、潜在问题上的差异，尤其纠结是用return返回值，还是把结果挂到res对象里。

方案一：Promise返回结果实现 #

class Users{
  static async findByEmail(email: any ) : Promise<Users | undefined>{
    const user: any = await Pools.execute(
      "SELECT * FROM users WHERE email = ?",
      [email])
      .then(rows => { 
        return rows[0];
       })
      .catch(err => console.log(err) );
      return user;
  };
}

router.post(
  "/api/users/signin",
  async (req: Request, res: Response , next: NextFunction) => {
     const { email, password } = req.body;
     const existingUser = await Users.findByEmail(email);
});

方案二：挂载到Response对象实现 #

declare global {
  namespace Express {
    interface Response {
      user?: Users;
    }
  }
}

class Users {
  static async findByEmail(req: Request, res: Response) {
    await Pools.execute(
      "SELECT * FROM users WHERE email = ?",
      [req.body.email])
      .then(rows => { 
         res.user = rows[0];
       })
      .catch(err => console.log(err) );
  };
}

router.post(
  "/api/users/signin",
  async (req: Request, res: Response , next: NextFunction) => {
    await Users.findByEmail(req, res);
    const existingUser = res.user;
});

对比分析与最佳实践建议 #

方案一（返回Promise）：推荐采用 #

这种方式是Node.js开发的通用最佳实践，优势很明显：

• 职责单一，复用性强：findByEmail只负责“根据邮箱查询用户”，不依赖Express的req/res对象，不管是HTTP接口、定时任务还是CLI工具，都能直接调用这个方法。
• 逻辑清晰，可读性高：调用方通过await直接拿到返回值，一眼就能明白代码意图，没有隐藏的副作用。
• 错误处理更规范：当前代码的.catch仅打印错误，可改为抛出错误让调用方统一处理（比如用Express的错误中间件），避免错误被静默吞掉。
• 性能无额外损耗：数据库查询是整个流程的核心耗时点，return返回结果的内存操作几乎不占时间，和方案二性能无实质差异。

优化建议：async/await本身是Promise的语法糖，没必要混用await和.then/.catch，改用try/catch风格更清爽：

class Users{
  static async findByEmail(email: any): Promise<Users | undefined>{
    try {
      const [rows] = await Pools.execute(
        "SELECT * FROM users WHERE email = ?",
        [email]
      );
      return rows[0] as Users | undefined;
    } catch (err) {
      console.error("查询用户失败：", err);
      // 抛出错误让调用方处理
      throw err;
    }
  };
}

// 调用时统一处理错误
router.post(
  "/api/users/signin",
  async (req: Request, res: Response, next: NextFunction) => {
    try {
      const { email, password } = req.body;
      const existingUser = await Users.findByEmail(email);
      // 后续业务逻辑...
    } catch (err) {
      next(err); // 交给Express全局错误中间件处理
    }
});

方案二（挂载到res对象）：不推荐采用 #

这种方式存在不少问题：

• 耦合性太高：findByEmail绑定了Express的res对象，只能在HTTP请求上下文里使用，完全无法复用在其他场景。
• 可读性差，隐式传递：调用方必须知道结果存在res.user里，新接手的开发者很可能找不到数据来源，增加维护成本。
• 潜在冲突风险：如果其他中间件也修改res.user，容易出现数据覆盖的情况，排查问题难度大。
• 错误处理不灵活：当前代码的错误仅打印，调用方无法感知查询失败，可能导致后续逻辑基于未定义的数据执行，引发更多问题。

核心结论 #

优先选择return返回Promise结果的实现方式，这在代码复用性、可读性、可维护性上都远优于方案二。性能上两者没有实质差异，核心耗时都在数据库查询环节。另外，写异步代码时尽量统一用async/await+try/catch，避免混用.then/.catch，让代码更整洁易读。

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