重构帮助-策略模式

Question

这里的目标是更新 UI。 我通常在客户端上执行此操作，但是此应用程序使用隐藏的代码。 无论如何，我的问题是我正在尝试清理这些 if else 语句，并且我认为策略模式可能是合适的。 我不需要为我做所有的事情，但如果你能给我一些指导，让我继续前进。 我是否要先创建一个接口，然后每个策略都实现该接口？ 泛型在这里有用吗？ 接口中应该包含哪些类型的方法？ 任何能让我继续前进的事情都将非常感激。

if (someObject.Status == 'A') {
                    btnRecordCall.Enabled = false;
                    btnAddMailOrStatusAction.Enabled = false;
                    btnPayments.Enabled = false;
                    btnAddressMaint.Enabled = false;
                    btnFilter.Enabled = false;
                    btnAddCoverage.Enabled = false;
                    btnPolicyForms.Enabled = false;
                    lblIsArchived.Text = "********** THIS CLAIM HAS BEEN ARCHIVED **********";
                } else if (someObject.Status == 'D') {
                    btnRecordCall.Enabled = false;
                    btnAddMailOrStatusAction.Enabled = false;
                    btnPayments.Enabled = false;
                    btnAddressMaint.Enabled = false;
                    btnFilter.Enabled = false;
                    btnAddCoverage.Enabled = false;
                    btnPolicyForms.Enabled = false;
                    lblIsArchived.Text = "- De-archive Request Pending";
                } else {
                    btnRecordCall.Enabled = true;
                    btnAddMailOrStatusAction.Enabled = true;
                    btnPayments.Enabled = true;
                    btnAddressMaint.Enabled = true;
                    btnFilter.Enabled = true;
                    btnAddCoverage.Enabled = true;
                    btnPolicyForms.Enabled = true;
                    lblIsArchived.Text = "";
                }

谢谢， 〜ck

Answer 1

如果无论状态如何，所有按钮都获得相同的启用（如您的示例所示），那么我会将其隐藏在按钮列表中，然后迭代它们。 然后你的策略参数可以归结为一个布尔值和一个字符串——无论如何，做到这一点都很容易。 如果它的复杂性更大，那么实现设置控件的接口的不同类将是有序的，可能具有保存整体逻辑的抽象超类。

Answer 2

首先，如果您要将策略模式与接口一起使用，您可能需要这样的接口。

public interface IStrategyUI
{
   void Execute();
}

然后是你的逻辑的几个类

public class StatusAStrategy : IStrategyUI
{
  public void Execute()
   {
      //implement
   }
}

然后你的 if 语句在技术上可以是一个工厂。

IStrategyUI logic;

switch(someObject.Status)
{
    case 'A':   logic = new StatusAStrategy();
            //etc etc

}

logic.Execute();

或者您可以将此对象作为接口返回，并将 switch 语句封装在另一个方法或工厂类中。

这样你就可以像这样使用它。

IStrategyUI logic = StrategyFactory.GetStrategy(status);
logic.Execute();

由你决定。

另外，我在这里没有使用收容类，但如果您愿意，也可以这样做。

您也可以与代表一起执行此操作。

遏制类

 class StrategyHolder
    {
        public Action Strategy { get; set; }

        public void Execute()
        {
            if(this.Strategy != null)
                 this.Strategy();
        }
    }

不同的策略方法。

void EnableStatusA()
{
   //do everything for status A
}

Switch 语句

StrategyHolder logic = new StrategyHolder();

switch(someObject.Status)
    {
        case 'A':   logic.Strategy = EnableStatusA;
                //etc etc

    }

return logic;

从外部调用此语句。

StrategyHolder logic = StrategyFactory.GetStrategy(status);
logic.Execute();

Answer 3

我认为您正在寻找的是 State 模式。 这与策略模式类似，只是每个状态对象在创建时通常会被赋予对上下文对象的引用（在您的情况下，这是表单）。 这允许不同的状态对上下文对象执行操作以响应事件。

实现状态模式时，通常最好让每个单独的状态继承抽象基类。 然后，基类可以在虚拟方法中实现默认操作，然后您只需重写每个状态不同的操作。

public interface IFormState
{
    void EnableDisableControls();
}

public class DefaultState : IFormState
{
    private MyForm context;

    public DefaultState(MyForm context)
    {
        this.context = context;
    }

    protected MyForm Context
    {
        get
        {
            return this.context;
        }
    }

    public virtual void EnableDisableControls()
    {
        this.context.btnRecordCall.Enabled = true;
        this.context.btnAddMailOrStatusAction.Enabled = true;
        this.context.btnPayments.Enabled = true;
        this.context.btnAddressMaint.Enabled = true;
        this.context.btnFilter.Enabled = true;
        this.context.btnAddCoverage.Enabled = true;
        this.context.btnPolicyForms.Enabled = true;
        this.context.lblIsArchived.Text = "";
    }
}

public class StateA : DefaultState
{
    public StateA(MyForm context)
        : base(context)
    {
    }

    public override void EnableDisableControls()
    {
        base.EnableDisableControls();

        this.Context.lblIsArchived.Text = "********** THIS CLAIM HAS BEEN ARCHIVED **********";
        // etc...
    }
}