控制器类中私有属性实例化ViaCepService出现500错误求助

问题场景 #

在控制器类内部以私有属性形式实例化服务类时，调用接口出现500错误：

import {ViaCepService} from "../../services/viacep/viaCepService";

export class ViaCepController {
  private viaCepService: ViaCepService = new ViaCepService()
  async buscarCep(req, reply) {
    return await this.viaCepService.buscarCep(req, reply)
  }
}

将服务实例化移到类外部后，代码可正常运行：

import {ViaCepService} from "../../services/viacep/viaCepService";

const viaCepService: ViaCepService = new ViaCepService()

export class ViaCepController {
  async buscarCep(req, reply) {
    return await viaCepService.buscarCep(req, reply)
  }
}

对应的服务类代码：

import {feignClient} from "../../shared/feignClient";

export class ViaCepService {
    buscarCep = async (req, reply) => {
        const cep: number = req.params.cep;
        const response: any = await feignClient.get(
            `https://viacep.com.br/ws/${cep}/json/`
        );
        return reply.send(response.data)
    }
}

核心原因：this指向丢失 #

问题出在控制器方法的this绑定上。当你把buscarCep作为路由处理函数直接注册时（比如在Express/Fastify这类框架中），方法的this不会自动指向ViaCepController的实例，而是会变成undefined（严格模式下）或全局对象。此时访问this.viaCepService会抛出Cannot read properties of undefined (reading 'viaCepService')错误，最终触发500状态码。

而将服务实例化移到类外部时，viaCepService是模块级变量，不需要依赖this就能访问，因此不会出现这个问题。

解决方案 #

方案1：将控制器方法改为箭头函数 #

箭头函数会自动捕获当前上下文的this，确保方法内的this始终指向控制器实例：

import {ViaCepService} from "../../services/viacep/viaCepService";

export class ViaCepController {
  private viaCepService: ViaCepService = new ViaCepService()
  buscarCep = async (req, reply) => { // 改为箭头函数
    return await this.viaCepService.buscarCep(req, reply)
  }
}

方案2：手动绑定this到控制器实例 #

在注册路由时，使用bind方法强制绑定this指向：

// 路由注册代码示例
const controller = new ViaCepController();
app.get('/cep/:cep', controller.buscarCep.bind(controller));

方案3：使用依赖注入框架（可选） #

如果你的项目使用依赖注入工具（如TypeDI、NestJS的DI系统），可以通过注入的方式管理服务实例，避免手动处理this绑定问题。

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