如何处理“开关/外壳”当软件版本添加新参数时

Question

当有新的软件版本发布并且它向 switch case 添加另一个索引时如何处理。在这种情况下，索引代表一个参数。例如，

Rel1: i = 1-5, 7（不包括 6）

Rel2: i = 1-7

for (int i = 1; i<=7;i++)
{
  switch (i)
  {
    case 1: /*process data*/ break;
    case 2: /*process data*/ break;
    case 3: /*process data*/ break;
    case 4: /*process data*/ break;
    case 5: /*process data*/ break;
    // case 6: // REL 2
    case 7: /*process data*/ break;
    default: break;
  }
}

我实际上可以通过在这些情况之间添加 if 语句来检查吗？还有更好的主意吗？

Answer 1

在生产代码中，如果我有这样的意外结果，我经常尝试捕获并记录它。
抛出异常可能是可以的，具体取决于它的处理方式。为了面向未来，设计这样的东西来优雅地处理新值通常是好的。

Answer 2

不要使用“for”循环，而是使用带有版本号生成器函数的“while”循环。

伪代码：

typedef enum {rel1, rel2} rel_t;

// Where 'ReleaseSequence' is a generator functor. Class that takes rel_t
// in constructor and creates an appropriate functor that returns the
// required sequence of numbers for a particular release

ReleaseSequence seq(rel1);
while (i = seq()) {
    switch (i) {
        case 1:...
        case 2:...
        case 3:...
        case 4:...
        case 5:...
        case 6:...
        case 7:...
        default:
               break;
    }
}

Answer 3

我发现看到嵌套在循环中的 switch 很奇怪......

看起来你正在执行一个管道。那你为什么不简单地将 is 定义为一。

例如，一个简单的函数指针数组就可以了。您可以为每个版本定义一个这样的管道，并为要忽略的参数提供“noop”函数。

Answer 4

如果您想捕获意外的值，难道不应该将该检查放在 default 部分吗？

...
default:
    assert(false);   // We should never get here!

Answer 5

#define 这是哪个版本，然后使用#ifdef。

  switch (i)
  {
    case 1: /*process data*/ break;
    case 2: /*process data*/ break;
    case 3: /*process data*/ break;
    case 4: /*process data*/ break;
    case 5: /*process data*/ break;
#ifdef REL_2
    case 6: /*process data*/ break;  // <-- executed only for REL_2
#endif
    case 7: /*process data*/ break;
    default: break;
  }