战略模式——正确实施

Question

我是第一次使用这种模式，并且使用 C#。

我只是想检查这是否是正确的实现。
我在 Winform 上有一个按钮，单击该按钮时将以特定格式输出一些数据，通过从下拉框中选择来定义。现在这将来可能会改变，因此我使用策略模式来封装发生的变化。

我有一个“策略接口”，它只公开一个方法：“显示数据（）”。

在我的按钮上单击我使用以下代码：

private void ConfirmButton_Click(object sender, EventArgs e)
    {
        IViewData viewData;

        switch (outputMedia)
        {
            case "Excel":
                viewData = new ExcelOutput(operation, study);
                viewData.DisplayData();
                break;
            case "Spotfire":
                viewData = new SpotfireOutput(operation, study);
                viewData.DisplayData();
                break;
        }


    }

这是使用此模式的可接受的方式吗？显然，如果识别出其他输出媒体，那么我将简单地创建一个新的子类并在 switch 语句上添加一个额外的“case”。

谢谢。

Answer 1

使用策略的正确方法是将 IViewData 对象的创建与其使用分开。创建本身可以由工厂方法处理。然后，您可以在单独的位置使用创建的 IViewData，而完全不知道对象的具体类。

例如

private IViewData CreateViewData()
{
    IViewData viewData;

    switch (outputMedia)
    {
        case "Excel":
            viewData = new ExcelOutput(operation, study);
            break;
        case "Spotfire":
            viewData = new SpotfireOutput(operation, study);
            break;
    }
    return viewData;
}

...

private void ConfirmButton_Click(object sender, EventArgs e)
{
    IViewData viewData = CreateViewData();

    viewData.DisplayData();
}

，现在，您可以通过重构工厂方法来进一步改进解决方案。您可能决定使用 Dictionary 而不是 switch，或者仅创建一次视图数据对象并缓存它们，或者（如您所建议的）用依赖项注入替换它...

Answer 2

首先，您可以将 viewData.DisplayData(); 移出开关。无需对每种情况重复进行。能够以相同的方式为所有这些调用 DisplayData 是您首先引入该接口的重点。

在每个 case 中以相同的方式传递参数看起来也有点重复。因此，也许可以使用 Dictionary> 来代替。但如果不同的类需要不同的构造函数，这就行不通了。

Answer 3

您应该将 switch 语句移至其自己的方法中，最好是在某个控制器类上。这意味着当您需要 IViewData 对象时，不必再次重写开关代码。

private void ConfirmButton_Click(object sender, EventArgs e)
{
    IViewData viewData = ViewDataController.GetViewDataController(outputMedia, operation, study);

   viewData.DisplayData();

}

然后在您的控制器中：

 public class ViewDataController
 {
    public static IViewData GetViewDataController(string outputMedia, string operation, string study)
    {
      IViewData viewData = null;

      switch (outputMedia)
      {
        case "Excel":
            viewData = new ExcelOutput(operation, study);
            break;
        case "Spotfire":
            viewData = new SpotfireOutput(operation, study);
            break;
      }
      return viewData;
     }

这将允许您在整个应用程序中重用相同的代码，如果您需要向 switch 语句添加任何条目，那么您只需在一处执行即可。