方法调用效率低下的 new Integer(int) 构造函数；改用 Integer.valueOf(int)

Question

我的目标是优化我的应用程序代码。我的代码如下所示：

int a = 10;
Map<String , Integer> myMap = new TreeMap<>();

myMap.put("first" , new Integer(a)); //[FindBugs] Method com.abc.xyz.Test.main(String[]) invokes inefficient new Integer(int) constructor; use Integer.valueOf(int) instead 

当我在 Netbeans 中通过 Findbugs 进行静态分析时，它显示存在类似“方法调用效率低下的新 Integer(int) 构造函数；请改用 Integer.valueOf(int)”的警告/错误.

我知道 new Integer(int) 与 Integer.valueOf(int) 之间的区别。

一个创建一个额外的对象，另一个不创建。还有一个不缓存，另一个缓存。

所以我已经像这样修改了我的代码......

m.put("first" , Integer.valueOf(a)); // Unnecessary boxing to Integer

但同样也会发出警告“Innecessary boxing to Integer”（由编辑器而非 findbugs）。

所以，我又改成这样了......

m.put("first" , a); //No warning at all...

最终完全没有警告。

我的问题：

1.) 这个link 建议在内部（通过编译器） m.put("first" , a); 正在转换 m.put("first" , Integer .valueOf(a));

（在那个例子中，有一个 List-ArrayList，这里我们有 Map-TreeMap...仅供参考）。 那为什么编辑器会发出警告？ 我该怎么办？优化的方式是什么？

2.) 如果不是Map，如果有像HashTable这样的数据结构，那么???

3.) 为什么编辑器对整数进行不必要的装箱。

4.) 为什么 m.put("first" , a) 有效？因为我正在传递原始变量，而map 的put() 只接受Object。那么是不是因为自动装箱？

Answer 1

1.) 此链接建议在内部（由编译器） m.put("first" , a);正在转换 m.put("first" , Integer.valueOf(a));

（在那个例子中，有一个 List-ArrayList，这里我们有 Map-TreeMap...仅供参考）。那为什么编辑器会发出警告？我该怎么办？优化的方式是什么？

是的，编译器知道m.put("first", a) 只接受对象，因此应用了自动装箱。在性能方面，使用自动装箱或写 Integer.valueOf(a) 不会有任何区别。

另一方面，new Integer(a) 并不比Integer.valueOf(a) 慢。不同之处在于对于小的绝对值（默认值为 -128 到 127）Integer.valueOf(a) 将使用缓存，即不会一直创建新对象。对于所有其他值，它无论如何都会调用new Integer(a)。

例子：

Integer.valueOf(1) == Integer.valueOf(1) 将返回 true
Integer.valueOf(1000) == Integer.valueOf(1000) 将返回 false
new Integer(1) == new Integer(1) 将返回 false，因为此处未使用缓存

2.) 如果不是Map，如果有HashTable这样的数据结构，那么？？？

你为什么这么问？有HashTable，但由于它是同步的，这意味着比HashMap 更多的开销，所以除非你需要同步，否则如果你需要排序，请坚持HashMap 或TreeMap。

3.) 为什么编辑器给 Integer 提供不必要的装箱。

这可能只是因为可读性（a 比 Integer.valueOf(a) 短）。

4.) 为什么 m.put("first" , a) 有效？因为我传递原始变量和地图的 put() 只接受对象。那么是因为自动装箱吗？

见 1

Answer 2

4.) 为什么 m.put("first" , a) 有效？因为我传递原始变量和地图的 put() 只接受对象。

自动装箱。

int 会自动转换为 Integer，反之亦然（在这种情况下可能会出现 NullPointerException）

3.) 为什么编辑器给 Integer 提供不必要的装箱。

因为您不需要编写这段代码。编译器会为你做这件事。

它通常更具可读性

2.) 如果不是Map，如果有HashTable这样的数据结构，那么大小写是一样的？？？

是的，在 JDK 集合中仅适用于对象。这意味着必须将原始类型装箱。它具有很小的运行时成本和巨大的内存开销。 Integer 比 int 占用大约 300% 的内存。

你无法逃脱它。避免装箱开销的唯一方法是使用专门的集合，如 GNU trove，它为每个原始类型提供一个类。仅当您计划将数百万个原始元素存储到集合中时才有用。

最后永远不要写 new Integer(x)。 Integer.valueOf(x) 做同样的事情，但它维护一个内部缓存以避免为一些常用值创建新实例。