c++ - 此 switch 语句中的代码味道？

Question

我想知道应该在哪里将这种样式的 switch 语句更改为 if else 语句。

switch (foo) // foo is an enumerated type
{
    case barOne:
        if (blahOne)
        {
            DoFunction(//parameters specific to barOne);
            break;
        }
   case barTwo:
        if (blahTwo)
        {
            DoFunction(//parameters specific to barTwo);
            break;
        }
   //etc.
   default:
       // Whatever happens if none of the case's conditionals are met
}

除非满足其中一种情况的条件，否则基本上会失败。这些情况非常相似，只是需要检查什么和需要传递什么不同，这就是我使用 switch 语句的原因。

使用 if else if 会更好吗？否则，它是否足够清晰以保留，但又不够清晰以保证对 fallthrough 发表评论？多态性也始终是一种选择，但对我来说似乎有点矫枉过正。

Answer 1

在某些情况下，这似乎会做一些奇怪的事情。如果 foo == bar1，并且 blahOne 为假，但 blahTwo 为真怎么办？然后你会失败并调用 foo == bar2 情况下的函数，即使 foo 不等于 bar2。

这在实践中可能出乎意料，但如果真的发生了，调试起来可能会很困难。在这种情况下，我会投票给 if else，因为流程更简单。

if (foo == barOne && blahOne)
{
    DoFunction(/*parameters specific to barOne*/);
}
else if (foo == barTwo && blahTwo)
{
    DoFunction(/*parameters specific to barTwo*/);
}
else
{
    // Handle the fallthrough case.
}

当然，如果意图是即使 foo != barTwo 也可以计算 blahTwo，那么转换可能是最好的方法，但在这种情况下我肯定会赞成一些解释性评论。