Angular2+构造函数中调用导入成员省略this.是否合规？

这是个很常见的疑惑，我来帮你拆解清楚这里面的关键问题：

1. 你的代码为什么能正常运行？ #

首先要明确TypeScript的一个特性——参数属性（Parameter Properties）：当你在构造函数的参数前加上private/public/protected修饰符时，TypeScript会自动帮你完成两件事：

• 把这个参数声明为类的实例属性（相当于在类里手动写了private _importedModule: SomeModule;）
• 在构造函数内部，这个参数本身也是一个局部变量，所以你直接写_importedModule.doesSomethingSpecial()时，其实是在访问构造函数的局部变量，而非实例属性。

这就是为什么你的代码能正常运行，语法上完全合法。

2. 为什么评审说你“作用域使用不当”？ #

这里的核心是局部变量和实例属性的作用域语义区别：

• 构造函数里的_importedModule局部变量，作用域仅限于构造函数内部，出了构造函数就无法访问；
• this._importedModule是类的实例属性，作用域覆盖整个类的所有方法（比如ngOnInit、自定义业务方法等）。

虽然在构造函数内部，两者指向的是同一个对象，但写法上混淆了“临时局部变量”和“类成员属性”的语义，这是编码风格和可维护性层面的问题。

3. 这为什么属于不良编码习惯？ #

（1）语义模糊，增加理解成本 #

其他开发者看到你直接写_importedModule时，第一反应会以为这是构造函数里定义的临时变量，而非类的成员属性。当构造函数代码较长时，这种写法会让代码的可读性下降，需要额外时间理清变量的来源和作用范围。

（2）破坏代码一致性 #

Angular官方的最佳实践，以及绝大多数TypeScript项目的编码规范中，类的成员属性都是通过this.来访问的。统一的写法能让整个项目的代码风格保持一致，降低团队协作时的沟通和维护成本。

（3）潜在的重构风险 #

假设后续你需要修改构造函数：比如把private _importedModule改成普通参数（去掉private），这时候直接调用_importedModule依然能运行，但this._importedModule会报错；反过来，如果之后你想在类的其他方法中使用这个模块，用局部变量的写法会直接报错，而this.的写法则能无缝迁移。

举个直观的例子：

// 错误的写法，ngOnInit里无法访问构造函数的局部变量
export class SomeComponent {
  constructor(private _importedModule: SomeModule) {
    _importedModule.doesSomethingSpecial();
  }

  ngOnInit() {
    _importedModule.doSomethingElse(); // 这里会直接报错！
  }
}

// 正确的写法，整个类都能访问实例属性
export class SomeComponent {
  constructor(private _importedModule: SomeModule) {
    this._importedModule.doesSomethingSpecial();
  }

  ngOnInit() {
    this._importedModule.doSomethingElse(); // 正常运行
  }
}

（4）影响IDE的支持 #

当你使用this._importedModule时，IDE能准确识别这是类的成员属性，提供更精准的代码提示、自动补全、全局重命名重构等功能。而局部变量的重构范围仅限于构造函数内部，容易出现遗漏。

总结 #

你的代码语法上没有错误，能正常运行，但不符合Angular和TypeScript的编码最佳实践。评审要求你改成this.的核心原因是：这种写法混淆了局部变量和实例属性的语义，降低了代码的可读性、可维护性和一致性，长期来看会增加团队的协作成本和潜在的bug风险。

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