Die zyklomatische Komplexität kann in einer Factory-Methode ohne Verwendung der Reflektion nicht reduziert werden

Question

In meiner Factory-Methode verwende ich die Switch-Anweisung, um konkrete Objekte zu erstellen. Dies führt zu einer sehr hohen zyklomatischen Komplexität. Hier ist ein Beispielcode:

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

Wie kann ich dies umgestalten, um zyklomatische Komplexität zu reduzieren? Wenn ich zum Erstellen von Objekten Objekte oder etwas anderes zum Erstellen von Objekten verwende, ist das besser als die obige Methode?

Answer 1

Es ist möglich, dass Sie ein Dictionary zu verwenden, um die switch Anweisung vollständig zu entfernen:

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