这是在 C# 中有效使用委托吗？

Question

我有一个 while 循环需要调用两种方法之一；调用哪个方法取决于被迭代对象的属性。我没有在此属性上执行 if / switch 语句，而是使用委托在 while 循环外执行 switch 语句。

这用代码更容易解​​释。

public void Plan()
{
   foreach (var itemHeader in estimate.ItemHeaders)
   {
      var scheduleMethod = new Schedule(ScheduleWall);

      switch (itemHeader.Type)
      {
         case TypeEnum.Stair: { scheduleMethod = new Schedule(ScheduleWall); break; }
         case TypeEnum.Wall: { scheduleMethod = new Schedule(ScheduleStair); break; }
      }

      foreach (var item in itemHeader.EstimateItems)
      {
         // this loop will run many, many times
         scheduleMethod(item);
      }
   }
}

private delegate void Schedule(EstimateItem item);

private void ScheduleWall(EstimateItem item)
{
   // do stuff..
}

private void ScheduleStair(EstimateItem item)
{
   // do stuff..
}

在循环内通过itemHeader.EstimateItems 调用委托的替代方法是在该循环内执行switch 语句。它只比较两个枚举，但由于该循环将运行多次，因此在循环之前进行一次比较似乎更有效。

我是否引入了不必要的复杂性？这是使用代表的不好的做法吗？

Answer 1

在这种情况下最有效的方法是没有委托：

public void Plan()
{
   foreach (var itemHeader in estimate.ItemHeaders)
   {
      if(itemHeader.Type == TypeEnum.Wall)
         foreach (var item in itemHeader.EstimateItems)
            ScheduleStair(item);
      else
         foreach (var item in itemHeader.EstimateItems)            
            ScheduleWall(item);
   }
}

这样你就直接调用了方法，甚至可以内联。