查找使用Java8 Streams，地图，筛选，减少名字和姓氏的演员工作过的电影

Question

我正在尝试使用Java8流API，并希望使用Java8流过滤器map reduce转换以下方法。

我有一个电影列表，每个电影对象都有一个演员列表以及其他字段。

我想找到所有有特定名字和姓氏的演员在其中工作过的电影。

下面的方法Java7，我循环播放电影列表，然后循环播放该电影的演员列表。如果找到了具有该名字和姓氏的演员，我将打破内部循环，并将该电影添加到返回的电影列表中。

注释代码有效，我可以得到正确的电影列表。

我的问题是我怎么能重写这个代码使用Java8流。我可以看到这是一个映射，过滤，减少问题，但我不能想出一个明确的解决方案。

public List<Movie> getMoviesForActor(String firstName, String lastName) {

    final List<Movie> allMovies = movieRepository.getAllMovies();
    final Predicate<Actor> firstNamePredicate = actor -> actor.getFirstName().equalsIgnoreCase(firstName);
    final Predicate<Actor> lastNamePredicate = actor -> actor.getLastName().equalsIgnoreCase(lastName);

    final List<Movie> movies = new ArrayList<>();
    //        for (Movie movie : allMovies) {
    //            boolean actorFound = false;
    //            for (Actor actor : movie.getActors()) {
    //                if(firstName.equalsIgnoreCase(actor.getFirstName()) && lastName.equalsIgnoreCase(actor.getLastName())) {
    //                    actorFound = true;
    //                    break;
    //                }
    //            }
    //            if(actorFound) {
    //                movies.add(movie);
    //            }
    //        }

    final List<Actor> actors = allMovies.stream()
            .flatMap(
                    movie -> movie.getActors().stream().filter(firstNamePredicate.and(lastNamePredicate))
            ).collect(Collectors.toList());
    return movies;
}

如果我对电影进行流式处理，并对其进行平面映射，然后在流式处理演员列表时，我如何再次获得只有这个名演员和姓氏演员的电影列表？

Answer 1

用现有代码编写它的更好方法（功能）是：

final Predicate<Movie> movieIncludesActor = movie -> movie.getActors()
        .stream()
        .anyMatch(firstNamePredicate.and(lastNamePredicate)); // check both the condition for all actors
final List<Movie> movies = allMovies.stream()
        .filter(movieIncludesActor) // movie which has such an actor
        .collect(toList());

Answer 2

使用Java8中的anyMatch短路终端操作，可以很容易地找到第一个匹配元素，同时在iterable上循环并在找到它后断开。然后将anyMatch的结果传递给filter操作符，以获得与给定条件匹配的所有电影。

我更建议您使用内联谓词，而不是单独定义它们，除非您在其他地方重用它们。这导致代码更加简洁，不那么冗长。看起来是这样的。

movies.stream()
    .filter(m -> m.getActors().stream()
        .anyMatch(
            a -> a.getFirstName().equalsIgnoreCase(firstName) 
                && a.getLastName().equalsIgnoreCase(lastName)))
    .collect(Collectors.toList());

出于某种原因，如果你真的需要使用问题陈述中给出的预定义谓词，你可以这样做，

movies.stream()
    .filter(m -> m.getActors().stream()
        .anyMatch(firstNamePredicate.and(lastNamePredicate)))
    .collect(Collectors.toList());

Answer 3

由于其他答案已经解决了在java-8中解决问题的方法，因此使用此解决方案，您可以使用全新的收集器。过滤在java-9中介绍。所以就把它留在这里，以备将来参考。

List<Movie> movies = allMovies.stream()
                .collect(Collectors.filtering(
                      m -> m.getActors().stream().anyMatch(firstNamePredicate.and(lastNamePredicate)),
                Collectors.toList()));