C#：枚举反模式

Question

Closed. This question needs to be more focused. It is not currently accepting answers.

---

想要改进这个问题？通过编辑这篇文章来更新问题，使其仅关注一个问题。

5 年前已关闭。

Answer 1

我认为枚举非常有用。我已经为 Enum 编写了一些扩展，这些扩展为其使用增加了更多价值

首先，有 Description 扩展方法

public static class EnumExtensions
{
    public static string Description(this Enum value)
    {
        var entries = value.ToString().Split(ENUM_SEPERATOR_CHARACTER);
        var description = new string[entries.Length];
        for (var i = 0; i < entries.Length; i++)
        {
            var fieldInfo = value.GetType().GetField(entries[i].Trim());
            var attributes = (DescriptionAttribute[])fieldInfo.GetCustomAttributes(typeof(DescriptionAttribute), false);
            description[i] = (attributes.Length > 0) ? attributes[0].Description : entries[i].Trim();
        }
        return String.Join(", ", description);
    }
    private const char ENUM_SEPERATOR_CHARACTER = ',';
}

这将允许我像这样定义 enum：

 public enum MeasurementUnitType
 {
    [Description("px")]
    Pixels = 0,
    [Description("em")]
    Em = 1,
    [Description("%")]
    Percent = 2,
    [Description("pt")]
    Points = 3
 }

并通过执行以下操作获取标签： var myLabel =矩形.widthunit.Description()（消除了对 switch 语句的任何需要）。

顺便说一句，如果rectangle.widthunit =MeasurementUnitType.Pixels，这将返回“px”，或者如果rectangle.widthunit =MeasurementUnitType.Pixels | 它将返回“px，em”。测量单位类型.Em。

然后，有一个

    public static IEnumerable<int> GetIntBasedEnumMembers(Type @enum)
    {
        foreach (FieldInfo fi in @enum.GetFields(BindingFlags.Public | BindingFlags.Static))
            yield return (int)fi.GetRawConstantValue();
    }

允许我遍历任何具有基于 int 的值的枚举并返回 int 值本身。

我发现这些在一个已经有用的概念中非常有用。

Answer 2

这完全取决于您尝试对枚举执行什么操作。

1. 如果您试图阻止开发人员将幻数传递到您的操作中，并且您希望保持数据库的数据引用完整性完整，那么，是的！使用 T4-Templates（使用您的 ORM）转到您的MeasurementUnitTypes 表并生成一个枚举，其中 ID、名称和描述列与枚举' int、Enum_Name 和描述属性匹配（将附加字段\数据添加到枚举 @danijels 的好方法）正如上面所建议的。如果您将新的测量类型添加到MeasurementUnitTypes 表中，您只需右键单击并运行T4 模板，就会为表中添加的新行生成枚举代码。我不喜欢应用程序中的硬编码数据不链接到我的数据库，因此提到了 T4 模板方法。否则它是不可扩展的......如果其他外部系统想要检索我们系统中使用的测量标准，那么它是硬编码在系统中的，您无法通过服务将其公开给客户端。就这样离开了。

2. 如果目的与数据无关，并且您有一些逻辑分配给特定的枚举，那么不！这违反了 SOLID（开闭原则），因为您会在应用程序中的某个地方应用一个开关或一堆 If 来操作每个枚举的逻辑，而且如果您做得非常糟糕，这些开关或 If 就会遍布整个节目......祝你好运，添加一个新的枚举...所以它不开放扩展，也不开放修改，因为您需要根据 SOLID 原则修改现有代码。

   如果您的选择是 2，那么我建议使用 @danijels 评论中的示例将您的枚举替换为以下内容：

   公共接口 IMeasurementUnitType
   {
       int ID { 获取； }
   
       字符串描述{获取; }
   
       // 只是添加来模拟系统中所需的操作
       字符串 GetPrintMessage(int 大小);
   }
   

上面的代码定义了每个测量应遵守的接口（代码契约）。现在让我们定义百分比和像素测量：

    public class PixelsMeasurementUnitType : IMeasurementUnitType
    {
        public int ID => 1;

        public string Description => "Pixel";

        public string GetPrintMessage(int size)
        {
            return $"This is a {Description} Measurement that is equal to {size} pixels of the total screen size";
        }
    }

    public class PercentMeasurementUnitType : IMeasurementUnitType
    {
        public int ID => 2;

        public string Description => "Persentage";

        public string GetPrintMessage(int size)
        {
            return $"This is a {Description} Measurement that is equal to {size} persent of total screen size (100)";
        }
    }

所以我们定义了两种类型，我们将在代码中使用它们，如下所示：

    var listOfMeasurmentTypes = AppDomain.CurrentDomain.GetAssemblies()
                        .SelectMany(s => s.GetTypes())
                        .Where(p => typeof(IMeasurementUnitType).IsAssignableFrom(p) 
                                    && !p.IsInterface)
                        .ToList();

这里我们获取扩展 IMeasurementUnitType 接口的所有类型，而不是接口本身。现在我们可以使用 Activator 创建类的实例来填充我们的 UI 控件：

    public IEnumerable<IMeasurementUnitType> GetInstantiatedClassesFromTypes(List<Type> types)
    {
        foreach (var type in types)
        {
            yield return (IMeasurementUnitType)Activator.CreateInstance(type);
        }
    }

您可以将上面的代码更改为任何类型的通用代码，现在生活发生了，客户端给出了一个名为 Point 的新测量单位类型作为新要求，我不需要更改任何代码，只需添加新类型（扩展代码而不是修改）。新类型将在应用程序中自动选取。

    public class PointMeasurementUnitType : IMeasurementUnitType
    {
        public int ID => 3;

        public string Description => "Point";

        public string GetPrintMessage(int size)
        {
            return $"This is a {Description} Measurement that is equal to {size} points of total screen size";
        }
    }

一个好主意是在启动应用程序时缓存您的类型以获得性能优势，或者尝试使用您选择的 DI 容器。

另外，有人可能会说，在您的应用程序中的某个地方，您需要区分类型，我同意，但是您希望将苹果与苹果放在一起。因此，请尽可能尝试应用与此类型相同的原则。如果此类型用于某种图形处理器（例如）类，则拥有 IGraphicsProcessor 并拥有区分这些类型的具体类，例如 PersentageAndPixelGraphicsProcessor（从 IGraphicsProcessor 扩展），或者如果它仅区分一种类型，则将其称为 PersentageGraphicsProcessor。

对于巨大的 SA 感到抱歉，但我真的很喜欢枚举，但是我觉得当您尝试使用枚举来分离逻辑时，这是一种强烈的反模式。

欢迎评论，

Answer 3

这不是一个答案，只是为枚举反模式列表做出了贡献。

今天早上进行代码审查时，我遇到了类似以下的情况，都在同一个类中。

两种情况：

1. 喝酒前
2. 喝酒后

..

    public enum ListEnum 
    { 
        CategoryOne,
        CategoryTwo,
        CategoryThree,
        CategoryFour
    }


    public class UIELementType
    {
        public const string FactoryDomain = "FactoryDomain";
        public const string Attributes = "Attributes";
    }

Answer 4

在非反模式中使用枚举。在一些有关重构的书籍中，此代码用于演示如何用多态性替换它。当你在代码中过度使用枚举时也没关系。

Answer 5

我认为有两个 switch 语句是非面向对象设计的症状正如本答案中进一步解释的那样。