java-8 - Java 8 -> Collectors.toMap -> 重复键

Question

我正在尝试将可用 Currency 的 List 转换为 Map，要根据 Currency Numeric code 进行查找，我想获取 String代码。这是代码。

但是上面的代码抛出了以下错误，我对 Java 8 还很陌生，所以我很头疼:

Exception in thread "main" java.lang.ExceptionInInitializerError
Caused by: java.lang.IllegalStateException: Duplicate key YUM
    at java.util.stream.Collectors.lambda$throwingMerger$0(Collectors.java:133)
    at java.util.HashMap.merge(HashMap.java:1254)
    at java.util.stream.Collectors.lambda$toMap$58(Collectors.java:1320)
    at java.util.stream.ReduceOps$3ReducingSink.accept(ReduceOps.java:169)
    at java.util.HashMap$KeySpliterator.forEachRemaining(HashMap.java:1556)
    at java.util.stream.AbstractPipeline.copyInto(AbstractPipeline.java:481)
    at java.util.stream.AbstractPipeline.wrapAndCopyInto(AbstractPipeline.java:471)
    at java.util.stream.ReduceOps$ReduceOp.evaluateSequential(ReduceOps.java:708)
    at java.util.stream.AbstractPipeline.evaluate(AbstractPipeline.java:234)
    at java.util.stream.ReferencePipeline.collect(ReferencePipeline.java:499)

但是上面的代码抛出以下错误，我对 Java 8 还很陌生，所以我很头疼

public class IsoCurrencyCode {

  private static final Set<Currency> ISO_CURRENCY = Currency.getAvailableCurrencies();

  private static final Map<Integer, Currency> NUMERIC_MAP = ISO_CURRENCY.stream().collect(Collectors.toMap(Currency::getNumericCode, Function.identity()));

  public static void main(String[] args) {
    //
    Currency currency = NUMERIC_MAP.get(971);
    System.out.println(currency.getCurrencyCode());
  }
}

它应该使用代码作为键将所有货币加载到 map 中。

Answer 1

Collectors.toMap()不接受重复的键。
既然你有 Currency::getNumericCode , toMap()遇到重复键时抛出此异常。

Caused by: java.lang.IllegalStateException: Duplicate key YUM

请注意，此处的错误消息具有误导性。键是 Integer同时 YUM不是。 YUM看起来像 Currency实例就是这样。
事实上，YUM指由 Currency 处理的值之一 ( toMap() )具有重复键，而不是键值本身。这是一个 Java bug fixed in Java 9 .

要解决您的问题，请使用 Collectors.groupingBy()收集到 Map<Integer, List<Currency>>在这种情况下，您可以通过键获得多个值，或者将重复键与 toMap() 合并重载，例如保留最后一个条目:

private static final Map<Integer, Currency> NUMERIC_MAP = 
ISO_CURRENCY.stream()
            .collect(Collectors.toMap(Currency::getNumericCode, Function.identity(), (v1, v2)-> v2);

---

要回答您的评论，您可以通过这种方式找到罪魁祸首代码(重复):

Map<Integer, List<Currency>> dupMap = 
ISO_CURRENCY.stream()
            .collect(Collectors.groupingBy(Currency::getNumericCode)
            .entrySet()
            .filter(e -> e.getValue().size() > 1)
            .collect(Collectors.toMap(Entry::getKey,Entry::getValue));

System.out.println(dupMap);