嵌套循环到Java流之谜

Question

我是Java8中的流的新手。我正在努力弄清楚如何在使用流的列表上实现嵌套的for循环。一个列表包含消息，消息中的一个字段是作者的id（不幸的是，不是obj引用）。另一个列表是作者，作者中的一个字段是作者ID。代码需要为消息列表中的每条消息在authors列表中找到匹配的作者。我拥有的工作Java代码（它不使用流或lambda）如下所示，我的问题是如何使用流重写它？我尝试了一些事情，我想一个平面地图和一个过滤器将是需要的，但我似乎不能得到它的正确。

public void showAuthors(List<Message> messages, List<Authors> authors) {
    String authorName=null;
    for (Message message : messages) {
        int authorId = message.getAuthorId();
        for (Authors author : authors) {
            if (author.getId() == authorId)
                authorName = author.getFullName();
                break;
        }
        System.out.println("Message id is " + message.getId() + 
                           " author is " + authorName);
    }
}

我在这里看过一些类似的问题，但我似乎就是做不对。

感谢任何帮助！

更新：作者列表的Id字段有一些空数据。这要求我在流操作中使用过滤器来创建AuthorId、name的映射。

Answer 1

您不需要流来处理消息，这里forEach更合适。尽管如此，流对于作者列表到映射转换还是很有用的。

您可以使用带有3个参数的toMap来避免authors列表中可能的authors重复（原始代码避免了这一点）。

您可以使用Optional来避免可能存在的authos缺失、null id等--每次map返回null，Optional变成Optional：：empty，因此对它的进一步处理一直跳过到终端操作，而ifPresence只是跳过空的选项。

Map<Integer, String> authorNames=authors.stream()
    .collect(Collectors.toMap(
        Author::getId,
        Author::getFullName,
        (oldValue, newValue) -> oldValue));

messages.forEach(
    m -> Optional.of(m)
        .map(Message::getAuthorId)
        .map(authorNames::get)
        .map(name -> "Message id is " + m.getId() + " author is " + name)
        .ifPresent(System.out::println)
);