このコードスニペットを書き直すと良い方法がありますか？

Question

は、次の点を考慮a、b、cとdがboolになる式です

if ((a || b) && (c || d)) 
{ 
    if (a && c) {...} 
    else if (b && d) {...} 
    else {...} 
} 
else {...} 

。

コードから明らかなように、これらの式のいくつかは再計算されています。 a）明快さに関してより良いアプローチがありますか？ b）効率性？

Answer 1

簡単な答え：if文の前にbool変数で式を格納できます。

式に関数呼び出しが含まれていない場合（つまり、文の間で変更されないことが明らかな場合）、コンパイラはこのように最適化することを期待します。

Answer 2

サプライズ：あなたは実際に外側のifステートメントが必要と思わない。 (a && c)または(b && d)が真実であるならば、つまり

if (((a && c) || (a && d)) || ((b && c) || (b && d))) 

に

if ((a || b) && (c || d)) 

、その後、外ifは常にtrueと評価されます：外ifから書き換えることができるロジックの法則を使用することを理解します、それは落とすことができます。代わりに、あなたはこの記述することができます。

if (a && c) {...} 
else if (b && d) {...} 
else if ((a && d) || (b && c)) {...}  // your original 'inner' else 
else {...}        // your original 'outer' else