1.问题场景
先看如下一段代码
if(activity.getTotalCounts()==activity.getParticipationCounts()) { long time = activity.getUpdatedAt().getTime()+60*30*1000; vo.setProbableOpenTime(new Date(time)); } 1234 1234activity.getTotalCounts() 和activity.getParticipationCounts()的值都是Integer类型,在两者数值都一样的时候,测试发现vo.setProbableOpenTime(new Date(time)); 这段代码竟然有时候能执行到,有时候却不行。比如 都等于100时,能够执行到这段代码,都等于200时却不行了,好奇怪的问题,都是一样的数值怎么就有时候成立,有时候不成立呢!于是有了下面这段测试代码
2.Integer数值比较单元测试
@Test public void IntegerTest() { Integer a = 100; Integer b = 100; Integer c = 300; Integer d = 300; System.out.println(a==b); System.out.println(c==d); System.out.println(c.equals(d)); } 1234567891011 1234567891011输出结果为: true false true 结果可知,Integer 用==比较数值确实有时候成立,有时候却不行。
3.问题的本质 想知道为啥有这么奇怪的结果,还是要看看源代码的,如下:
* This method will always cache values in the range -128 to 127, * inclusive, and may cache other values outside of this range. public static Integer valueOf(int i) { assert IntegerCache.high >= 127; if (i >= IntegerCache.low && i <= IntegerCache.high) return IntegerCache.cache[i + (-IntegerCache.low)]; return new Integer(i); } 123456789 123456789Integer a = 100实际上调用了valueOf(int i)方法。 这里low是-128,high是127,从方法的注释中可以得知,在[-128,127]之间,Integer总是缓存了相应数值对象,这样每次取值时都是同一个对象,而不在此区间时,则不一定会缓存,那么取出两个同样数值大小的对象则不是同一个对象,每次都会new Integer(i);,所以其对象的地址不同,则用==比较是自然不会成立。Integer缓存的目的正如注释所言better space and time performance by caching frequently requested values.但是个人以为这样的设计又何尝不是雷区!
为了防止不小心掉入这样的陷阱,对于基本类型的比较,用“==”;而对于基本类型的封装类型和其他对象,应该调用public boolean equals(Object obj)方法(复杂对象需要自己实现equals方法)。
