重複したコードの場合または重複した条件の場合

Question

次のコードサンプルが与えられた場合、より良いと思いますか、これをより洗練された方法でコーディングする別の方法が考えられますか？

if(a || b) { 
    doAB(); 
} 
if(b) { 
    doB(); 
} 

重複コード（2回doAB（））（2回 'B' をチェックする）場合で

重複条件：

if(a) { 
    doAB(); 
} 
if(b) { 
    doAB(); 
    doB(); 
} 

または混合...

if(a || b) { 
    doAB(); 
    if(b) { 
    doB(); 
    } 
} 

編集：

同僚は、この提案された：私は最初のものを推測してい

int mask = a?1:b?2:0; 
switch(mask) { 
    case 2: 
    doB(); 
    case 1: 
    doAB(); 
} 

Answer 1

完全に異なる考え方です：if/elseの使用を避けてください。

OO言語では、if/elseの代わりに多態性を使用できます。何らかの状態を求めるのではなく、特定のメソッドを呼び出す。あなたは単にオブジェクトのメソッドを呼び出すだけです。オブジェクトの型はメソッドが正しいことを確実にします。

つまり、ある時点で、どの具体的なクラスをインスタンス化する必要があるかを決定します。後で、あなたはもう心配しません。

あなたが知っているのは、の場合でも、（キーワード/言語ビルドintとして）のsmalltalk言語はありません。

もちろん、別のものと入れ替わってはいけません。代わりに、あなたは一歩前に戻って、あなたの問題がどのように再構成されているかを把握することができます。多型を利用する。

ここではかなり良いと言えば、videoこれは実際にどのように見えるのでしょうか？

Answer 2

を二番目の可能性が最も高い誤っている与えられた最高です。シンプルでエラーが起こりにくいフォームを使用します。もしそうであれば

第二の例のように、ほとんどが

if (a) { 
    doAB(); 
} else if (b) { 
    doAB(); 
    doB(); 
} 

をする必要があり、それはBすることはできませんし、その逆

が、その場合には次の2つの条件を必要としません。これは==！b

// a || b is always true 
doAB(); 
if(b) { 
    doB(); 
}