c# – 无法在不使用反射的情况下降低Factory方法中的圈复杂度

在我的工厂方法中,我使用Switch语句来创建具体对象.这导致非常高的圈复杂度.这是一个示例代码：

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

我怎样才能重构这个以减少圈复杂度？如果我使用反射创建对象或其他东西来构建对象,它比上面的方法更好吗？

解决方法

您可以使用Dictionary完全删除switch语句：

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