2

有没有更好的方法将“Map<String, Collection<String>>”转换为“Map<String, List<String>>”?

Map<String, Collection<String>> collectionsMap = ...
Map<String, List<String>> listsaps =
    collectionsMap.entrySet().stream()
    .collect(Collectors.<Map.Entry<String, Collection<String>>,
        String, List<String>>toMap(
            Map.Entry::getKey,
            e -> e. getValue().stream().collect(Collectors.toList())
        )
    );

感谢您帮助我们改善

4

3 回答 3

4

对于这样的情况,我会考虑使用Map.forEach使用副作用来执行操作。映射上的流有点麻烦,因为需要编写额外的代码来流映射条目,然后从每个条目中提取键和值。相比之下,Map.forEach将每个键和值作为单独的参数传递给函数。看起来是这样的:

Map<String, Collection<String>> collectionsMap = ...
Map<String, List<String>> listsaps = new HashMap<>(); // pre-size if desired
collectionsMap.forEach((k, v) -> listsaps.put(k, new ArrayList<>(v)));

如果您的地图很大,您可能需要预先确定目的地的大小,以避免在其填充期间重新散列。要正确执行此操作,您必须知道HashMap存储桶的数量而不是元素的数量作为其参数。这需要除以默认的加载因子 0.75,以便在给定一定数量的元素的情况下正确地预先调整大小:

Map<String, List<String>> listsaps = new HashMap<>((int)(collectionsMap.size() / 0.75 + 1));
于 2018-02-28T19:01:32.917 回答
3

1)在Collectors.toMap()您不需要重复泛型类型,因为这些是推断出来的。

所以 :

collect(Collectors.<Map.Entry<String, Collection<String>>,
        String, List<String>>toMap(...)

可以替换为:

collect(Collectors.toMap(...)

2)将集合转换为列表的方式也可以简化。

这个 :

e -> e. getValue().stream().collect(Collectors.toList())

可以写成:

e -> new ArrayList<>(e.getValue())

你可以写:

Map<String, List<String>> listsaps =
            collectionsMap.entrySet()
            .stream()
            .collect(Collectors.toMap(
                    Map.Entry::getKey,
                    e -> new ArrayList<>(e.getValue())
                )
            );
于 2018-02-22T13:41:53.117 回答
1

我认为这更容易阅读:

Map<String, List<String>> listsaps = new HashMap<>();
collectionsMap.entrySet()
    .stream()
    .forEach(e -> listsaps.put(e.getKey(), new ArrayList<>(e.getValue())));

如果您只想将条目转换为列表但并不真正关心更改集合的类型,那么您可以使用map.replaceAll

collectionsMap.replaceAll((k, v) -> new ArrayList<>(v));
于 2018-02-22T13:37:33.253 回答