在我的工厂方法中,我使用 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));
}
}
关于c# - 不使用反射无法降低工厂方法中的圈复杂度,我们在Stack Overflow上找到一个类似的问题: https://stackoverflow.com/questions/18398377/