问题描述

最近版本组织清理编译告警,其中有这么一条比较有意思,之前没见过,拿出来说一说

serializable class anonymous com.demo.Main$1 has no definition of serialVersionUID

编译告警指向了这段代码:

private static List<String> defaultAttrList = new ArrayList<String>() {
    {
        add(ResourceConsts.RES_NAME);
        add(ResourceConsts.RES_TYPE);
        add(ResourceConsts.RES_IP);
        add(ResourceConsts.RES_VERSION);
    }
};

乍一看好像没什么问题,我用双大括号的方式定义并初始化了一个ArrayList,往里面塞了几个值,代码简洁易懂。

但问题并没有看起来那么简单,原因就在双大括号

探究

双大括号的写法实际上创建了一个匿名类,我们将源文件编译后也会发现,生成了一个Main$1.class的文件,它就对应这个匿名类。反编译后的代码如下:

class Main$1 extends ArrayList<String> {
    Main$1() {
        this.add("1");// 10
    }// 11
}

可以看到,我们创建了一个名为Main$1的匿名类,继承自ArrayList,而ArrayList的类定义如下:

public class ArrayList<E> extends AbstractList<E> implements List<E>, RandomAccess, Cloneable, Serializable {
    // ......
}

正是因为ArrayList实现了Serializable接口,所以Main$1也需要定义serialVersionUID

解决方法

既然是由于匿名类引起的编译告警,我们可以干掉匿名类,用静态域来初始化List,像下面这样即可消除告警:

private static List<String> defaultAttrList = new ArrayList<>();
static {
    defaultAttrList.add(ResourceConsts.RES_NAME);
    defaultAttrList.add(ResourceConsts.RES_TYPE);
    defaultAttrList.add(ResourceConsts.RES_IP);
    defaultAttrList.add(ResourceConsts.RES_VERSION);
}

参考资料

  1. 永远不要使用双花括号初始化实例,否则就会OOM!
10-28 07:26