头图

线上问题年年有,今年特别多。记几次线上惨痛的踩坑记录,希望大家以史为鉴。


1. 包装类型自动解箱导致空指针异常

public int getId() {
    Integer id = null;
    return id;
}

如果调用上面的方法会发生什么?id是Integer类型,而方法的返回值int类型,会自动拆箱转换,由于id是null,转换成int类型的时候,就会报NullPointerException异常。

无论是《阿里Java开发手册》、《代码整洁之道》还是《Effective Java》都建议方法返回值类型尽量写成包装类型,类似Integer。还有实体类、接收前端传参类、给前端的响应类中的属性都要写成包装类型,避免拆箱出错。

2. 包装类型用==判断相等,导致判断不正确

先看一段代码运行结果:

public class IntegerTest {
    public static void main(String[] args) {
        Integer a = 100;
        Integer b = 100;
        Integer c = 200;
        Integer d = 200;
        System.out.println(a == b); // 输出 true
        System.out.println(c == d); // 输出 false
    }
}

很多人会很疑惑,为什么输出的两个结果会不一样?

当给Integer类型赋值时,会调用Integer.valueOf()方法

static final int low = -128;
static final int high = 127;

public static Integer valueOf(int i) {
    if (i >= IntegerCache.low && i <= IntegerCache.high)
        return IntegerCache.cache[i + (-IntegerCache.low)];
    return new Integer(i);
}

当value值在-128到127之间时,会复用缓存。当不在这个区间时,才会创建对象。

而==比较的是内存地址,不同的对象的内存地址不相同,所以就出现上述的结果。

Integer重写了equals()方法:

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

当使用equals()方法时,比较的是int值是否相等。

所以,包装类判断是否相等的时候,绝不能用==判断,一定要用equals()方法判断。

3. Switch传参是null导致空指针异常

猜一下下面代码的运行结果:

public class Test {
    public static void main(String[] args) {
        String name = null;
        switch (name) {
            case "yideng":
                System.out.println("一灯");
                break;
            default:
                System.out.println("default");
        }
    }
}

你是不是认为会输出default,其实代码会抛出NullPointerException异常。

当switch比较两个对象是否相等的时候,会调用name.hashCode()方法和name.equals()方法,因为name是null,结果就抛出了NullPointerException异常。

所以调用switch方法前,一定要对传参进行判空。

4. 创建BigDecimal类型时精度丢失

猜一下下面代码的运行结果:

public class Test {
    public static void main(String[] args) {
        BigDecimal bigDecimal = new BigDecimal(0.1);
        System.out.println(bigDecimal);
    }
}

你以为会输出0.1,其实输出结果是:

0.1000000000000000055511151231257827021181583404541015625

What?这么一大串是什么东西?

为什么会出现这种情况呢?原因是,当我们用new BigDecimal(0.1)创建对象是,会调用BigDecimal的这个构造方法:

public BigDecimal(double val) {
    this(val,MathContext.UNLIMITED);
}

把传参0.1当成了double类型,double计算的时候会把数值转换成二进制,而0.1转换成二进制是无法除尽的,所以就带了一大串小数位。

当需要创建BigDecimal类型时,应该怎么做呢?

可以先把数值转换成字符串类型,再创建BigDecimal对象,类似这样:

BigDecimal bigDecimal = new BigDecimal(String.valueOf(0.1));

又来一个问题,BigDecimal是怎么解决精度丢失问题?

答案是BigDecimal会先把数值乘以10的整数倍,去除小数位,转换成long类型,然后进行运算,最后把运算结果除以10的整数倍。

5. group分组时主键重复,导致异常

下面代码的分组能成功吗?

public class SteamTest {
  
    static class User {
        // 用户ID
        private Integer id;
        // 用户名
        private String name;
    }

    public static void main(String[] args) {
        List<User> users = Arrays.asList(
                new User(1, "Tom"),
                new User(1, "Tony"),
                new User(2, "Jerry")
        );
         // 用户集合按id进行分组
        Map<Integer, User> userMap = users.stream()
                .collect(Collectors.toMap(User::getId, user -> user));
      System.out.println(userMap);
    }
}

结果报异常了,Exception in thread "main" java.lang.IllegalStateException: Duplicate key SteamTest.User(id=1, name=Tom)

原因是主键冲突,有两个id=1的数据,按id进行分组时程序就不知道怎么处理了。

可以这样做

public class SteamTest {
  
    static class User {
        // 用户ID
        private Integer id;
        // 用户名
        private String name;
    }

    public static void main(String[] args) {
        List<User> users = Arrays.asList(
                new User(1, "Tom"),
                new User(1, "Tony"),
                new User(2, "Jerry")
        );
         // 用户集合按id进行分组,主键冲突的时候,取第一个user
        Map<Integer, User> userMap = users.stream()
                .collect(Collectors.toMap(User::getId, user -> user, (user1, user2) -> user1));
      System.out.println(userMap); // 输出 {1:{"id":1,"name":"Tom"},2:{"id":2,"name":"Jerry"}}
    }
}

6. 真假ArrayList导致添加异常

下面的add()方法能添加成功吗?

public class Test {
    public static void main(String[] args) {
        List<Integer> list = Arrays.asList(1, 2);
        list.add(3);
    }
}

结果是抛异常了,Exception in thread "main" java.lang.UnsupportedOperationException

抛出了不支持这个方法的异常,为什么呢?我们看一下Arrays.asList()方法的源码:

public static <T> List<T> asList(T... a) {
    return new ArrayList<>(a);
}

返回了一个ArrayList,为什么还不能添加成功了?

真相是此ArrayList非彼ArrayList,跟我们常用的ArrayList只是重名,这个ArrayList只是Arrays对象一个内部类,内部并没有实现add()方法,所以添加的时候会报错。

这不是明摆着坑人吗?实现了list接口,为啥不实现add()方法?

其实作者是故意这样设计的,除了没有实现add()方法,还没有实现addAll()、remove()、clear()等修改方法,目的就是创建后再不让用户修改,这样的集合有什么用呢?

其实在某些不可变场景还是很实用的,比如已结束的订单状态集合:

List<String> list = Arrays.asList("Failure", "Cancelled","Completed");

这种集合一般不会变的,使用过程中也不允许修改,避免出错。

7. 总结

每一次踩坑,背后都有至少一次的线上问题记录,这些总结都是用教训换来的,不只是自己,其他人肯定也遇到过。我们如何才能避免在以后的开发中再出现类似的问题呢?

  • 站在使用者的角度,编写详细的单元测试,打印必要日志,追踪代码执行结果
  • 站在创造者的角度,探究框架的架构设计和源码实现,理解作者的意图

你在线上还踩过那些坑?


一灯架构
44 声望11 粉丝