使用Java8 streams API和lambdas操作映射

Question

Map<Integer, Integer> allHighScoresPerLevel=highScores.get(levelId);
Set<Map.Entry<Integer, Integer>> sortedHighScores=allHighScoresPerLevel.entrySet()
               .stream().limit(Configuration.MAX_HIGHSCORES_DISPLAYED)
               .map((e)->e).sorted((o1,o2)->(o1.getValue().compareTo(o2.getValue())))
               .collect(Collectors.toCollection(TreeSet::new));

java.util.concurrent.ConcurrentHashMap$MapEntry cannot be cast to java.lang.Comparable

1）做这件事的正确方法是什么？我做错了什么？

2）流是线程安全的吗？我正在多线程环境中工作，所以我是否应该使用返回TRAAD安全集合的收集器？

Answer 1

您把事情弄得过于复杂了：除非对象具有可比性（Map.Entrys不具有可比性），否则TreeSet需要一个比较器，并且排序将由TreeSet完成。

这样（使用静态导入）：

Set<Entry<Integer, Integer>> sortedHighScores =  allHighScoresPerLevel.entrySet().stream()
            .limit(Configuration.MAX_HIGHSCORES_DISPLAYED)
            .collect(toCollection(() -> new TreeSet<> (Entry.comparingByValue())));

您没有提供足够的信息来知道代码是否是线程安全的-但是如果原始映射是线程安全的，那么您就可以了。