Long类型比较遇到的坑

首先来看下这个例子：

public class LongCompareTest {
    public static void main(String[] args) {
        Long userId = 256L;
        Long createBy = 256L;
        if (userId == createBy){
            System.out.println("用户id相等，是同一用户，允许操作。。。");
        }else {
            System.out.println("用户id不相等，不是同一用户，无权限操作");
        }
    }

}

运行结果如下：

用户id不相等，不是同一用户，无权限操作

奇怪，为什么会不相等？我们来看下源码：

public static Long valueOf(long l) {
        final int offset = 128;
        if (l >= -128 && l <= 127) { // will cache
            return LongCache.cache[(int)l + offset];
        }
        return new Long(l);
    }

源码中显示，Long中有一个静态的内部类LongCache，专门用于缓存-128至127之间的值，一共256个元素。

如果值在[-128, 127]之间，会放在缓存里面，而超过这个范围就要new一个新的对象，也就是说==不能判断对象是否相等。当然，如果值是在[-128, 127]之间的话是测不出来什么问题的。

解决方案：

可以使用.longValue() 或者…equals()进行比较。

public class LongCompareTest {
    public static void main(String[] args) {
        Long userId = 256L;
        Long createBy = 256L;

//        if (userId == createBy){
//            System.out.println("用户id相等，是同一用户，允许操作。。。");
//        }else {
//            System.out.println("用户id不相等，不是同一用户，无权限操作");
//        }
        if (userId.longValue() == createBy.longValue()){
            System.out.println("用户id相等，是同一用户，允许操作。。。");
        }else {
            System.out.println("用户id不相等，不是同一用户，无权限操作");
        }


    }

}

public class LongCompareTest {
    public static void main(String[] args) {
        Long userId = 256L;
        Long createBy = 256L;

//        if (userId == createBy){
//            System.out.println("用户id相等，是同一用户，允许操作。。。");
//        }else {
//            System.out.println("用户id不相等，不是同一用户，无权限操作");
//        }
//        if (userId.longValue() == createBy.longValue()){
//            System.out.println("用户id相等，是同一用户，允许操作。。。");
//        }else {
//            System.out.println("用户id不相等，不是同一用户，无权限操作");
//        }

        if (userId.equals(createBy)){
            System.out.println("用户id相等，是同一用户，允许操作。。。");
        }else {
            System.out.println("用户id不相等，不是同一用户，无权限操作");
        }
    }

}

运行结果都如下：

用户id相等，是同一用户，允许操作。。。

最佳解决方案：用.eques()
看下eques()源码：

 public boolean equals(Object obj) {
        if (obj instanceof Long) {
            return value == ((Long)obj).longValue();
        }
        return false;
    }

它是先通过.longValue()方法获取Long对象的基本类型long的值之后再做比较的。我们用longValue会比较容易忘记后面一个包装类调longValue方法。

所以所我们最好用.eques()方法，当然也可以用.longValue()的！