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的那条数据给过滤掉了吗?

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

问题定位

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

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

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

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

好吧,虽然数据库的“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类进行编译、再反编译看看。

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

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

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

问题修复

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

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

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

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

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

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

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

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

到此这个问题彻底解决。

本文用于测试构造的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;
}
上一篇:python网络编程--线程Semaphore(信号量)


下一篇:Microsoft SQL Server Express各版本对比