2018-02-06 guava Lists.transform的坑

由于忙于写业务逻辑,平时也没怎么注意,一直在使用guava的Lists.transform,用来int和long的互转等等。

    public static void main(String[] args) {
        List<Integer> ints = Lists.newArrayList(1,2,3,4,5);
        final List<Integer> waitToProcess = Lists.newArrayList();
        List<Integer> collected = Lists.transform(ints, new Function<Integer, Integer>() {
            @Override
            public Integer apply(Integer input) {
                if (input.equals(2)) {
                    waitToProcess.add(input);
                }
                return input;
            }
        });
        System.out.println(collected.hashCode());
        Iterator<Integer> iterator = collected.iterator();
        while(iterator.hasNext()) {
            iterator.next();
        }
        System.out.println(waitToProcess.size());
    }

输出的结果是令人疑惑的。waitToProcess = 2。
因为Collected被访问了两次它的apply Function

后来在tranform里面,写了别的代码, 把它当作for循环来使用,还自以为这个是在lambda表达式出现之前的一种比较优雅的写法。

这个写法后来出现线上的BUG。创建东西的时候,有时候创建了上百条重复的数据。
这是一个模拟的复现的代码。

虽然很低级,但是一直没有发现。

Lists.transform这段代码返回的是一个TransformingRandomAccessList
而这个List,也就是collected,你如果不访问他,apply是不会执行的。

Lists.transform保存的只是原有的列表和向新列表转化的Function,每次遍历就重新计算一次

guava中TransformingRandomAccessList的源码

    @Override  
    public T get(int index) {  
        return function.apply(fromList.get(index));  
    }  

所以要慎用这个类的方法。最好看清楚源码再使用。如果要做别的事情,不如直接使用for循环。

©著作权归作者所有,转载或内容合作请联系作者
平台声明:文章内容(如有图片或视频亦包括在内)由作者上传并发布,文章内容仅代表作者本人观点,简书系信息发布平台,仅提供信息存储服务。

推荐阅读更多精彩内容

  • Android 自定义View的各种姿势1 Activity的显示之ViewRootImpl详解 Activity...
    passiontim阅读 173,466评论 25 708
  • Lua 5.1 参考手册 by Roberto Ierusalimschy, Luiz Henrique de F...
    苏黎九歌阅读 13,907评论 0 38
  • 工厂模式类似于现实生活中的工厂可以产生大量相似的商品,去做同样的事情,实现同样的效果;这时候需要使用工厂模式。简单...
    舟渔行舟阅读 7,842评论 2 17
  • 为什么现在都想战友?因为大家都老了。为什么老了以后想战友?因为这些年经济发展了,人心浮躁了,真情少了,战友情就越显...
    深秋之喾阅读 735评论 0 0
  • 我很开心,在每天清晨时,看到你睡在我的身边。在一起的这些天,平稳,渐渐进入了感觉。当然是对我而言。前方还有许多的阻...
    木可来了阅读 204评论 0 0