哪个更好，Stream还是Iterator[关闭]

Question

想改进这个问题吗？更新问题，以便通过编辑这篇文章用事实和引用来回答。

我在玩Java*流API，在Lagecy系统中有以下代码：

Iterator itr = configMap.entrySet().iterator();
        List<String> resultList = new ArrayList<String>();
        while(itr.hasNext()){
            Entry e = (Entry)itr.next();
            Config rc = (Config)e.getValue();

            if (rc != null && rc.getVisibleTo() != null && (rc.getVisibleTo().equalsIgnoreCase("0010") || rc.getVisibleTo().equalsIgnoreCase("0111") || rc.getVisibleTo().equalsIgnoreCase("0011"))){        
                resultList .add(rc.getName());
            }
        }

我编写了与上述代码等价的流，如下所示：

List<String> resultList = configMap.entrySet()
                   .parallelStream()
                   .map(r -> r.getValue())
                   .filter(r -> r.getVisibleTo() != null)
                   .filter(r -> {return 
                              r.getVisibleTo().equalsIgnoreCase("0010")
                           || r.getVisibleTo().equalsIgnoreCase("0111")
                           || r.getVisibleTo().equalsIgnoreCase("0011");
                        })
                   .map(r -> r.getName())
                   .collect(Collectors.toList());

无论哪种方式，我都得到了想要的结果。我的问题是，在这种情况下，哪种性能方面是更好的写作方式？如果我选择其中任何一个而不是另一个，我实际上是否获得了任何价值？地图中包含大约 1000 个值。

Answer 1

首先，如果性能是你关心的问题，你应该根据事实来决定，所以测量，不要猜测（即使用JMH）。

对于简单的情况，迭代或经典循环往往比流更快。另一方面，流可以快得多，并且通常可以通过并行性进一步加速(其中工作负载很容易并行化)。在大多数情况下，这并不重要(以我的经验来看),因为要么数据集太小，要么使用它的代码性能不够关键。

关于可读性/可维护性，我倾向于使用流/功能方法更好地分离关注点。但是一个for-each循环也可以改进迭代代码。因此，没有简单的是/否答案，这取决于上下文。

Answer 2

因为您是以并行流的形式编写的，它试图多线程化并划分工作，所以考虑到您的工作有点小，您可能会看到由于管理线程的开销而导致的性能下降。这里的另一个陷阱是，由于您使用的是< code>CommonForkJoinPool(这里的默认线程池)，这不是一个很好的选择，因为阻塞任务会降低整个应用程序的性能(这虽然与您的小规模工作没有太大关系，但需要记住)。

话虽如此，如果您使用单线程配置Mapig.entrySet（）.stream（），我会说性能差异可能会稍微慢一些，但可读性远远弥补了这一点。因此，选择变成了在稍微快一点的代码或更好的可读性之间做出的决定，这完全取决于你（但我个人会选择流）。

Answer 3

我将稍微重写< code>sream示例:

List<String> resultList = configMap.entrySet()
        .parallelStream()
        .map(Map.Entry::getValue)
        .filter(Objects::nonNull) // value might be a null
        .filter(r ->
                ((Predicate<String>) "0010"::equals)
                        .or("0111"::equals) // null-save equivalent to  "0111".equals(value)
                        .or("0011"::equals)
                        .test(r.getVisibleTo())
        )
        .map(Config::getName)
        .collect(Collectors.toList());

并且会使用它。有关详细性能解释，请参见以下问题Java 8:Java 8的流与集合以及迭代器与流的性能

正如@4castle在下面的评论中所建议的那样，我们可以通过使用配置地图.values（）而不是配置地图.entrySet（）来使其更短：

List<String> resultList = configMap.values()
        .parallelStream()
        .filter(Objects::nonNull)
        .filter(r ->
                ((Predicate<String>) "0010"::equals)
                        .or("0111"::equals) // null-save equivalent to  "0111".equals(value)
                        .or("0011"::equals)
                        .test(r.getVisibleTo())
        )
        .map(Config::getName)
        .collect(Collectors.toList());