“方法 foo() 不必要地装箱了一个布尔常量”声纳警告答案

【问题标题】："Method foo() needlessly boxes a boolean constant" Sonar warning“方法 foo() 不必要地装箱了一个布尔常量”声纳警告
【发布时间】：2017-08-25 13:58:58
【问题描述】：

SonarQube 的分析器在我们的 Java 代码中报告了大量此类消息。

方法 foo(String, String) 不必要地装箱一个布尔常量

在许多情况下，它为Boolean 返回类型方法返回true。

我想知道这在多大程度上是 2017 年 Oracle Java 8 的（性能？）问题？它真的最终会创建新的Boolean 实例还是自动优化为Boolean.TRUE？

更新

Sonar 规则键是fb-contrib:NAB_NEEDLESS_BOOLEAN_CONSTANT_CONVERSION。

【问题讨论】：

【解决方案1】：

此问题背后的原因是您无缘无故地调用了一个方法。我们如何验证这一点：让我们来看看这个（复杂的;））代码

  Boolean foo(String s) {
   return true;
  }

可以做的是反编译这个。这为我们提供了以下字节码指令（为简洁起见，稍微简化了一点）

ICONST_1
INVOKESTATIC java/lang/Boolean.valueOf (Z)Ljava/lang/Boolean;
ARETURN

如您所见，有一个方法调用可以从常量 1 创建一个 Boolean。

如果我们现在将代码更改为：

  Boolean foo(String s) {
   return Boolean.TRUE;
  }

生成的字节码是：

GETSTATIC java/lang/Boolean.TRUE : Ljava/lang/Boolean;
ARETURN

获取静态常量并返回它应该更有效。

【讨论】：