java 使用逻辑运算符代替 if-else if 进行返回

Question

我在 equals 方法中有以下代码。

public boolean equals(Object o){
    if (o == null) return false;
    if (o == this) return true;
    if (!(o instanceof Vertex)) return false;
    return ((Vertex) o).label().equals(label);
}

我的 IDE 突出显示了 if 语句，并希望我基本上执行此操作。

public boolean equals(Object o){
    return (o != null) && ((o==this) || ((o instanceof Vertex) && ((Vertex) o).label().equals(label);
}

有人告诉我，编译器通常足够智能，可以进行优化，并且通常应该为可读性编写代码。因此，显然第二个代码示例不像第一个代码示例那么容易阅读。我的 IDE 是否只是令人烦恼，还是这样做有一些实际的性能优点？

Answer 1

首先，只有在您知道这是瓶颈时才进行优化，否则要提高代码的可读性。

您可以检查字节码来查看，但我怀疑它们即使不完全相同，也非常接近。即使字节码存在细微差异，我也看到 JIT 编译器将内容优化到没有差异的程度。您始终可以进行性能测试来确定。

Answer 2

大多数 IDE 都是可配置的，允许您指定其输出的警告样式类型。您是对的，按照编译器的建议执行生成的代码没有任何好处。编译器确实应该能够优化这些事情。

Answer 3

您的 IDE 建议的版本几乎没有任何性能优势。此外，你说得对，JVM（不仅仅是编译器）可以做很多优化。

正如其他人指出的那样，尽量提高可读性。你编写的代码被人类读取，并由Java编译器和虚拟机编译、修改和优化。

顺便提一句。一个简短的提示：您的代码可以像这样进行优化（省略等于 null 的条件，因为 o 在第二个之后保证是 Vertex 的实例健康）状况）：

public boolean equals(Object o){
    if (o == this) return true;
    if (!(o instanceof Vertex)) return false;
    return ((Vertex) o).label().equals(label);
}

Answer 4

不要试图让编译器变得聪明。您正在编写供人阅读的代码。让编译器担心将其转换为机器可读的格式。

Answer 5

这样做的好处是，如果您内联代码，那么您可以轻松地将其分配给布尔值，但您也可以使用三元运算符使其更具可读性，这将比优化版本更具可读性，但会具有相同的功能，如果您了解如何做到这一点，可能会帮助您稍后简化编码。

public boolean equals(Object o){
    return o == null || !(o instance of Vertex) ? false :
       o == this ? true :
       ((Vertex) o).label().equals(label);
}

这应该仍然可读。

我还没有尝试编译这个，可能需要一些额外的括号，但我要寻找一个概念，而不是你可以复制和复制的东西。粘贴。

Answer 6

为什么不自己测试一下性能呢？这是一个示例：

public static long timeIt(Runnable runnable) {
    long start = System.nanoTime();
    runnable.run();
    long end = System.nanoTime();
    return end - start;
}

将为您计时 run 方法。所以你可以这样称呼它：

timeIt(new Runnable() { public void run() { OPERATION_TO_BE_TIMED; }})