在For each中使用Lambda和Streams并返回结果

Question

我最近一直在学习lambda和streams，很早就被抛到了最深处。

我目前有一个书籍的数组列表，用户键入一个单词，如果这个单词等于书籍作者或标题，那么会调用并返回books toString（书籍的所有属性的格式都很好）。没有lambda很容易。但对于lambda，我似乎不知道如何才能把这一切都解决掉。

此外，使用lambda，我必须过滤掉数组中状态为“已损坏”或“已删除”的所有书籍。

问题是，我必须返回流的结果以最终显示在gui中，但似乎无法返回流的任何值。

我有一个谓词，它将尝试进行输入参数的匹配。我不知道这是否正确，我已经厌倦了尝试。

我只是想知道我需要做些什么必要的改变才能让它正常工作？

public String getBookByTitleOrAuthor(String titleOrAuthor) {
    books.stream()
         .filter(BookPredicate.matchTitleOrAuthor(titleOrAuthor))
         .filter(returnedBook -> returnedBook.getBookStatus() != 
            Book.bookStatus.Damaged && returnedBook.getBookStatus() != 
            Book.bookStatus.Deleted)
         .forEach(returnedBook -> returnedBook.toString());

}


// My predicate
public static Predicate<Book> matchTitleOrAuthor(String titleOrAuthor) {
    return b -> titleOrAuthor.equals(b.getTitle()) || 
    titleOrAuthor.equals(b.getAuthor());
}

提前谢谢你们！如果这是一个愚蠢的问题，对不起。

我的图书状态枚举和getBookstatus：

 public bookStatus getBookStatus() {
         return this.bookStatus;
    }

      public enum bookStatus {
       Available,
       Reserved,
       Borrowed,
       Damaged,
       Deleted

     }

Answer 1

我们可以用很多方法做到这一点。我们应该在这里编写干净的代码，尽量避免在单个过滤器中使用多个加法条件，并将计算量大的过滤器移到较低的阶。其次，您将在此处获得匹配书籍的列表，如：

public List<String> getBookByTitleOrAuthor(String titleOrAuthor) {
        return books
            .stream()
            .filter(returnedBook.getBookStatus() != Book.bookStatus.Damaged)
            .filter(returnedBook.getBookStatus() != Book.bookStatus.Deleted)
            .filter(matchTitleOrAuthor(titleOrAuthor))
            .map(Book::toString)
            .collect(Collectors.toList());

}

如果只需要第一个匹配项，请使用以下选项：

public String getBookByTitleOrAuthor(String titleOrAuthor) {
        return books
            .stream()
            .filter(returnedBook.getBookStatus() != Book.bookStatus.Damaged)
            .filter(returnedBook.getBookStatus() != Book.bookStatus.Deleted)
            .filter(matchTitleOrAuthor(titleOrAuthor))
            .map(Book::toString)
            .findFirst()
            .orElse(null);
}

如果您想要书籍列表，则无需地图：

public List<Book> getBookByTitleOrAuthor(String titleOrAuthor) {
        return books
            .stream()
            .filter(returnedBook.getBookStatus() != Book.bookStatus.Damaged)
            .filter(returnedBook.getBookStatus() != Book.bookStatus.Deleted)
            .filter(matchTitleOrAuthor(titleOrAuthor))
            .collect(Collectors.toList());

}

如果您在这里看到，代码更干净、易于阅读，并且匹配过滤器仅在前两个为真损坏和删除时应用。

Answer 2

forEach返回void，而您需要字符串。不过，我不会选择字符串。该方法表示“我按标题或作者返回一本书”，因此用户希望看到一个图书实例。

public Book getBookByTitleOrAuthor(String titleOrAuthor) {
    return books.stream()
                .filter(BookPredicate.matchTitleOrAuthor(titleOrAuthor))
                .filter(b -> {
                               final Book.bookStatus status = b.getBookStatus();
                               return status != null && 
                                      status != Book.bookStatus.Damaged && 
                                      status != Book.bookStatus.Deleted; 
                })  // can be moved into a BookPredicate method as well
                .findAny()
                .orElseThrow(() -> new IllegalArgumentException("There is no book for the given author or title."));
}