Cで重複したコードを削除する最も良い方法＃

Question

クラスを単純化する最も良い方法は、ほぼ同じような作業を行う2つのメソッドが含まれています。

enum State 
{ 
    Processing, Stoped 
} 

public static void CheckState(State state, Element elem) 
{ 
    if (elem.State == state) 
     //some work 
} 

public static void CheckValue(int value, Element elem) 
{ 
     if (elem.Value == value) 
      //some work 
} 

したがって、Elementクラスのオブジェクトには、異なるタイプの2つのフィールドがあります。重複したコードを削除する最良の方法は何ですか？

Answer 1

条件に関数を渡すことができます。あなたが処理のために必要と思っているので、要素を渡します。

public static void CheckAndProcess(Func<bool> CheckCondition, Element elem) 
{ 
    if (CheckCondition()) 
    { 
     //some work 
    } 
} 

使用法：

CheckAndProcess(()=>(elem.State == state), elem); 
CheckAndProcess(()=>(elem.Value == value), elem); 

Answer 2

あなたはあまりにも、無名関数せずにそれを行うことができます。

public static void CheckElement(Element element, State state = null, int? value = null) 
{ 
    if ((state != null && element.State == state) || (value != null && element.Value == value.Value)) 
    { 
     //some work 
    } 
} 

使用法：

//for state 
CheckElement(element, myState); 
//or 
//for value 
CheckElement(element, value: myValue);