これはメソッドを再利用/共有する良い方法ですか？

Question

メソッドコール、たとえばClassA.search（a、b、flag）が3つのコントローラで使用されているこのコードが発生しました。これはメソッドの簡略化されたバージョンです。

public List<Result> search(Object a, Object b, boolean flag) { 
    //do some code logic here, common to the 3 controllers 
    //at the middle there is: 
    if (flag) { 
     //code that affects 2 Controllers 
    } else { 
     //code affects only 1 
    } 
    //some more common code 
    //some more code with the flag if else 
} 

コードは再利用されるので、これは良い考えですか？または、コードを再利用できるが、メソッド呼び出し元（クライアント）コードのカスタマイズ（この3つのメソッドを3つの異なるメソッドに分割しても、共通コードのリファクタリングされたメソッドを宣言できるように）にこのフラグを導入しない方が良い方法はありますか？

Answer 1

まず、抽出物は、関数での行をコメント：

public void search(Object a, Object b, boolean flag) 
{ 
    commonToThree(); 
    if (flag) 
    { 
     affectTwoControllers(); 
    } 
    else 
    { 
     affectsOnlyOne(); 
    } 
    alsoCommon(); 
} 

は今、コードのにおいですflagブール引数、取り除く：

public void searchWithTrueFlag(Object a, Object b) { 
    commonToThree(); 
    affectTwoControllers(); 
    alsoCommon(); 
} 

public void searchWithFalseFlag(Object a, Object b) { 
    commonToThree(); 
    affectsOnlyOne(); 
    alsoCommon(); 
} 

Answer 2

いいですがそれほど優れていません。 1つのブール値は理にかなっていますが、それ以上の追加を開始すると正しい方向に進まないことになります。

それは常に可能ではないのですが、一般的に行うために、より良いコードを生成：

functionOne: 
    sharedCodeOne() 
    specificCode 
    sharedCodeTwo() 

functionTwo: 
    sharedCodeOne() 
    specificCode 
    sharedCodeTwo() 

いつものように、一般的な主張をするのは難しい：これは常に明らかに実用的/ことはできません。

Answer 3

これは比較的簡単な方法です。代替案がありますが、より複雑になる可能性があります。 （訪問者を渡すか、コントローラーのメソッドを呼び出してそのコントローラーのために何をするのかなど）

このアプローチは、コードの3つのセクション間でローカル変数を共有する場合に最も適しています。

Answer 4

私は答えにしようとは異なるアプローチを取ると思います一般的な方法でこの質問：

主な目標はクリーンコードです。正確にはこれはもちろん、特定のケースに依存します。しかし、コピーと貼り付けのコードをいくつかの場所に置くことは悪いことです。変更する必要がある場合はいくつかの場所を変更する必要があるからです。同様に、複数のパーツが使用している共通部分を抽出しようとするのは悪いすべての状況でそれら。

あなたのコードを読んでいる人（または今から数週間後にこれをやる必要がある人）は、いつも起こっていることをできるだけ早く理解しなければならないと想像してください。したがって、いくつかの行をコピーしたり、共通の方法で抽出したりする方が良い場合は、特殊なケースに依存します。

フラグは本質的に悪いものではありませんが、実際にはJavaメソッドで過度に使用されるべきものではありません。しばしば、このような状況は、メソッドのオーバーロードと他のメソッドのオーバーロードによってうまく解決することができます。そのため、共通のケースは1つに、特殊な追加はもう1つで行われます。あるいは、いくつかのサブメソッドを持ち、いくつかの呼び出しでメソッドを構成することができますが、あまりにも多くのパラメーターを渡す必要がない場合にのみ意味があります。

1つのルール（完全に主観的ですが、多くのプロジェクトから学んだレッスンに基づいています）私はあなたに与えることができます：具体的な実装を好む一般的なアプローチ。コードが増え、賢明に見えないかもしれませんが、理解し、拡張し、デバッグするのがはるかに簡単です。