4

開発者間で一貫したコード スタイルを提供する方法として、checkstyle の実装を開始しました。チェックの 1 つは、すべてのインスタンス変数が「this.」を使用した参照であることを要求します。私は "this." を使用する習慣がなかったので、何千ものインスタンスを修正する必要があります。

実行して適切なコードを追加できるツールまたは IntelliJ プラグインはありますか?

4

3 に答える 3

5

チェックの 1 つは、すべてのインスタンス変数が「this」を使用して参照されることを要求します。

したがって、「checkstyle」は間違っています。

私は「これ」を使う習慣がありません。

その通りです。冗長です。よかったね。

そのため、修正が必要な数千のインスタンスがあります。

だからあなたは間違っています。

「修正が必要」なインスタンスありません。

やらないでください。冗長です。冗長な作業を行わないでください。誰もお金を払いたくないのに、なぜ支払う必要があるのでしょうか? それを必要としないように、checkstyle 構成を変更します。ツールにお金の使い方を教えさせないでください。また、「開発者間で一貫したコード スタイル」が機能する製品を提供するために不可欠であることにだまされないでください。コードが判読可能で保守可能である限り、そうではありません。私は、この問題に関して、考える以上に多くの時間とお金が浪費されているのを見てきました。

于 2013-02-04T09:13:21.743 に答える
1

チェックの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 に答える