围绕执行相同检查的多个 if 语句的代码重构 C# 问题

Question

我有一个方法，其中包含大量 if 语句，尽管我不确定如何改进代码，但看起来有点愚蠢。

这是一个例子。这个逻辑位于视图内部，现在位于控制器中，这要好得多，但我是否缺少一些东西，也许是一种设计模式阻止我检查 panelCount <; NumberOfPanelsToShow 并处理 panelCount 每个条件？也许不是，只是觉得丑！

非常感谢

if (model.Train && panelCount < NumberOfPanelsToShow)
{
    panelTypeList.Add(TheType.Train);
    panelCount++;
}
if (model.Car && panelCount < NumberOfPanelsToShow)
{
    panelTypeList.Add(TheType.Car);
    panelCount++;
}
if (model.Hotel && panelCount < NumberOfPanelsToShow)
{
    panelTypeList.Add(TheType.Hotel);
    panelCount++;
}
...

Answer 1

假设 Model.Train、Model.Car、Model.Plane 只是关于“模型”类型的布尔指示符（而不是您创建 Train : Model、Plane : Model 等）

public enum ModelType { Train, Car, Plane };

public class Model { 
    ... 
    public ModelType {get;set;}
    ... 
}

，并且在您的测试中：

if(panelCount < NumberOfPanelsToShow)
switch (Model.ModelType)
{
    case ModelType.Train :
        ...
        break;

    case ModelType.Plane :
...
}

但是，因为 Car、PLane 和火车是不同的，你真的应该有一个基本类型模型，从模型派生汽车，飞机和火车，然后你可以重载方法来处理每种类型

if (panelCount < NumberOfPanelsToShow)
{
    panelCount += AddModel(model);
}

private int AddModel(Plane model)
{ // do plane stuff here and on success return 1 else 0; }

private int AddModel(Train model)
{ // do train stuff here and on success return 1 else 0; }

private int AddModel(Car model)
{ // do car stuff here and on success return 1 else 0; }

Answer 2

通常重构的方法是多态地进行，并负责决定对对象做什么，但由于您的决定是基于同一对象的不同方面，所以我不确定您可以在这里做到这一点。

我很想重构一个方法：

public AddIfSpace(bool thingToTest, TheType typeToAdd, ref currentCount)
{
    if (currentCount<NumberOfPanelsToShow && thingToTest)
    {
     panelTypeList.Add(typeToAdd);
     currentCount++;
    }
}

然后调用它：

AddIfSpace(model.Car,TheType.Car,ref panelCount);
AddIfSpace(model.Train,TheType.train,ref panelCount);
AddIfSpace(model.Hotel,TheType.Hotel,ref panelCount);

根据变量的范围，您可能需要更少或更多的参数，并且如果范围有限，您可以将其作为 panelTypeList 类型的扩展方法，这样您就结束了 这至少可以避免

panelTypeList.AddIfSpace(model.Car,TheType.Car,ref panelCount);
panelTypeList.AddIfSpace(model.Train,TheType.Train,ref panelCount);
panelTypeList.AddIfSpace(model.Hotel,TheType.Hotel,ref panelCount);

代码中的重复逻辑，并且意味着如果您想更改检查的工作方式，您可以在一个位置进行修改，并且如果您很好地命名该方法，应该传达意图并使代码更具可读性。

Answer 3

我将对此进行尝试，做出一些假设。

看起来您有某种“Panel”类，您正在尝试用数据模型对象（汽车、酒店、火车...）中的信息填充该类。相反，为什么不使用带有多态面板类（或 IPanel 接口）的 MVVM 类型模式），以及 CarPanel、HotelPanel、TrainPanel 等的子类...使用 ViewModel 对象，您的视图代码变得更加简单：

var newPanel = PanelFactory.GetPanel(data);  //Returns a CarPanel, HotelPanel, TrainPanel...
panelTypeList.Add(newPanel);
panelCount++;

Answer 4

现在没有 VS 来测试编译代码，但这应该传达逻辑。
您可以使用尽可能多的类型构建数组列表，而无需使用大量 if-else 或 switch 语句重复测试逻辑。

    TheType[] typesWithPanel = new TheType[] { TheType.Train, TheType.Car, TheType.Hotel};
    if (typesWithPanel.Contains(model.ModelType) && panelCount < NumberOfPanelsToShow)
    {
        panelTypeList.Add(model.ModelType);
        panelCount++;
    }

Answer 5

我可能会做的是在模型上有一个枚举类型来切换。

if(panelCount < NumberOfPanelsToShow)
{
  switch(model.modelType)
  {
   case Model.Car:
    // do stuff
  break;
  }
  panelCount++;
}

不过，我们确实需要更多信息来确定如何处理这些案例以及在任何给定时间可以应用多少条件。

更新

鉴于您提供的附加信息，此处的其他一些答案似乎更合适。鉴于您当前的情况，这个答案是不够的。

Answer 6

不确定您如何称呼这种设计模式。但我曾经读过，每当您发现自己为不同类型执行重复的代码块（正如您正在做的那样）时，最好将该功能放入对象本身中。

您可以这样分解：

private const NumberOfPanelsToShow = 4;
private int panelCount = 0;

private void addToPanel(IPanelAddable element, PanelList panelTypeList) {
  if(panelCount < NumberofPanelsToShow) {
    element.addToPanelTypeList(panelTypeList);
    panelCount++;
  }
}

IPanelAddable 可以是定义 addToPanelTypeList 方法（或其他方法）的接口。然后在你原来的方法中：

if (model.Train)
{
    addToPanel(TheType.Train);
}
if (model.Car)
{
    addToPanel(TheType.Car);
}
if (model.Hotel)
{
    addToPanel(TheType.Hotel);
}

实际上你想避免上述情况，简化上面的 /wa 基类等......那么你的 TheType.* 对象显然需要实现“addToPanelTypeList”方法：

panelTypeList.Add(this);

你可以使用接口或继承为了达到同一个目标...

Answer 7

您可以随时查看规范模式。对于布尔逻辑来说非常方便：

public interface ISpecification<T>
{
    public bool IsSatisfiedBy(T candidate);
}