如果不是绝对必要,我是否应该删除 Angular CLI 创建组件时生成的 OnInit 实现?
Should I remove the OnInit implementation which is generated when a component is created by the Angular CLI if it isn't strictly necessary?
我的团队正在开展一个 Angular 项目,并试图建立团队代码风格约定。一位队友鼓励我删除使用 Angular CLI.
创建组件时生成的 OnInit
实现
我的队友赞成删除 OnInit
实现的论点是,这是不必要的代码,如果不删除未使用的代码,可能会导致其他开发人员误解,因为他们无法认识到实现是不必要的。总的来说,这个论点对我来说似乎是合理的,但是删除一些生成的代码的想法让我有些不舒服。
我很好奇 Angular 社区中的其他人是否考虑过这种类型的编码约定,以及是什么因素导致了这个决定。您是否要求队友删除并非绝对必要的生成代码?你会认为这是一个不合理的约定吗?
所以真的没有正确答案,由您和您的团队决定此类惯例。出于与删除 commented-out 代码块相同的原因,我更喜欢删除空函数;它只会占用 space 并分散重要代码的注意力。
这两种方式都不是什么大问题,但 SonarQube 等静态代码分析工具会将其视为代码异味(参见 https://rules.sonarsource.com/javascript/RSPEC-1186)。
我的团队正在开展一个 Angular 项目,并试图建立团队代码风格约定。一位队友鼓励我删除使用 Angular CLI.
创建组件时生成的OnInit
实现
我的队友赞成删除 OnInit
实现的论点是,这是不必要的代码,如果不删除未使用的代码,可能会导致其他开发人员误解,因为他们无法认识到实现是不必要的。总的来说,这个论点对我来说似乎是合理的,但是删除一些生成的代码的想法让我有些不舒服。
我很好奇 Angular 社区中的其他人是否考虑过这种类型的编码约定,以及是什么因素导致了这个决定。您是否要求队友删除并非绝对必要的生成代码?你会认为这是一个不合理的约定吗?
所以真的没有正确答案,由您和您的团队决定此类惯例。出于与删除 commented-out 代码块相同的原因,我更喜欢删除空函数;它只会占用 space 并分散重要代码的注意力。
这两种方式都不是什么大问题,但 SonarQube 等静态代码分析工具会将其视为代码异味(参见 https://rules.sonarsource.com/javascript/RSPEC-1186)。