C# 出厂默认返回

Question

我创建了一个简单的工厂，在扩展它以涵盖其他实现之前，我想知道从开关/默认值返回的正确/推荐行为是什么。

在下面的例子中，我使用枚举(SerialisationTypes)来确定所需的具体实现，默认情况下，我计划通过无参数方法返回 JSON 序列化器实现，但这是否是开关默认值的正确行为语句还是应该抛出异常？

namespace Helper.Core.Serialisation
{
    internal class SerialisationFactory
    {
        internal ISerialiser Create()
        {
            return Create(SerialisationTypes.JsonSerialiser);
        }

        internal ISerialiser Create(SerialisationTypes type)
        {
            switch (type)
            {
                case SerialisationTypes.JsonSerialiser:
                    return new JSonSerialiser();
                default:
                    return new JSonSerialiser();
            }
        }
    }
}

Answer 1

您最好抛出异常，因为用户正在请求您未实现的内容。

此外，在您的代码中，如果枚举具有其他值(例如 XmlSerialiser、ProtbufSerializer 等)，那么您将始终返回一个 JSonSerialiser，这将导致难以发现错误。最好抛出异常，然后修复代码。