java - 冗余 If 语句警告

Question

if ( a > b) {
  return true;
}
return false;

对于上面的代码，Netbeans 给出了 “Redundant if statement” 警告并建议将其更改为:

return a > b;

我认为第一个版本更易于阅读并且更喜欢在我的代码中使用它。我想知道它与建议的相比是否有任何缺点。

Answer 1

这个

if ( a > b) {
  return true;
}
return false;

包括插入a的值(value)在堆栈上，压入 b 的值在堆栈上，弹出两者并检查 > 的结果.如果是 true , 推送 true 的值在堆栈上，然后将其弹出并返回。如果是 false , 分支到字节码的更下方，推送 false 的值在堆栈上，将其弹出并返回。

如果是

return a > b;

您正在插入 a 的值和 b在堆栈上，然后弹出值并推送 > 的结果将这些值压入堆栈。然后弹出该值并返回它。

所以

return a > b;

在字节码级别上不明显更高效。

(IMO 我发现第二个更具可读性，我相信大多数人也会。)