Checkstyle 警告 RequireThis
就个人而言,我发现它更具可读性(例如,不是从 IDE 而是从代码审查工具读取时),但是除了可读性之外,还有其他原因进行此检查吗?
这只是惯例,有一些优点和缺点。
如果不使用 IDE,则此检查最有用。它可能有助于代码审查消除代码引用的歧义,提高可读性并防止与局部变量或参数访问混淆。
另一方面,大多数编码风格不建议在任何地方都使用this
,因为这可能被认为是不必要的、多余的和过于冗长的。
请考虑以下事项:
int b,c;
public MyConstructor(int a, int b){
b = a;
c = b;
}
这是合法的。 它编译。 它可能不会做你想要它做的事。 添加this
可以确保您不会意外地隐藏您不想隐藏的内容。