java - 这段代码有什么严重的错误？它在语义上对我来说似乎是正确的

Question

在博文中看到这种代码让作者发疯..为什么？

public boolean foo() {
    boolean b = bar();
    if (b == true) {
        return true;
    }
    else {
        return false;
    }
}

Answer 1

因为它有一个更短的版本:

public boolean foo() {
    return bar();
}

正如其他人所指出的，整个函数 foo 可能是多余的。然而，在某些情况下无法消除它，例如，如果它是在您正在实现的接口(interface)中定义的，或者如果 bar 是私有(private)的。

构造 if (b == true) 对于没有经验的程序员来说尤其危险，因为它可能被错误地写成 if (b = true)。此问题不会被编译器捕获，但会始终默默地执行 if 的真实分支。