使用编写for循环。地图（）

Question

我有这个代码，它是用for循环实现的。我想用一个字母来写。流和。map（）函数。我试着用这个。map（）函数。但不幸的是，我得到了以下错误：

不兼容的类型。必填列表

这里是旧代码：

 public Iterable<Record> findAll(final List<Long> id) {
        final List<Record> result = new LinkedList<Record>();
        final List<List<Long>> partitions = ListUtils.partition(id, 10);
        for (List<Long> partition : partitions) {
            Iterables.addAll(
                    result,
                    this.repository.findAll(partition) 
            );
        }
        return result;
    }

这是我使用。地图（）

   public Iterable<Record> findAll(final List<Long> id) {
        final List<Record> result = new LinkedList<Record>();
        final List<List<Long>> partitions = ListUtils.partition(id, 10);

         List<List<Long>> allPartitions = partitions.stream().map(partition ->{
            return Iterables.addAll(result, this.repository.findAll(partition));
        }).collect(Collectors.toList());

        return result;
    }

有没有关于我如何解决这个问题的建议？或者我应该注意什么？

Answer 1

我想你可以用forEach重放for循环：

public Iterable<Record> findAll(final List<Long> id) {
    final List<Record> result = new LinkedList<Record>();
    final List<List<Long>> partitions = ListUtils.partition(id, 10);
    partitions.forEach(partition -> Iterables.addAll(result, this.repository.findAll(partition)));
    return result;
}

Answer 2

假设this.repository.findAll（分区）返回Iterable

public Iterable<Record> findAll(final List<Long> id) {
    return ListUtils.partition(id, 10).stream()
        .flatMap(partition -> StreamSupport.stream(
                this.repository.findAll(partition).spliterator(), false))
        .collect(Collectors.toList());
}

只有当findAll返回Iterable而不是Collection时，才需要这种复杂的Stream构造。如果它返回Collection，您可以简单地使用：

public Iterable<Record> findAll(final List<Long> id) {
    return ListUtils.partition(id, 10).stream()
        .flatMap(partition -> this.repository.findAll(partition).stream())
        .collect(Collectors.toList());
}

在这方面，你不应该给你的方法的调用方带来同样的负担，并考虑从“代码>迭代中改变返回类型。

另一个需要考虑的问题是，当您检索所有记录时，在这些小块中执行此操作是否真的有任何好处。
那么为什么不直接使用返回this.repository.findAll（id）；...