踩坑笔记之Integer数值比较

很多东西，表面上看起来挺简单，实际上别有洞天，一不小心就是一个坑。记录一下昨天遇到的Integer数值比较所遇到的“奇葩BUG”。

1.问题场景

先看如下一段代码

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

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));
    }

输出结果为：
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);
    }

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

昨天解决完问题，突然灵光一现，我嚓，这是多有趣的一道面试题啊，下次面试别人时，我得问问，哈哈！