List里有可能存String类型元素吗?

这其实是我遇到的一个线上bug,在这里分享给大家。

如果是用反射,那就很简单了,毕竟泛型只是在编译期进行约束,对运行期是无能为力的。

想想看,如果不使用反射,有没有办法做到呢?

问题起因

在我们公司的实际业务中,有一段类似于这样逻辑的代码,文章最后会放出做测试构造的getList()方法:

    /**
     * 主要业务逻辑
     */
    public static void main(String[] args) {
        // 从数据库查询数据列表,不用关注里面的实现细节
        List<DataBO> list = getList();

        // 获取所有“a”字段的值的集合
        List<Integer> integerList = toList(list, "a");

        if (integerList.contains(1)) {
            System.out.println("集合里包含1,处理对应的逻辑");
        } else {
            System.out.println("集合里不包含1,处理对应的逻辑");
        }
    }

    /**
     * 这是公司提供的一个公共工具方法,获取集合中,每个对象的某个字段的值的集合
     *
     * @param list 数据对象集合
     * @param key 字段
     * @return 值的集合
     */
    public static <T> List<T> toList(List<DataBO> list, String key) {
        return list.stream()
                .filter(x -> x.get(key) != null)
                .map(x -> (T)x.get(key))
                .collect(Collectors.toList());
    }

其中的DataBO对象简化如下:

public class DataBO {

    /** 数据库的一条数据,key是列,value是值 */
    private Map<String, Object> map = new HashMap<>();

    public Object get(String key) {
        return map.get(key);
    }

    public void set(String key, Object value) {
        map.put(key, value);
    }

    @Override
    public String toString() {
        return "DataBO{" + "map=" + map + '}';
    }

}

原本我这里的业务需求是,取列表数据中,所有“a”字段的值出来,判断其中是否含有1。

已知数据库里“a”字段定义为int类型,并且确认了有一条数据在“a”字段上存的是1。但是代码上线一跑,出bug了。

查出来怎么就走到“不包含1”的分支里去了呢?也没有报错,难道底层服务的getList()方法有什么特殊处理,把数据库a=1的那条数据给过滤掉了吗?

问题定位

于是我加了点日志,把listintergerList的元素打印出来,看看里面到底存了什么东西。于是又上线一版,观察一看,神奇的事情出现了,里面明明有1啊??!为啥会走到下面“不包含1”的分支呢?见鬼了!

于是我只能本地debug了一下,才发现数据库查到的集合里,“a”字段返回的是个字符串”1″!而ArrayList的contains()方法,底层是用equals()去比较是否存在的。”1″.equals(1),结果肯定是false,所以认为不存在。

好吧,虽然数据库的“a”字段定义为int类型,但是底层服务估计哪里有bug,把Integer类型的字段,转换成了String类型返回给上层服务了。

但转念一向,不对啊,我明明定义的是List<Integer>类型的变量,如果是这样的话,就算查出来”a”字段不是个Integer类型的值,那toList()方法也应该是抛个java.lang.ClassCastException才对,怎么可能正常往下走呢?List<Integer>变量指向的对象里,为什么会存进去一个字符串呢?为什么toList()方法的.map(x -> (T)x.get(key))这一行没有报错呢?

问题解析

问题很明显就是出在了toList()方法里,那个强制类型转换并没有生效。开头我们说了,java的泛型,只是在编译期进行约束,对运行期是无能为力的。那么我们首先就应该想到的就是java的泛型擦除机制,我们对demo类进行编译、再反编译看看。

反编译可以发现,原来toList()方法中,强制类型转换被擦除了。所以返回的其实并不是List<Integer>对象,而是List对象,没有泛型限制。很明显是这个方法有bug,其实就是泛型方法使用错误了。

toList()是公司内部提供的公共工具类方法,那么我们如果还是要做到原来这个方法的效果,该怎么修改这个方法呢?

问题修复

本来这个线上bug到这里就已经搞清楚了,如果只是要快速修复上线也很容易就能解决,把toList()方法返回的集合改成List,然后判断集合是否包含字符串”1″就行。

但我们想,如果后面又有别的同事遇到这个问题了怎么办呢,也会一脸懵逼,最好还是希望toList()方法抛出个java.lang.ClassCastException。所以我们这么修改下toList()方法,增加一个参数,告诉方法你希望返回一个什么类型的值:

这样的话,如果toList()方法还是返回原来的List<Integer>,就会抛异常:

而且如果前后限制的类型不一致,编译期也会报错,泛型就起作用了:

到此这个问题彻底解决。

本文用于测试构造的getList()方法:

    /**
     * 查数据库,获取数据对象的集合
     *
     * @return 数据对象的集合
     */
    public static List<DataBO> getList() {
        // 这个list是从数据库查出来的
        List<DataBO> list = new ArrayList<>();
        DataBO db1 = new DataBO();
        db1.set("a", "1");
        DataBO db2 = new DataBO();
        db2.set("a", 2);
        list.add(db1);
        list.add(db2);
        return list;
    }
Tags: