使用 Action 字典而不是 switch 语句

Question

我只是在查看我的一些旧代码（有一些空闲时间），我注意到一个相当长的 switch 语句。由于获得了新知识，我将其重构为以下形式：

private Dictionary<string, Action> createView
    {
        get
        {
            return new Dictionary<string, Action>()
            {
                {"Standard", CreateStudySummaryView},
                {"By Group", CreateStudySummaryByGroupView},
                {"By Group/Time", CreateViewGroupByHour}
            };
        }
    }

你会考虑这种好的做法，还是这只是一个多余和不必要的例子？我热衷于确保我学习的新技术不会仅仅为了它而变得聪明，并且它们实际上为代码增加了好处。

谢谢。

Answer 1

长 switch 语句是一种典型的恶臭，并且始终是重构的目标。

此处执行的“标准”步骤是Replace Conditional with Polymorphism。这是 Martin Fowler 的书Refactoring（11 年前于 1999 年出版）中列出的步骤之一。

现在将函数视为对象（例如，使用 Action）非常容易，这可能是一个很好的解决方案。

不，我不认为你是为了聪明而聪明。如果我以后想添加另一个选项，我可以很容易地看到需要做什么。

Answer 2

如果代码在编写后基本上是静态的并且不会进行太多更改，那么我会坚持使用switch。至少从表面上看，您在这里的字典方法非常适合更具动态性-尽管这是基于更多要求的。

至于使用这种方法用代码替换所有地方的开关，我个人在大多数情况下不会这样做。我的诚实意见是，这只是为了聪明，但它仍然很容易维护。在我看来，个人品味高于最佳实践是最大的因素。

另一方面，正如其他人所说，这可能是长 switch 语句的可行解决方案。再说一次，Strategy Pattern 之类的东西也是支持行为变化的好方法。

Answer 3

根据您的应用程序，您可以避免总是构造一个新的字典对象，而是将其声明为类成员，在首次访问时初始化并始终返回相同的实例。但很难说它是否真的符合您的需求。 我的意思是这样的

public class MyClass 
{
   Dictionary<string, Action> dict = null; 

    private Dictionary<string, Action> createView
    {
        get
        {
            if(dict  == null) 
            {
              dict  = new Dictionary<string, Action>()
              {
                {"Standard", CreateStudySummaryView},
                {"By Group", CreateStudySummaryByGroupView},
                {"By Group/Time", CreateViewGroupByHour}
              };
            }

            return dict;
        }
    }

}

编辑

从概念的角度来看，我用字典TryGetValue 替换long swicth/case 是一个很好的解决方案。

希望对你有帮助……

Answer 4

这种方法非常好。

我不仅仅将它与Action 一起使用。它对过滤器和选择器也非常有效。比如：

var filters = new Dictionary<string, Func<MyEntity, bool>>()
{
    // ...
};

var query = entities.Where(filters["X"]);