java - 如何在没有重复操作开销的情况下进行过滤和映射

Question

在某些情况下，使用 Java 8 Stream 让我重复执行某些操作，如果没有 Stream 就可以避免，但我认为问题不在于流，而在于我。

一些例子:

private class Item {
    String id;
    List<String> strings;
}

// This method, filters only the Items that have the strToFind, and
// then maps it to a new string, that has the id and the str found
private void doIt(List<Item> items, String strToFind) {
    items.stream().filter(item -> {
        return item.strings.stream().anyMatch(str -> this.operation(str, strToFind));
    }).map(item -> {
        return item.id + "-" + item.strings.stream()
            .filter(str -> this.operation(str, strToFind)).findAny().get();
    });
}

// This operation can have a lot of overhead, therefore
// it would be really bad to apply it twice
private boolean operation(String str, String strToFind) {
    return str.equals(strToFind);
}

如您所见，函数 operation 被每个项目调用两次，我不希望这样。我首先想到的是直接映射并在找不到时返回“null”，然后过滤空值，但如果这样做，我将失去对 Item 的引用，因此无法使用 id。

Answer 1

我想你可能想要这种行为:

items.stream().map(item -> {
        Optional<String> optional = item.strings.stream().filter(string -> operation(string, strToFind)).findAny();
        if(optional.isPresent()){
            return item.id + "-" + optional.get();
        }
        return null;
    }).filter(e -> e != null);

编辑:因为当你之后做 map 时，你会丢失过滤器中获得的信息，但没有什么能阻止你只在 map 中进行操作，然后再过滤。

编辑 2:正如@Jorn Vernee 指出的那样，您可以进一步缩短它:

private void doIt(List<Item> items, String strToFind) {
    items.stream().map(item -> item.strings.stream().filter(string -> operation(string, strToFind)).findAny()
            .map(found -> item.id + "-" + found).orElse(null)).filter(e -> e != null);
}