优化这些用于在Java8中创建hashmap的嵌套for循环的最佳方法是什么？

Question

在Java8中，用流替换嵌套的for循环的正确方法是什么？

好的，我正在构建一个服务，它使用另一个服务，这个服务提供了一个我无法控制的类。我必须在类出现时使用它，因为它是在每次构建项目时从文件动态生成的。它是一个完全混乱的嵌套类和嵌套列表吨。命名约定也是一团糟，但为了可读性，我在这里做了很多清理。我需要深入挖掘列表，从一个对象（下层）获得两个值（code和cgyCode）并使用一个作为键（code）和一个作为值（cgyCode）创建一个hashmap。

我有这样的功能逻辑：

//Code passing in object Rpn rpn which contains the nested lists        

HashMap<String, String> codeMap = new HashMap<String, String>();

            for (Object datDtl : rpn.getDatDtlOrRpnDtl()) {

                if (datDtl instanceof Rpn.DatDtl) {

                    for (Rpn.DatDtl.Upper upper : ((Rpn.DatDtl) datDtl).Upper()) {

                        for (Rpn.DatDtl.Upper.Lower lower : Upper.getLower()) {

                            codeMap.put(lower.getCode(), lower.getCgyCode());
                        }
                    }
                }
            }

HashMap<String, String> codeMap = new HashMap<String, String>();
                for (Object datDtl : rpn.getDatDtlOrRpnDtl()) {

                    if (datDtl instanceof Rpn.DatDtl) {

                        for (Rpn.DatDtl.Upper upper : ((Rpn.DatDtl) datDtl).Upper()){

                             upper.getLower().stream().forEach(lower -> codeMap.put(lower.getCode(), lower.getCgyCode()));

              }
         }
    }

下面我将演示一些我尝试过的其他事情，但最终我无法解决映射。

失败1：

HashMap<String, String> codeMap = new HashMap<>();
rpn.getDatDtlOrRpnDtl().stream()
                    .filter(datDtl -> datDtl instanceof Rpn.DatDtl)
                    .flatMap(datDtl -> ((Rpn.DatDtl)datDtl).getUpper().stream()
                    .flatMap(upper -> upper.getLower().stream()
                    .forEach(lower -> codeMap.put(lower.getCode(), lower.getCgyCode()))));

上面的代码似乎可以工作，除了。foreach函数给出的错误是“variable(s)R类型的实例不存在，所以void符合stream”。这是在我的包含lambda的功能循环中工作的相同逻辑，所以这让我觉得我到达“更低”的方式是不正确的。

HashMap<String, String> codeMap = rpn.getDatDtlOrRpnDtl().stream()
                    .filter(datDtl -> datDtl instanceof Rpn.DatDtl)
                    .flatMap(datDtl -> ((Rpn.DatDtl)datDtl).getUpper().stream()
                    .flatMap(upper -> upper.getLower().stream()
                    .collect(Collectors.toMap(lower -> lower.getCode(), lower -> lower.getCgyCode())));

编辑：

这个解决方案让我找到了答案，但为了给其他可能遇到这个问题的人提供文档，我将对最终有效的解决方案进行格式化：

Map<String, String> codeMap = rpn.getDatDtlOrRpnDtl().stream()
                    .filter(datDtl -> datDtl instanceof Rpn.DatDtl)
                    .flatMap(datDtl -> ((Rpn.DatDtl) datDtl).Upper().stream())
                    .flatMap(upper -> upper.getLower().stream())
                    .forEach(lower -> codeMap.put(lower.getCode(), lower.getCgyCode());

Answer 1

你的两次尝试都很接近。

您的代码没有缩进以显示它，而是在失败尝试中的collect调用之后放置了一些括号。

这不是流操作的正确顺序。最后一个操作应该是调用collect。正如所写的那样，您的代码将生成流作为其结果，而不是映射。

Map<String, String> codeMap = Rpn.getDatDtlOrRpnDtl().stream()
                    .filter(datDtl -> datDtl instanceof Rpn.DatDtl)
                    .flatMap(datDtl -> ((Rpn.DatDtl) datDtl).Upper().stream())
                    .flatMap(upper -> upper.getLower().stream())
                    .collect(Collectors.toMap(lower -> lower.getCode(), lower -> lower.getCgyCode());