在 Java 8 中以对象字段作为键/值从对象流中优雅地创建映射

Question

我有以下课程

class Person {
    public String name;
    public int age;
    public List<String> hobbies;

    Person(String name, int age, List<String> hobbies)
    {this.name = name; this.age = age; this.hobbies = hobbies;}
}

我如何创建一个年龄映射到诸如Map<Integer, Set<String>> 之类的爱好？

我编写的 Java 8 方法是：

Map<Integer, Set<String>> collect8 = persons.stream()
    .collect(
        toMap(
            p -> p.age,
            p -> p.hobbies.stream().collect(toSet()),
            (hobbies1, hobbies2) ->
                Stream.concat(hobbies1.stream(), hobbies2.stream()).collect(toSet())
        )
     );

有没有更惯用的方式来使用Collectors.groupingBy() 呢？

作为一个相关问题，我发现没有 Java 流的版本更具可读性。

Map<Integer, Set<String>> collect7 = new HashMap<>();

for(Person p: persons) {
    Set<String> hobbies = collect7.getOrDefault(p.age, new HashSet<>());
    hobbies.addAll(p.hobbies);
    collect7.put(p.age, hobbies);
}

如果更容易阅读，我们是否应该使用非流代码？特别是当流式版本（如此处所示）没有带有数据转换的中间流但很快以终端操作结束时？

Answer 1

查看 Java 8 Collectors 文档中的类似示例：

Map<City, Set<String>> namesByCity
     = people.stream().collect(groupingBy(Person::getCity, TreeMap::new,                                           
           mapping(Person::getLastName, toSet())));

您可以在这里使用相同的方法。

Answer 2

正如您自己指出的那样：Stream 解决方案可能不像您当前的非Stream-解决方案那样可读。使用groupingBy 解决问题可能看起来不如您预期的那么好，因为您想将List 转换为Set。

我用groupingBy、mapping 和reducing 构建了一个解决方案，但该解决方案并不那么容易阅读，甚至包含一个错误。您可以在以下位置阅读更多相关信息：Java 8 stream.collect( ... groupingBy ( ... mapping( ... reducing ))) reducing BinaryOperator-usage 我真的建议查看 Holger 给出的答案，因为它还包含一个使用自定义 Collector 的更简单的解决方案以及 Java 9 的 flatMapping 的一点 Outlook，这对我来说很接近到您的非Stream-解决方案。

但我想出的另一个使用groupingBy 的解决方案实际上是这样的：

Map<Integer, Set<String>> yourmap;
yourmap = personList.stream()
                    .flatMap(p -> p.hobbies.stream()
                                           .flatMap(hobby -> Stream.of(new SimpleEntry<>(p.age, hobby)))
                            )
                    .collect(Collectors.groupingBy(Entry::getKey,
                             Collectors.mapping(Entry::getValue, Collectors.toSet())));

为此，您需要以下导入：

import java.util.AbstractMap.SimpleEntry;
import java.util.Map.Entry;

当然你也可以选择Tuple或Pair或者你最喜欢的。

但在任何方面都不是更好。

我会继续使用您当前的非Stream-解决方案。它更具可读性，可以做它应该做的事情。