JAVA中Long与Integer比较容易犯的错误

短命女 2021-09-10 22:46 354阅读 0赞

今天使用findbugs扫描项目后发现很多高危漏洞，其中非常常见的一个是比较两个Long或Integer时直接使用的==来比较。 其实这样是错误的。

因为Long与Ineger都是包装类型，是对象。  而不是普通类型long与int ， 所以它们在比较时必须都应该用equals，或者先使用longValue()或intValue()方法来得到他们的基本类型的值然后使用==比较也是可以的。

但是有一种特殊情况， 其实Long与Integer都将 -128~127 这些对象缓存了。  可以看看Long类型源码里面有一个LongCache类，代码如下：

**\[java\]**  [view plain][]  [copy][view plain]

[![在CODE上查看代码片][CODE]][CODE_CODE] [![派生到我的代码片][ico_fork.svg]][ico_fork.svg 1]

1.  **private** **static** **class** LongCache \{  
2.      **private** LongCache()\{\}  
3.    
4.      **static** **final** Long cache\[\] = **new** Long\[-(-128) + 127 + 1\];  
5.    
6.      **static** \{  
7.          **for**(**int** i = 0; i < cache.length; i++)  
8.          cache\[i\] = **new** Long(i - 128);  
9.      \}  
10.     \}

先看看这个例子：

**\[java\]**  [view plain][]  [copy][view plain]

[![在CODE上查看代码片][CODE]][CODE_CODE] [![派生到我的代码片][ico_fork.svg]][ico_fork.svg 1]

1.  **public** **class** Test05 \{  
2.    
3.      **public** **static** **void** main(String\[\] args) \{  
4.          Long a = 5L;  
5.          Long b = 5L;  
6.    
7.          System.out.println("a == b ? " + (a == b));  
8.    
9.          Long c = 129L;  
10.         Long d = 129L;  
11.         System.out.println("c == d ? " + (c == d));  
12.     \}  
13. \}

打印的结果是：

**\[css\]**  [view plain][]  [copy][view plain]

[![在CODE上查看代码片][CODE]][CODE_CODE] [![派生到我的代码片][ico_fork.svg]][ico_fork.svg 1]

1.  a == b ? true  
2.  c == d ? false

原因

首先来看看 Long a = 5L ; 它是如何将一个基本类型long包装成一个对象Long的 。

可以写一个测试类，然后反编译一下，看看java它是如何解析Long a = 5L这样一条命令的 。

测试类如下：

**\[java\]**  [view plain][]  [copy][view plain]

[![在CODE上查看代码片][CODE]][CODE_CODE] [![派生到我的代码片][ico_fork.svg]][ico_fork.svg 1]

1.  **public** **class** Test06 \{  
2.      Long l = 3L;  
3.  \}

然后使用javap -verbose Test06 就能看到反编译的结果了， 下面是输出的部分：

**\[css\]**  [view plain][]  [copy][view plain]

[![在CODE上查看代码片][CODE]][CODE_CODE] [![派生到我的代码片][ico_fork.svg]][ico_fork.svg 1]

1.  \{  
2.  java.lang.Long l;  
3.    
4.  public com.spring.test.Test06();  
5.    Code:  
6.     Stack=3, Locals=1, Args\_size=1  
7.     0:   aload\_0  
8.     1:   invokespecial   \#10; //Method java/lang/Object."<init>":()V  
9.     4:   aload\_0  
10.    5:   ldc2\_w  \#12; //long 3l  
11.    8:   invokestatic    \#14; //Method java/lang/Long.valueOf:(J)Ljava/lang/Long;  
12.    11:  putfield        \#20; //Field l:Ljava/lang/Long;  
13.    14:  return  
14.   LineNumberTable:  
15.    line 3: 0  
16.    line 5: 4  
17.    line 3: 14  
18.   
19.   LocalVariableTable:  
20.    Start  Length  Slot  Name   Signature  
21.    0      15      0    this       Lcom/spring/test/Test06;  
22.   
23.   
24. \}

从Code中的8可以看出调用了Long的一个类方法Long.valueOf(Long) ， 所以可以得到的结论是Long a = 5L实际上等于 Long a = Long.valueOf(5) ;

然后再看看Long.valueOf()方法是如何定义的：

**\[java\]**  [view plain][]  [copy][view plain]

[![在CODE上查看代码片][CODE]][CODE_CODE] [![派生到我的代码片][ico_fork.svg]][ico_fork.svg 1]

1.   **public** **static** Long valueOf(**long** l) \{  
2.  **final** **int** offset = 128;  
3.  **if** (l >= -128 && l <= 127) \{ // will cache  
4.      **return** LongCache.cache\[(**int**)l + offset\];  
5.  \}  
6.         **return** **new** Long(l);  
7.     \}

一目了然，会先判断基本类型的值如果在-128~127之间，就会直接从LongCache里面取出缓存的对象返回，否则就new一个新的Long对象返回 。

现在就不难理解Test05程序执行得到的结果了，因为a与b等于5，在-127~128之内，所以都是直接从LongCache里面返回的一个Long对象，所以他们在使用==比较的时候，就是相等的（对于对象类型来说，==比较的是两个对象的引用指向堆中的地址） ，而c与d等于129，不在-127~128之间，所以他们他们是分别new出来的两个新的Long对象，使用==来比较自然是不相等的了。

Long重写了equals方法：

**\[java\]**  [view plain][]  [copy][view plain]

[![在CODE上查看代码片][CODE]][CODE_CODE] [![派生到我的代码片][ico_fork.svg]][ico_fork.svg 1]

1.   **public** **boolean** equals(Object obj) \{  
2.  **if** (obj **instanceof** Long) \{  
3.      **return** value == ((Long)obj).longValue();  
4.  \}  
5.  **return** **false**;  
6.     \}

它是先通过.longValue()方法获取Long对象的基本类型long的值之后再做比较的。

所以对于Integer与Long的比较，最好是使用equals来比较才能确保得到我们想要的结果。

Integer与Long一样，这里就不举例了。

[view plain]: http://blog.csdn.net/is_zhoufeng/article/details/38443507#
[CODE]: https://code.csdn.net/assets/CODE_ico.png
[CODE_CODE]: https://code.csdn.net/snippets/446106
[ico_fork.svg]: https://code.csdn.net/assets/ico_fork.svg
[ico_fork.svg 1]: https://code.csdn.net/snippets/446106/fork