否定が 3 つ以上あるメソッドを報告します。 このようなメソッド名は混乱を招く可能性があります。

例:


  void doSmth(int a, int b, boolean flag1, boolean flag2) {
    if (!flag && !flag2) {
      if (a != b) {
        doOther();
      }
    }
  }

否定がなければ、メソッドがより分かりやすくなります。


  void doSmth(int a, int b, boolean flag1, boolean flag2) {
    if (flag1 || flag2 || a == b) return;
    doOther();
  }

インスペクションの構成: