Nie można zredukować złożoności cyklicznej w metodzie Factory bez użycia odbicia

Question

W mojej metodzie fabrycznej używam instrukcji Switch do tworzenia konkretnych obiektów. Powoduje to bardzo wysoką złożoność cykliczną. Oto przykładowy kod:

private static UnitDescriptor createUnitDescriptor(string code) 
{ 
    switch (code) 
    { 
     case UnitCode.DEG_C: 
      return new UnitDescriptorDegC(); 

     case UnitCode.DEG_F: 
      return new UnitDescriptorDegF(); 

     : 
     : 
     default: 
      throw new SystemException(string.format("unknown code: {o}", code); 
     } 
    } 

Jak mogę to zmienić, aby zmniejszyć złożoność cykliczności? Jeśli używam odbicia do tworzenia obiektów lub czegoś innego do budowania obiektów to czy jest to lepsze niż powyższa metoda?

Answer 1

Jest to możliwe, aby korzystać z Dictionary usunąć oświadczenie switch całości:

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)); 
    } 
}