这个开关盒构造有更好的实现吗？

Question

我正在开发一个嵌入式系统，其中必须访问一个寄存器，然后递增以实现我正在寻找的结果，因为机器正在通知并配置为对我的访问和更改做出反应改变国旗。所以switch的参数必须保持原样，否则会改变嵌入式系统的行为。

但可能会出现我不想让任何案例被调用的情况。但我仍然需要访问并增加 switch 的参数。

（更深入我正在逐步将一系列模拟值转换为数字值转换。索引用于与当前转换保持同步并将其与相应的案例相关联以正确处理数字。可能会发生索引与当前转换不同步的状态，因此必须运行转换序列，而不会调用任何案例（以防止设置错误数据），直到序列完成并且可以执行重新同步）

我目前这样做的方式是这样的：

switch (RunIndex++)/*RunIndex may only be accessed one time per execution
of this construct and has to be incremented in the same step. thats given.*/
{
   if (RunIndexSyncedWithADCensured == false)
   {
     break;
   }
   case 0:
     Case0RelatedData = SomeOperationsForCase0(RawData);
     break;
   case 1:
     Case1RelatedData = SomeOperationsForCase1(RawData);
     break;
   case 2:
     Case2RelatedData = SomeOperationsForCase2(RawData);
     break;
   default:
     RunIndex = 0;
     break;
}

这个结构可以完成这项工作，但它看起来有点争议，考虑将它提交到生产代码中我感觉不太好。

那么有没有更好看的方法来实现同样的效果，而不需要额外的变量或赋值？

注意：

这也可能是相关的，这是由两部分组成的中断函数的第一部分。 第一部分处理必须发生的事情if() 转换完成。第二部分，还有什么要做的if()这个转换也结束了序列。因此，在不进入第二部分的情况下简单地从函数返回是没有选择的。并且目前没有循环结构，if(...)break; 可能会爆发。 （这也是我将 if 放在 switch 范围内的原因，因为它至少按标准是一种有效的突破方式。）

Answer 1

首先，switch() 中的if() 永远不会被执行。
考虑下面的代码 sn-p：

#include <stdio.h>

int main(int argc, char *argv[])
{
    int i = 2;

    switch(i) {
        if (i == 2) {
            printf("I M HERE\n");
        }
        case 1:
            printf("1\n");
        break;
        case 2:
            printf("2\n");
        break;
        default:
            printf("default\n");
        break;
    }
    return 0;
}

对于您的代码：您希望打印字符串 I M HERE。但事实并非如此。
上述代码 sn -p 的输出为：

2

No statements before case/default(switch constructs): is executed inside switch

现在回答

我不想让任何案例被调用。但我仍然需要访问并增加开关的参数

只需将if() 移到 switch() 之外。

if (RunIndexSyncedWithADCensured) {
    switch (RunIndex++) {
        case 0:
            Case0RelatedData = SomeOperationsForCase0(RawData);
            break;
            /* Other cases here */
        default:
           RunIndex = 0;
           break;
    }
} else
    RunIndex++;

Answer 2

为什么不先保存值，然后增加它并在开关中使用保存的值？顺便说一句，这还包括两个访问，第一个是从 RunIndex 读取值，第二个是递增它。

int runIndex = (RunIndex++);
if (RunIndexSyncedWithADCensured )
{
    switch (runIndex)/*RunIndex may only be accessed one time per execution
    of this construct and has to be incremented in the same step. thats given.*/
    {
        case 0:
            Case0RelatedData = SomeOperationsForCase0(RawData);
            break;
        case 1:
            Case1RelatedData = SomeOperationsForCase1(RawData);
            break;
        case 2:
           Case2RelatedData = SomeOperationsForCase2(RawData);
           break;
        default:
           RunIndex = 0;
           break;
    }
}

Answer 3

由于您使用相邻的索引号，您可以创建一个函数指针数组来替换开关。这就是优化器无论如何都会把开关变成的东西。因此，您得到的不是晦涩的开关，而是：

if (RunIndexSyncedWithADCensured)
{
  SomeOperationsForCase[RunIndex](RawData);
}

RunIndex++;
if (RunIndex > MAX)
{
  RunIndex = 0;
}

---

与 switch 语句设计完全无关：如果RunIndex 是敏感的易失变量，例如某些硬件寄存器，那么您不应该直接在任何形式的计算中使用它。复制一份：

volatile int RunIndex; 
...

int index = RunIndex; // read from RunIndex

if (RunIndexSyncedWithADCensured)
{
  SomeOperationsForCase[index](RawData);
}

index++;
if (index > MAX)
{
  index  = 0;
}

RunIndex = index; // write to RunIndex

这是所有此类易失性变量的标准做法。