重构 c#：当要更新的属性在每种情况下不同时如何删除重复的代码

Question

在对原型进行了一段时间的修改之后，我最终得到了许多方法，这些方法可以更新对象上的布尔标志，然后更新接口并根据新值进行一些处理。这些几乎都是相同的 - 但它们更新的值是不同的

，例如 - 想象我们有一堆彩色框要更新 - 我可能有一些看起来像这样的方法：

        protected void SetBlueBoxVisibility(bool blueBoxVisibility)
    {
        Project project = LoadProject();
        project.ShowBlueBox = blueBoxVisibility
        ReDrawSomeThings();
        CalcualteSomeStuff();
        Project.UpdateBoxStatus();
        SaveProject(project);
        ShowBlueBoxPanel(blueBoxVisibility);
        RaiseStatusUpdated();
    }

    protected void SetRedBoxVisibility(bool redBoxVisibility)
    {
        Project project = LoadProject();
        project.ShowRedBox = redBoxVisibility
        ReDrawSomeThings();
        CalcualteSomeStuff();
        Project.UpdateBoxStatus();
        SaveProject(project);
        ShowRedBoxPanel(redBoxVisibility);
        RaiseStatusUpdated();

    }       

现在，显然 - 大部分内容都是重复的 -当我要改变任何事情时，这是一种痛苦。特别是如果我最终得到二十种不同的盒子颜色而不是两种！

我在想，必须有一种方法可以去除更改的代码，并以更通用的方法收集相同的内容 - 但我很难弄清楚如何做到这一点。

我听说过关闭 - 但我还没有充分了解它们，不知道它们是否会在这里提供帮助。

我认为以下内容可能在正确的行 - 但我不知道如何告诉通用方法要操作哪个属性 - [要更新的项目变量]

        protected void SetRedBoxVisibility(bool redBoxVisibility)
    {
        SetGenericBoxVisibility([Project Variable To Update],redBoxVisibility)
        ShowRedBoxPanel(redBoxVisibility);
        RaiseStatusUpdated();   
    }

    protected void SetBlueBoxVisibility(bool blueBoxVisibility)
    {
        SetGenericBoxVisibility([Project Variable To Update],blueBoxVisibility)
        ShowBlueBoxPanel(blueBoxVisibility);
        RaiseStatusUpdated();   
    }

    protected void SetGenericBoxVisibility([Project Variable To Update], boxVisibility)
    {
        Project project = LoadProject();
        project.**[Project Variable To Update]** = boxVisibility
        ReDrawSomeThings();
        CalcualteSomeStuff();
        Project.UpdateBoxStatus();
        SaveProject(project);
    }

任何指针如何处理这种事情会很有用:)

Answer 1

好吧，你可以像这样提取它：

protected void SetGenericBoxVisibility(Action<Project> propertySetter,
                                       Action<bool> panelShower,
                                       bool boxVisibility)
{
    Project project = LoadProject();
    propertySetter(project);
    ReDrawSomeThings();
    CalculateSomeStuff();
    Project.UpdateBoxStatus();
    SaveProject(project);
    panelShower();
    RaiseStatusUpdated();
}

那么：

protected void SetBlueBoxVisibility(bool blueBoxVisibility)
{
    SetGenericBoxVisibility(project => project.ShowBlueBox = blueBoxVisibility,
                            () => ShowBlueBoxPanel(blueBoxVisibility));
}

protected void SetRedBoxVisibility(bool redBoxVisibility)
{
    SetGenericBoxVisibility(project => project.ShowRedBox = redBoxVisibility,
                            () => ShowRedBoxPanel(redBoxVisibility));
}

不可否认，这不是非常好......

Answer 2

我认为您可能遇到更大的问题 - 每个框一种更新方法根本不合适。您拥有 SetGenericBoxVisibility，但随后通过继续使用 Set*BoxVisibility 方法来撤消所有好的工作。我不知道你正在使用哪种技术 - 如果它是 WPF 查看 MVVM 那么你可以简单地更新你的 ViewModels。如果是 WinForms，您可能应该创建某种字典 - Dictionary; _boxLookup 其中 BoxType 是您定义的类型的枚举。然后，要设置框可见性，请执行 _boxLookup[BoxType.Red].Property = value; 操作，或者您可以使用采用 BoxType 参数来操作框的方法。

然而，更多的上下文将非常有用，因为即使该解决方案也不是理想的。移动一些其他代码或深入了解您正在使用多个盒子解决的问题的大局应该会带来更多的“啊哈”时刻。

Answer 3

我认为要真正重构它，您需要制作一个 IBox 界面。基本上，该接口充当契约，定义所有 Box 对象至少必须具有哪些方法和属性。

interface IBox {
 //your generic properties and method stubs
 Bool visibility;
}

现在，为每个“盒子”实现接口

class blueBox : IBox
{
 //here you will have your concrete implementations of the above methods and properties
public Bool visibility   {get; set;} // this doesn't make sense with auto getter setters. you would need to write your bluebox specific getter and setters

}

class redBox : IBox
{
//more concrete implementation

}


public myMethod_To_Do_Stuff(IBox myBox) { // see how I am passing the interface not the conrete classes

myBox.visibility = true;

}