4

我们已经开始实施 checkstyle 来为开发人员提供一致的代码风格。其中一项检查要求所有实例变量都是使用“this.”引用的。我没有使用“this.”的习惯，因此有数千个需要修复的实例。

是否有可以运行并添加适当代码的工具或 IntelliJ 插件？

4

3 回答 3

5

在eclipse中：Window - Preferences - Java - Editor - Save actions - Configure... - Member access -

在这里，您选中“使用此”复选框。

在每次保存代码时，它都会进行更正。

我刚找到这个页面。

对于那些想要有效使用 eclipse 的人来说，阅读永远不会太晚。我想也会有类似于 Netbeans 和其他的东西。

于 2013-02-04T08:51:11.223 回答
5

其中一项检查要求使用“this”引用所有实例变量。

所以'checkstyle'是错误的。

我一直没有使用“this”的习惯。

完全正确。这是多余的。对你有益。

所以有成千上万的实例需要修复。

所以你错了。

您没有任何“需要修复”的实例。

不要这样做。这是多余的。不要做多余的工作。没有人愿意为此付钱，他们为什么要付钱呢？修改您的 checkstyle 配置，以免需要它。不要让工具告诉你如何花钱，也不要误以为“开发人员之间一致的代码风格”对于交付工作产品至关重要。它不是，只要代码清晰可维护。我看到在这个问题上浪费的时间和金钱比我想的要多。

于 2013-02-04T09:13:21.743 回答
1

其中一项检查要求所有实例变量都是使用“this”的引用。

IMO，最好的解决方案是与开发团队的其他成员一起提出这个问题，并同意关闭那个愚蠢的检查。如果添加冗余this关键字可以提高可读性，那么您不得不怀疑阅读代码的人的 Java 技能。

如果他们反击，我会很想以此表达我的厌恶：

public class SomeClass {
  private String thisName;

  public String getName() {
      return this.thisName;
  }

  public void setName(String notThisName) {
      this.thisName = notThisName;
  }

  // and so on
}
于 2013-02-04T09:30:52.970 回答