if 和 if-else 之间有明显的区别吗？

Question

鉴于以下代码片段，有什么明显的区别吗？

public boolean foo(int input) {
   if(input > 10) {
       doStuff();
       return true;
   }
   if(input == 0) {
       doOtherStuff();
       return true;
   }

   return false;
}

与

public boolean foo(int input) {
   if(input > 10) {
      doStuff();
      return true;
   } else if(input == 0) {
      doOtherStuff();
      return true;
   } else {
      return false;
   }
}

或者单退出原则在这段代码中会更好...

public boolean foo(int input) {
   boolean toBeReturned = false;
   if(input > 10) {
      doStuff();
      toBeReturned = true;
   } else if(input == 0) {
      doOtherStuff();
      toBeReturned = true;
   }

   return toBeReturned;
}

是否有任何明显的性能差异？您认为其中一个比其他的更易于维护/可读吗？

Answer 1

在第二个示例中，您非常清楚地表明这两个条件是互斥的。
对于第一个，情况不太清楚，并且在两个 if 之间添加对 input 的赋值（不太可能）的情况下，逻辑将会改变。
假设将来有人在第二个 if 之前添加 input = 0。
当然这种情况不太可能发生，但是如果我们在这里谈论可维护性，if-else明确表示存在互斥的条件，而一堆if则不会，而且它们之间并不像if那样依赖-else 块。

编辑：现在我看到，在这个特定的示例中，return 子句强制相互排他性，但我们再次讨论的是可维护性和可读性。

不管怎样，关于性能，如果这是用Java编码的，你不应该关心几个if块的性能，如果它是在一个非常慢的硬件中嵌入C，也许，但肯定不是用java。

Answer 2

使用最能描述您意图的形式。

如果事情这么简单，就不要遵循单一退出原则——它只会让事情变得更加混乱。

Answer 3

• 在第一个：

  最终有人会出于某种奇怪的原因，在你不注意的时候添加一些添加语句，这将使该方法在某些奇怪的条件下失败，每个人（或者最糟糕的是一个人）都会花费 4 小时。查看源代码并调试应用程序，终于发现中间有东西。

• 第二个肯定更好，不仅可以防止这种情况，而且还有助于清楚地说明，不再是这个或这个其他。

  如果我们在 if 中编写的所有代码最多 10 行，这实际上并不重要，但不幸的是事实并非如此，存在其他程序员出于某种原因认为if 主体应该是 > 200 行长...无论如何。

• 我不喜欢第三个，它迫使我寻找 return 变量，而且更容易找到 return 关键字

关于速度性能，它们（几乎）相同。不用担心这个。

Answer 4

在你的最后一个例子中，不要这样做：

public boolean foo(int input) {
   boolean toBeReturned = false;
   if(input > 10) {
      doStuff();
      toBeReturned = true;
   } else if(input == 0) {
      doOtherStuff();
      toBeReturned = true;
   }

   return toBeReturned;
}

但是这个（注意Java的final的使用）：

public boolean foo(int input) {
   final boolean toBeReturned;    // no init here
   if(input > 10) {
      doStuff();
      toBeReturned = true;
   } else if(input == 0) {
      doOtherStuff();
      toBeReturned = true;
   } else {
      toBeReturned = false;
   }
   return toBeReturned;
}

通过这样做，你可以明确你的意图，这对于支持“意图编程”的IDE来说是天赐之物（不需要“编译”来查看潜在的错误，即使在部分 AST 上，好的 IDE 也可以实时检查不完整的源代码并立即发出警告）。

这样您就可以确定不会忘记初始化返回值。如果您稍后决定毕竟需要另一个条件，那么这很好。

我一直这样做，自从我开始使用 IntelliJ IDEA（版本 4 左右，很久以前）以来更是如此，这让我避免了很多愚蠢的分散注意力的错误......

有些人会认为这代码太多了这样一个简单的例子，但完全没有抓住重点：重点是使意图清晰，以便代码易于阅读并且以后可以轻松扩展，而不会意外忘记分配toBeReturned，也不会意外忘记要从后面的子句中返回，您可以添加。

否则，如果“简洁”是游戏的名称，那么我会写：

public boolean foo(int a) {
  return a > 10 ? doStuff() : a == 0 ? doOtherStuff() : false; 
}

doStuff 和 doOtherStuff 都将返回 true。

Answer 5

从语义上来说——不。从性能角度来看，这取决于编译器，即它是否可以发现两个条件不能同时为真。我敢打赌标准 Sun 编译器可以。是否采用单出口原则取决于喜好。我个人讨厌它。

Answer 6

版本 #1 和 #2 可能比 #3 更快，但我认为性能差异很小。我宁愿关注可读性。

就我个人而言，我永远不会使用版本 #2。在 #1 和 #3 之间，我会选择为相关案例生成最具可读性的代码。我不喜欢我的方法中有很多退出点，因为它使代码难以分析。然而，在某些情况下，当我们针对某些特殊情况立即退出并继续主要情况时，流程会变得更加清晰。

Answer 7

当两个示例不相似时，请考虑这种情况：

    public boolean foo(int input) {
        if (input > 10) {
            // doStuff();
            return true;
        }
        System.out.println("do some other intermediary stuff");
        if (input == 0) {
            // doOtherStuff();
            return true;
        }

        return false;
    }

与

    public boolean foo(int input) {
        if (input > 10) {
            // doStuff();
            return true;
        } 
        //System.out.println("doing some intermediary stuff... doesn't work");
        else if (input == 0) {
            // doOtherStuff();
            return true;
        } else {
            return false;
        }
        return false;
    }

第一种方法可能更灵活，但是两个公式在不同的情况下都有其用途。

关于性能，我认为对于任何由理智的程序员编写的常规 Java 应用程序来说，差异很小，值得考虑:)。

Answer 8

在您的情况下，第二个 if 仅在第一个 if 失败时才会被调用，因此这里不太重要，但如果您的第一个 if 做了某事并且没有返回，则第二个 if （然后总是 false）仍将被测试，除非它处于 else-if 中。

换句话说，在某些情况下，if-else-if 和 if-if 之间的差异很重要，但这不是其中之一。

示例：尝试这个，然后在删除 else 后尝试它。你会得到两个不同的输出：

int someNumber = 1;
if(someNumber < 5)
{
    someNumber += 5;
    Console.WriteLine("First call.");
}
else if(someNumber >= 5)
{
    Console.WriteLine("Second call.");
}

Answer 9

第一个和第二个片段之间确实没有区别。然而第三个片段效率很低。因为您要等到方法中的最后一行代码才将程序的控制权返回给调用者，所以浪费了处理能力/内存，而前两个代码片段一旦确定其中一个条件为真，就会立即返回控制权。