如何重构具有多个切换案例的应用程序？

我有一个应用程序，它使用整数作为输入，并基于输入调用不同类的静态方法。每次添加新数字时，我们需要添加另一种情况，并调用不同类的不同静态方法。交换机中现在有50个案例，每当我需要添加另一个案例时，我都会发抖。有一个更好的方法吗。

我做了一些思考，想到了这个主意。我使用策略模式。我没有交换条件，而是有一个策略对象映射，键为输入整数。调用方法后，它将查找对象并为该对象调用通用方法。这样，我可以避免使用switch case结构。

你怎么看？

交换机中现在有50个案例，每当我需要添加另一个案例时，我都会发抖。

我喜欢多态。我喜欢SOLID。我喜欢纯粹的面向对象编程。我讨厌看到这些代表不好的表现，因为它们被教条地应用了。

您没有很好的理由来重构策略。重构有一个名字。这称为用多态替换条件。

我从c2.com找到了一些有关您的建议：

实际上，只有经常重复相同或非常相似的条件测试才有意义。对于简单的，很少重复的测试，用多个类定义的冗长程度替换简单的条件语句，并可能将所有这些条件从实际上需要条件操作的代码移开，这将导致教科书示例中发生代码混淆。相对于教条式的纯洁性，它更喜欢清晰度。-丹·穆勒

您有一个带有50个盒子的开关，您的替代选择是生产50个对象。哦，还有50行的对象构造代码。这不是进步。为什么不？因为此重构没有任何作用将数字从50减少。当您需要在其他地方的同一输入上创建另一个switch语句时，可以使用此重构。那时，这种重构非常有用，因为它可以将100重新变成50。

只要您像唯一的开关一样指代“开关”，我就不建议这样做。现在从重构中获得的唯一好处是，它减少了一些愚蠢的人复制和粘贴50格开关的机会。

我建议对这50种情况进行仔细研究，以找出可以排除的共同点。我的意思是50？真？您确定需要很多情况吗？您可能在这里尝试做很多事情。

单独的策略对象图，在代码的某些功能中初始化，其中有几行代码看起来像

     myMap.Add(1,new Strategy1());
     myMap.Add(2,new Strategy2());
     myMap.Add(3,new Strategy3());

要求您和您的同事以更统一的方式实现在单独的类中调用的功能/策略（因为您的策略对象都必须实现相同的接口）。这样的代码通常比

     case 1:
          MyClass1.Doit1(someParameters);
          break;
     case 2:
          MyClass2.Doit2(someParameters);
          break;
     case 3:
          MyClass3.Doit3(someParameters);
          break;

但是，无论何时需要添加新的数字，它仍然不会使您摆脱编辑此代码文件的负担。这种方法的真正好处是不同的：

• 现在，地图的初始化与实际调用与特定编号相关联的函数的调度代码分离了，而后者不再包含这50个重复，它看起来像myMap[number].DoIt(someParameters)。因此，无论何时有新号码到达，都无需触摸此调度代码，并且可以根据“打开-关闭”原理来实现。而且，当您获得需要扩展调度代码本身的要求时，您将不必再更改50个位置，而只需更改一个。

• 映射的内容是在运行时确定的（而switch构造的内容是在编译时确定的），因此这使您有机会使初始化逻辑更加灵活或可扩展。

是的，有一些好处，这无疑是迈向更多SOLID代码的一步。但是，如果重构值得回报，那么您或您的团队将必须自行决定。如果您不希望更改派发代码，不更改初始化逻辑并且的可读性switch不是真正的问题，那么现在的重构可能就不再那么重要了。

我强烈支持@DocBrown的答案中概述的策略。

我将建议对答案进行改进。

来电

 myMap.Add(1,new Strategy1());
 myMap.Add(2,new Strategy2());
 myMap.Add(3,new Strategy3());

可以分发。您不必回到同一文件即可添加另一个策略，该策略甚至更好地遵循了“开放-封闭”原则。

假设您Strategy1在文件Strategy1.cpp中实现。您可以在其中包含以下代码块。

namespace Strategy1_Impl
{
   struct Initializer
   {
      Initializer()
      {
         getMap().Add(1, new Strategy1());
      }
   };
}
using namespace Strategy1_Impl;

static Initializer initializer;

您可以在每个StategyN.cpp文件中重复相同的代码。如您所见，这将是很多重复的代码。为了减少代码重复，您可以使用模板，该模板可以放在所有Strategy类都可以访问的文件中。

namespace StrategyHelper
{
   template <int N, typename StrategyType> struct Initializer
   {
      Initializer()
      {
         getMap().Add(N, new StrategyType());
      }
   };
}

之后，您在Strategy1.cpp中唯一需要使用的是：

static StrategyHelper::Initializer<1, Strategy1> initializer;

StrategyN.cpp中的相应行是：

static StrategyHelper::Initializer<N, StrategyN> initializer;

通过为具体的Strategy类使用类模板，可以将模板的使用提高到另一个层次。

class Strategy { ... };

template <int N> class ConcreteStrategy;

然后，而不是Strategy1使用ConcreteStrategy<1>。

template <> class ConcreteStrategy<1> : public Strategy { ... };

将helper类更改为将Strategys 注册到：

namespace StrategyHelper
{
   template <int N> struct Initializer
   {
      Initializer()
      {
         getMap().Add(N, new ConcreteStrategy<N>());
      }
   };
}

将Strateg1.cpp中的代码更改为：

static StrategyHelper::Initializer<1> initializer;

将StrategN.cpp中的代码更改为：

static StrategyHelper::Initializer<N> initializer;