细节之Integer数值(==)比较问题

来源:互联网 发布:淘宝火影忍者手游cdk 编辑:程序博客网 时间:2024/06/07 02:46

1.问题场景

先看如下一段代码

if(activity.getTotalCounts()==activity.getParticipationCounts())  {    long time = activity.getUpdatedAt().getTime()+60*30*1000;    vo.setProbableOpenTime(new Date(time)); }
  • 1
  • 2
  • 3
  • 4
  • 1
  • 2
  • 3
  • 4

activity.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));    }
  • 1
  • 2
  • 3
  • 4
  • 5
  • 6
  • 7
  • 8
  • 9
  • 10
  • 11
  • 1
  • 2
  • 3
  • 4
  • 5
  • 6
  • 7
  • 8
  • 9
  • 10
  • 11

输出结果为: 
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);    }
  • 1
  • 2
  • 3
  • 4
  • 5
  • 6
  • 7
  • 8
  • 9
  • 1
  • 2
  • 3
  • 4
  • 5
  • 6
  • 7
  • 8
  • 9

Integer 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方法)。