2018-02-06 guava Lists.transform的坑

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

    public static void main(String[] args) {
        List ints = Lists.newArrayList(1,2,3,4,5);
        final List waitToProcess = Lists.newArrayList();
        List collected = Lists.transform(ints, new Function() {
            @Override
            public Integer apply(Integer input) {
                if (input.equals(2)) {
                    waitToProcess.add(input);
                }
                return input;
            }
        });
        System.out.println(collected.hashCode());
        Iterator 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循环。