guava 源码之 Lists transform的坑

google的guava提供了很方便的操作,特别是集合操作;
但今天遇到了个神坑,看了源码才搞定,这里通过源码来分享下;

经过

  • 项目里的代码将PO List转成DTO List,这个时候是通过Lists的transform方法来做的
  • 但是DTO还需要其他属性来丰富,这个时候,通过for循环将其他的一些属性设置到DTO List的对象中去
  • 结果,没设置进去
  • 开发大杀器(debug)上,断点到forfor(DTO dto : dtoList)循环里,在for循环代码块里发现一个比较奇特的现象,dto的数据变了,但dtoList里的数据没变
  • 想想没道理呀,这两货不是同一个引用吗,怎么会出现这种情况呢?
  • 然后开始怀疑人生,找了帮手一起看了下,也看不出来,后来,帮手提醒了一句:看看对象的地址是什么
  • 这个时候开始发现,debug的时候,每次鼠标放到dtoList上的时候,这个List里边的元素的对象地址一直在变
  • 找到原因了就好办了,一步一步debug,最终发现问题出在了Lists.transform上了

源码解析

下边是一个比较简单的demo

1
2
3
4
5
6
7
List<Integer> intList = Lists.newArrayList(1000000, 200000);
List<Integer> intList1 = Lists.transform(intList, new Function<Integer, Integer>() {
public Integer apply(Integer input) {
return input + 100000000;
}
});
intList1.get(0);

代码比较简单,就是对intList加工后返回intList1;
这个时候debug断点打到intList1.get(0);这行,然后用鼠标上浮的方式查看intList1里边的元素的地址
下边是两次鼠标放上去看的内存地址
Alt text
Alt text

从截图中可以看出,这个List的真实类型为TransformingRandomAccessList,这个List有什么特殊呢

1
2
3
4
5
6
7
8
9
10
11
12
13
14
15
16
17
18
19
20
21
22
23
24
25
26
27
28
29
30
31
32
33
34
35
36
37
38
39
40
41
42
43
44
45
46
47
48
49
50
51
52
private static class TransformingRandomAccessList<F, T> extends AbstractList<T>
implements RandomAccess, Serializable {
final List<F> fromList;
final Function<? super F, ? extends T> function;
TransformingRandomAccessList(List<F> fromList, Function<? super F, ? extends T> function) {
this.fromList = checkNotNull(fromList);
this.function = checkNotNull(function);
}
@Override
public void clear() {
fromList.clear();
}
@Override
public T get(int index) {
return function.apply(fromList.get(index));
}
@Override
public Iterator<T> iterator() {
return listIterator();
}
@Override
public ListIterator<T> listIterator(int index) {
return new TransformedListIterator<F, T>(fromList.listIterator(index)) {
@Override
T transform(F from) {
return function.apply(from);
}
};
}
@Override
public boolean isEmpty() {
return fromList.isEmpty();
}
@Override
public T remove(int index) {
return function.apply(fromList.remove(index));
}
@Override
public int size() {
return fromList.size();
}
private static final long serialVersionUID = 0;
}

看到源码就明白了,每次getiterator都会从新调用function.apply方法,而上边的function的apply方法是return input + 100000000;重新new了对象来返回,我在项目的代码里也是重新new了DTO来返回的;
到这里所有的都明白了;

总结

  • Lists的transform是挺好用的,但是只适合不对transform后的List的元素做丰富属性的操作;如果非要用,那么在循环List的时候,重新把元素add到新的List中,然后返回新的List,但这种做法太别扭了;transform的方法还是考虑场景慎用吧
  • 不知道这是guava故意这么做的还是真的是个坑,个人还没有get到这么做的意图
坚持原创技术分享,您的支持将鼓励我继续创作!