可以替换链式开关/ goto的设计模式？

Question

我有一个代码用于将我的应用程序资源更新为当前应用程序版本 应用程序更新后调用此代码.

int version = 1002;   // current app version

switch(version)
{
   case 1001:
      updateTo1002();
      goto case 1002;

   case 1002:
      updateTo1003();
      goto case 1003;

   case 1003:
      updateTo1004();
      goto case 1004;
      break;

   case 1004:
      updateTo1005();
      break;
}

这里我们通过跳转到指定的case块来调用cascade方法.我想知道 - 在这种情况下,使用go(通常被视为这种不良做法!)是一种好习惯吗？我不想一个接一个地调用方法 - 像这样:

updateTo1002()
{
   // do the job
   updateTo1003();
}
updateTo1003()
{
   // do the job
   updateTo1004();
}

有没有设计模式描述这样的问题？

Answer 1

好吧,如果我们想要"面向对象",为什么不让对象做对话呢？

var updates = availableUpdates.Where(u => u.version > ver).OrderBy(u => u.version);
foreach (var update in updates) {
  update.apply();
}

Answer 2

在示例中,版本正在增加并始终按顺序调用较早的版本.我认为if这里的一系列陈述可能更合适

if (version == 1001 ) { 
  updateTo1002();
}

if (version <= 1002) {
  updateTo1003();
}

if (version <= 1003) {
  updateTo1004(); 
}

if (version <= 1004) {
  updateTo1005();
}

有些人评论说,随着版本数量的增加,这种方法是不可维护的(想想50左右).在这种情况下,这是一个更容易维护的版本

private List<Tuple<int, Action>> m_upgradeList;

public Program()
{
    m_upgradeList = new List<Tuple<int, Action>> {
        Tuple.Create(1001, new Action(updateTo1002)),
        Tuple.Create(1002, new Action(updateTo1003)),
        Tuple.Create(1003, new Action(updateTo1004)),
        Tuple.Create(1004, new Action(updateTo1005)),
    };
}

public void Upgrade(int version)
{
    foreach (var tuple in m_upgradeList)
    {
        if (version <= tuple.Item1)
        {
            tuple.Item2();
        }
    }
}

Answer 3

我讨厌不提供支持信息的空白语句,但goto相当普遍地被淘汰(有充分理由)并且有更好的方法来实现相同的结果.您可以尝试使用"责任链"模式来实现相同的结果,而不需要goto实现可以变成的"spaghetti-ish"goo.

责任链模式.

Answer 4

goto总是被认为是不好的做法.如果你使用goto,通常很难阅读代码,你总是可以用不同的方式编写代码.

例如,您可以使用链表创建一系列方法和一些处理链的处理器类.(请参阅pst的答案以获得良好的例子.)它更加面向对象和可维护.或者如果你必须再添加一个方法调用beetween 1003和case 1004呢？

当然看到这个问题.