装箱的值将取消装箱，然后立即重新装箱

Question

我收到Findugs错误“盒装值未装箱，然后立即重新装箱”。

这是代码：

Employee emp = new Employee()
Long lmt = 123L;

emp.setLimit(Long.valueOf(lmt)); 

在此，员工limit字段的类型为Long。你能告诉我这是什么错误吗？

Answer 1

问题在于您正在转换Long - ＆gt; long - ＆gt; Long。

所以在后台：

1. Long.valueOf(lmt)将Long转换为long
2. emp.setLimit(<long>);再次将long转换为Long

从Java 5开始，自动装箱发生=＆gt;您的代码应如下所示：

Employee emp = new Employee()
Long lmt = 123L;

emp.setLimit(lmt); 

甚至：

Employee emp = new Employee()
long lmt = 123L;

emp.setLimit(lmt); 

Answer 2

之所以发生这种情况，是因为Long.valueOf(long)会将您的lmt从Long拆至long，只是为了再次获得Long。正如您所说limit是Long，您不需要使用Long.valueOf，只需使用var：

emp.setLimit(lmt); 

Answer 3

emp.setLimit(Long.valueOf(lmt));

Long.valueOf获取long值，但您传递Long值 - 强制取消装箱。然而，紧接着，Long.valueOf重新设置了值，表达式再次评估为Long。 FindBugs检测到不必要的转换链Long - ＆gt; long - ＆gt; Long。