使用“if/elseif/else”与“if/else{if/else}”相比

Question

我发现自己经常使用这样的模式：

if (a > b) {
    foo();
}
elseif (c > d) {
    bar();
}
else {
    baz();
}

这里的要点是，第二个条件与第一个条件没有明显的联系，除非您仔细遵循程序逻辑。这是一件非常糟糕的事情吗？ 出于可维护性的原因，更好地表述上述内容吗

if (a > b) {
    foo();
}
else {
    if (c > d) {
        bar();
    }
    else {
        baz();
    }
 }

？有没有更好的模式我完全错过了？ “不明显连接”位似乎是我的代码中更常见的错误来源之一。

Answer 1

这并不重要。

我更喜欢 Leaky Rowboat* 模式：

if (a > b) 
{
    foo();
    return;
}

if (c > d) 
{
    bar();
    return;
}
baz();

当您返回某些东西时，这种模式会更好：

if (a > b) 
    return foo();

if (c > d) 
    return bar();

return baz();

_{*提早保释，快速保释}

Answer 2

我认为第一个绝对是更好的选择。我唯一一次使用第二个方法是将代码放在外部 else 中，而不在内部 if/else 中。

当我看到 else if 时，我立即寻找 if。所以我想说这显然是有联系的。

Answer 3

我认为这是代码味道。你在这里做什么或者为什么这样做并不是很明显。事实上，您认为它们之间没有明显的联系，而且它们是错误的常见来源，这一事实告诉您不要这样做。

重写此代码，以便清楚在这些条件下进行分支的原因。理想情况下，您将能够阅读代码并让它表达您的意图和/或您的规范。

taller_than_wide = a > b;
more_expensive_than_normal = c > d;

if (taller_than_wide) {
      foo();
}
elseif (more_expensive_than_normal) {
      bar();
}
else {
      baz();
}

Answer 4

我避免使用第二种方法，因为它会导致大型条件语句出现大量缩进。

Answer 5

我当然会使用第一个，因为它比第二个更具可读性。

第二个选项将迫使读者记住哪些条件必须为真才能到达嵌套的 if ，它在每个时刻以及第三个或第四个嵌套的 if 中读取> 这变得非常烦人、非常脆弱并且在逻辑上难以遵循。