複合AND / OR ifステートメントのスタイル設定

読みやすくするために、複雑な複合AND / OR ifステートメントをどのようにスタイルしますか？どのようにインデントし、改行をどこに配置しますか？私の特定の状況は次のようなものです。すべてを1行に分割するよりも間違いなく優れていますが、それでも面倒です。

if (
    (
        x == y
        && a != b
        && p.isGood() 
        && (
            i + u == b
            || q >= a
        )
    )
    || k.isSomething()
    || m > n
) {
    doSomething();
}

小さなステップごとにブール変数を作成します。

bool step1 = i + u == b || q >= a;
bool step2 = a != b && p.isGood() && group1;
bool step3 = group2 || k.isSomething() || m > n;
if (step3) { doSomething(); }

もちろん、これはLacrymologyの答えに似ていますが、各ステップの名前が異なる点が異なります。

名前場合はstep1、step2とstep3の良好な概念的な意味をなすかの方法で、これはこれまでで最も読みやすいでなければなりません。p.isGood()そしてk.isSomething()それらの機能は高価である場合や、非常にタイトなループでこのコードを実行している場合、このオプションではありませんので、時には、それはあなたの元のコードではないだろう状況で起動することができます。

一方、新しい変数を作成するとパフォーマンスが低下することを心配する必要はありません。優れたコンパイラーはそれらを最適化します。

長方形の衝突検出の例（前述のパフォーマンスヒットのため、おそらく使用しないでしょう）：

if((a.x + a.width >= b.x || b.x + b.width >= a.x)
 && (a.y + a.height >= b.y || b.y + b.width >= a.y)
)
{ collision(); }

になるかもしれません：

bool horizMatch = a.x + a.width >= b.x || b.x + b.width >= a.x;
bool vertMatch = a.y + a.height >= b.y || b.y + b.width >= a.y;
if(horizMatch && vertMatch) { collision(); }

また、コードをそのままにしておきたい場合は、それでもまったく問題ないと思います。あなたのコードは非常に読みやすいと正直に思います。明らかに、私は正確にa b x y i u p k m nは何なのかわかりませんが、構造に関しては、私には良さそうです。

通常、条件が複雑になった場合は、コードをよりモジュール化してリファクタリングします。

このレベルの複雑さで、私はもっとこのようなことをしたい

bool doIt = x == y && a != b && p.isGood();
doIt &= ( i + u == b || q >= a);
doIt |= k.isSomething() || m > n;

if(doIt)
{
    doSomething();
}

いですが、読みやすく、コンパイラがリファクタリングする方法を知っていることは確かです。

一方、IFステートメントを記述している状況で自分自身を見た場合、解決策を再考します。私はそれを簡単に行う方法があるか、少なくともその条件の一部を抽象化する方法があるからです（たとえば：x == y && a != b && p.isGood()本当にただ意味しthis->isPolygon()、私はその方法を作ることができます。

私は時間の経過とともに垂直方向の配置にあまり夢中になりませんが、マルチライン式の一般的な形式は...

if (   (   (expr1 == expr2)
        || (expr3 == expr4)
        || (expr5 == expr6)
       )
    && (   (expr7 == expr8)
        || (expr9 == expra)
       )
   )
{
  blah;
}

キーポイント...

• 閉じ括弧は、中括弧と同様に、開いた括弧と垂直に整列します。
• 1行に収まる部分式は1行に収まり、左に垂直に配置されます。読みやすくするために、これらの単一行部分内の挿入演算子も垂直方向に揃えられています。
• 閉じ括弧は自然にほぼ空白の行を作成し、視覚的にグループ化するのに役立ちます。

時々、フォーマット+して*このような他の演算子をます。非常に少数の複雑な式は、積和または積積形式（ブールの「和」および「製品」を参照できる）をとるので、おそらく一貫したスタイルが価値があるほど十分に一般的です。

ただし、これには注意してください。インデントを使用して複雑すぎる式を読みやすくするよりも、リファクタリング（式の一部を関数に移動するか、中間部を変数に計算して保存する）を行う方が良い場合がよくあります。

あなたが右カレンを右手に積み重ねることを好むなら、私はそれを嫌いではありませんが、私はそれがそれほど悪くないと思います。ただし、あまりにも遠くに行くと、間違いによって、インデントが括弧の動作を誤って表す可能性があります。

if (   (   (expr1 == expr2)
        || (expr3 == expr4)
        || (expr5 == expr6))

    && (   (expr7 == expr8)
        || (expr9 == expra)))
{
  blah;
}

http://www.codinghorror.com/blog/2006/01/flattening-arrow-code.html

JohnFxの回答とLacrymologyの回答に同意します。私は小さな目標を達成する関数のセット（できれば静的）を構築し、それらをスマートな方法で構築します。

それでは、このようなものはどうですか？これは完璧な解決策ではありませんが、機能します。これをさらにクリーンアップする方法がありますが、より具体的な情報が必要です。注：コンパイラーは賢いので、このコードは同じくらい速く実行する必要があります。

// Currently based on members or global vars
// (which is often a bad idea too)
function doSomethingCondirionally()
{
  if (k.isSomething() || m > n)
  {
    doSomething();
    return;
  }

  // Else ... 
  if (x != y) return;
  if (a == b) return;
  if (!p.isGood()) return;

  // Final, positive check
  if (i + u == b || q >= a)
  {
    doSomething();
  }
}

その価値については、あなたの例が私が書いた複雑な述語によく似ていることに驚いた。複雑な述語は保守性や読みやすさのために最大ではないが、ときどき登場することにも同意します。

この部分が正しいことを強調してみましょう。&& a != b 論理的なコネクタを行末に置かないでください。視覚的に見逃すのは簡単すぎます。行の最後に演算子を決して配置しない別の場所は、そのような演算子を使用する言語での文字列連結です。

これを行う：

String a = b
   + "something"
   + c
   ;

これをしないでください：

String a = b +
   "something" +
   c;

条件式がそれほど複雑な場合、通常は条件に分割する必要があることを示しています。おそらく、1つの句を中間変数に割り当てることができます。おそらく、1つの節をヘルパーメソッドに変えることができます。私は通常、1行にそれほど多くのANDやORを持たないことを好みます。

コードを複数のステートメントに分割して、理解しやすくすることができます。しかし、本当の忍者はこのようなことをするでしょう。:-)

if
(
    (
        x == y
    &&
        a != b
    &&
        p.isGood()
    &&
        (
            i + u == b
        ||
            q >= a
        )
    )
||
    k.isSomething()
||
    m > n
)
{
    doSomething();
}