Question

我有这段代码，我想返回一个postCodes列表：

List<String> postcodes = new ArrayList<>();
List<Entry> entries = x.getEntry(); //getEntry() returns a list of Entry class
for (Entry entry : entries) {
    if (entry != null) {
       Properties properties = entry.getContent().getProperties();
       postcodes.addAll(Arrays.asList(properties.getPostcodes().split(",")));
   }
} 
return postcodes;

这是我尝试使用stream（）方法和以下链式方法：

...some other block of code
List<Entry> entries = x.getEntry.stream()
    .filter(entry -> recordEntry != null)
    .flatMap(entry -> {
        Properties properties = recordEntry.getContent().getProperties();
        postCodes.addAll(Arrays.asList(properties.getPostcodes().split(",")));
});

Answer 1

你的代码有几个问题，即：

postCodes.addAll是一个副作用，因此您应该避免这样做，否则当代码并行执行时，您将收到n on-deterministic 结果。
flatMap需要一个流，而不是布尔值;这是您的代码目前尝试传递给flatMap的内容。
flatMap使用一个也消耗一个值并返回一个值的函数，并且考虑到你决定使用一个lambda语句块，那么你必须在lambda语句块中包含一个return语句，指定值返回。在您的代码中不是这种情况。
流管道由终端操作驱动，这些操作将流转换为非流值，而当前的代码根本不会执行< / em>因为你刚刚设置了这些成分，但实际上并没有从流中询问结果。

您的查询的接收方类型应为List<String>而不是List<Entry>，因为在您当前的代码中，对Arrays.asList(properties.getPostcodes().split(","))的调用会返回List<String>，然后您将其添加到累加器中电话addAll。

感谢Holger指出它，你经常无法确定变量是否被命名为entry或recordEntry。

这就是说我如何重写你的代码：

List<String> entries = x.getEntry.stream() .filter(Objects::nonNull) .map(Entry::getContent) .map(Content::getProperties) .map(Properties::getPostcodes‌) .flatMap(Pattern.co‌mpile(",")::splitAsS‌tream) .collect(Collectors.toList());

如果认为合适，您可能希望使用Collectors.toCollection指定返回列表的特定实现。

修改
通过shmosel的一些好建议，我们实际上可以在整个流管道中使用方法引用，从而实现更好的代码意图，并且更容易理解。

或者您可以继续这种方法：

List<String> entries = x.getEntry.stream() .filter(e -> e != null) .flatMap(e -> Arrays.asList( e.getContent().getProperties().getPostcodes().split(",")).stream() ) .collect(Collectors.toList());

如果你觉得更舒服。

Java 8使用stream，flatMap和lambda

1 个答案: