Node.js Express控制器调用类方法时报productsService undefined错误

这个错误是类实例方法作为回调传递时丢失this上下文导致的。

你在路由里写rescue(ProductsController.getAll)时，是把getAll函数本身从Controller实例上单独拆出来作为参数传递，Express后续触发这个回调时，函数内部的this不再指向你实例化的ProductsController对象，类的方法默认运行在严格模式下，此时this值为undefined，自然读不到this.productsService属性。

任选一种方案即可解决：

• 显式绑定this指向
  传参时用bind永久绑定方法的this为Controller实例：

  // routes文件修改
  productsRouter
    .route('/products')
    .get(rescue(ProductsController.getAll.bind(ProductsController)))
  

• 箭头函数定义类方法
  箭头函数没有自身的this，会继承定义位置外层的this（也就是Controller实例），修改Controller的方法定义：

  class ProductsController {
    constructor() {
      this.productsService = ProductsService;
    }
  
    // 改为箭头函数赋值
    getAll = async (_req, res) => {
      const products = await this.productsService.getAll();
      return res.status(StatusCodes.OK).json(products);
    }
  }
  

  也可以选择在构造函数里统一绑定：

  constructor() {
    this.productsService = ProductsService;
    this.getAll = this.getAll.bind(this);
  }
  

• 包裹一层匿名回调
  路由里不直接传方法引用，通过匿名函数触发实例方法，保证调用时上下文正确：

  productsRouter
    .route('/products')
    .get(rescue((req, res, next) => ProductsController.getAll(req, res, next)))
  

你当前各层的依赖赋值写法虽然能跑，但不符合SOLID的依赖倒置原则：现在类内部硬编码了依赖的其他层实例，后续做单元测试很难Mock依赖。建议改成构造函数注入依赖的形式，以Service层为例：

class ProductsService {
  // 依赖通过构造函数传入，不在内部硬编码引用
  constructor(productsModel) {
    this.model = productsModel;
  }

  async getAll() {
    const [products] = await this.model.getAll();
    return products;
  }
}

// 导出时注入Model实例
module.exports = new ProductsService(ProductsModel);

Controller层也按同样方式注入Service实例即可，后续替换依赖、写单测都会方便很多。

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