C++ if 语句位于 switch 中的 case 之前

nuc*_*ear 8 c++ if-statement case switch-statement

我的任务是为我的公司重写一些旧软件,并在源代码中发现了一个有趣的构造。

switch(X){
    if(Y==Z){
       case A: ... break;
       case B: ... break;
    }
    case C: ... break;
    default: ...;
}
Run Code Online (Sandbox Code Playgroud)

编译器警告我,if 语句中的代码将不会被执行,但在测试时似乎 if 语句未经过验证,但 case 语句却经过验证。

有没有什么原因,也许是早期的 C++ 时代,因为有些代码到现在已经有 20 多年的历史了,为什么你会编写这样的构造?它在生产代码中,所以它似乎确实做了一些事情?

如果我能像编译器那样忽略它,我将感谢任何内部原因。

编辑:它也发生了多次,所以它似乎不是一个简单的错误。这就是让我困惑的地方。

Edit2:这是我用于测试的示例。我不确定它有多大帮助,但是原始代码位于一个 1200 行的怪物函数中,因此从中创建一个基本上是不可能的。

for (int i=0; i<5;i++)
{
    switch(i)
    {
        if (i==0 || i ==1)
        {
            cout << "reached before case";
            case 0: cout << "inside case 0" << std::endl; break;
            case 1: cout << "inside case 1" << std::endl; break;
            case 2: cout << "inside case 2" << std::endl; break;
        }
        case 3: cout << "inside case 3" << std::endl; break;
        default: cout << "inside default" << std::endl;
    }
}
Run Code Online (Sandbox Code Playgroud)

eer*_*ika 4

有什么理由......为什么你会写这样的构造吗?

只有作者确切知道(即使他们可能也不知道)。如果有可用的源版本控制元数据,则关联的提交消息可能会有用。在没有更多信息的情况下,我们只能猜测他们在想什么。一些可能的答案:

  • 作者假设if语句的条件会产生一些影响,但他们错了,而且这个错误没有经过测试。
  • 这是一些重构的残余结果。
    • 也许删除了一些使该语句有意义的代码。
    • 或者也许它是从其他确实有意义的地方复制而来的。
    • 或者也许曾经有一系列的 if 语句,并且打算用 switch 来代替它们,但这个改变是半途而废的。