リフレクトを使用せずにファクトリメソッドの循環的複雑さを減らすことができません

Question

私のファクトリメソッドでは、Switchステートメントを使用して具体的なオブジェクトを作成します。これは非常に高いサイクロマティックな複雑さをもたらす。ここにサンプルコードがあります：

私はどのようにこれをリファクタリングして周期的な複雑さを減らすことができますか？オブジェクトを作成するためにリフレクションを使用する場合は、上記の方法よりも優れていますか？

Answer 1

あなたは完全にswitchステートメントを削除するDictionaryを使用することが可能です：デフォルトの場合

class MyClass 
{ 
    private static Dictionary<string, Func<UnitDescriptor>> dict = new Dictionary<string, Func<UnitDescriptor>>(); 

    static MyClass() 
    { 
     dict.Add(UnitCode.DEG_C,() => new UnitDescriptorDegC()); 
     dict.Add(UnitCode.DEG_F,() => new UnitDescriptorDegF()); 
     // Other mappings... 
    } 

    private static UnitDescriptor createUnitDescriptor(string code) 
    { 
     Func<UnitDescriptor> value; 
     if (dict.TryGetValue(code, out value)) 
     { 
      return value(); 
     } 

     throw new SystemException(string.Format("unknown code: {0}", code)); 
    } 
}