什么更好？枚举或类型检查？

Question

假设我想将一些xml解析为强类型类.当我得到xml时,我不知道它应该是A型还是B型,直到我打开它并看一看.我可以看看,并返回这样的枚举:

BaseType x = null;
TypeInfoEnum typeInfo = BaseType.GetTypeInfo(xml);

if(typeInfo == TypeInforEnum.TypeA)
{
    x = BaseType.ParseXmlToTypeA(xml);

    // do other work on Type A
}

else if(typeInfo == TypeInfoEnum.TypeB)
{
    x = BaseType.ParseXmlToTypeB(xml);

    // do other work on Type B
}

或者我可以在一种方法中处理解析并检查类型:

BaseType x = BaseType.ParseXml(xml);

if(x.GetType() == typeof(TypeA))
{
    // do work on Type A
}
else if(x.GetType() == typeof(TypeB))
{
    // do work on Type B
}

只是想从你喜欢的设计立场获得别人的想法.现在,细节不是很重要.我只是根据xml中的内容从单个XML源创建两种不同的类型.没什么复杂的.

更新:

谢谢你到目前为止的答案.这里的类型并不重要,但作为一个例子,类层次结构可能如下所示:

class BaseType
{
    public string CommonData { get; set; }
}

class TypeA : BaseType
{
    public string TypeASpecificData { get; set; }
}

class TypeB : BaseType
{
    public string TypeBSpecificData { get; set; }
}

由于此功能将被转换为其他人将使用的程序集,我喜欢使用Enum的第一个选项,因为让API的用户检查某些内容的类型似乎很尴尬,即使用Enum似乎更加语义彻底.

Answer 1

在第一个选项中,您基本上复制了信息(类型+枚举),没有明显的好处.因此,考虑到这两个选项,我会选择第二个选项,虽然我更喜欢更惯用is而不是GetType比较:

BaseType x = BaseType.ParseXml(xml);

if(x is TypeA)
{
    // do work on Type A
}
else if(x is TypeB)
{
    // do work on Type B
}

但是,您可以考虑第三种选择:

BaseType x = BaseType.ParseXml(xml);
x.DoWork();

与DoWork作为碱基类型的一个抽象方法,其在类型A和类型B覆盖:

public abstract class BaseType
{
    public abstract void DoWork();
}
public class TypeA : BaseType
{
    public override void DoWork() {
        // do work on Type A
    }
}
public class TypeB : BaseType
{
    public override void DoWork() {
        // do work on Type B
    }
}