在 Java 流中执行简化操作时出现异常 [重复]

Question

我是新手Java8，在下面的示例中，我创建了一个Map，其中keyvalue为String，value为ArrayList的整数。

Map<String,List<Integer>> mapLstInteger=new HashMap<String,List<Integer>>() {
            {
                put("A",Arrays.asList(1,2,3));
                put("B",Arrays.asList(4,5,6));
                put("C",Arrays.asList(7,8,9));
            }
        };

我编写了以下代码来针对每个键执行数组列表元素的总和，并尝试将总和值存储在单独的 ArrayList 中。

List<Integer> sumLst=mapLstInteger.entrySet().stream().map(e->e.getValue())
        .reduce((inputLst, outputLst)->{
            int sum=0;
            for(int count=0;count<inputLst.size();count++)
            {
                sum=sum+inputLst.get(count);
            }
            outputLst.add(sum);
            return outputLst;
        }).get();

当我尝试执行以下代码时，我收到以下异常。

线程“main”中出现异常Java . lang . unsupportedoperationexception Java . util . abstract list . add(abstract list . Java:148)com . calculation . sum . client . client . lambda＄1(client . Java:43)Java . util . stream . reduce ops＄2 reducing sink . accept(reduce ops . Java:123)Java . util . stream . reference pipeline＄3＄1 . accept

谁能让我知道我在上面的代码中做错了什么

Answer 1

您应该执行以下操作:

    mapLstInteger.values().stream()
                 .flatMapToInt(list -> list.stream()
                                           .filter(Objects::nonNull)
                                           .mapToInt(Integer::intValue)).sum();

添加了过滤器，以确保在空整数的情况下不会得到空指针。一般来说，如果你被迫在一个流中使用一个传统的循环，你可能做错了。通过将整型列表转换为整型值，我们可以很容易地进行求和，如上所示。

最初误解了这个问题，以为你想要总和，唉，这里有一个针对实际问题的更新解决方案：

    mapLstInteger.values().stream()
                 .map(list -> list.stream()
                                  .filter(Objects::nonNull)
                                  .mapToInt(Integer::intValue).sum())
                                  .collect(Collectors.toList());

Answer 2

发生这种情况是因为您使用的是由数组.asList 生成的原始抽象列表。
该列表

public void add(int index, E element) {
    throw new UnsupportedOperationException();
}

但不管怎样，回到你的问题上。您也可以通过自定义的< code >收集器获得您想要的内容，在这里您可以提供您的自定义< code >列表

mapLstInteger.values()
             .stream()
             .collect(Collector.of(
                     () -> new ArrayList<>(),  // Supplier
                     (output, toSumList) -> {  // Accumulator
                         output.add(toSumList.stream()
                                             .mapToInt(Integer::intValue)
                                             .sum());
                     },
                     // The Combiner implementation will be called
                     // in case of a "parallel" Stream. 
                     // No need to worry about it here. 
                     // But in case, we would need to merge the partial results
                     (output, partial) -> {
                        output.addAll(partial);
                        return output;
                     }
             ));

更简洁的版本是

mapLstInteger.values()
             .stream()
             .map(l -> l.stream().mapToInt(Integer::intValue).sum())
             .collect(Collectors.toCollection(ArrayList::new));

这将正确输出 [6， 15， 24]

Answer 3

首先，您使用的Arras：：asList记录为返回由指定数组支持的固定大小的列表，我认为固定大小应该告诉您您做错了什么。

而不是你正在使用一种创建哈希地图的反模式 - 通过创建一个匿名的内部类，通过该地图扩展哈希地图

那么，您就违反了< code>reduce的规范，它应该总是返回一个新的对象，但是您总是放入< code>outputLst。

然后，当您只关心它的值时，您正在创建一个< code>Map，创建一个< code>List

甚至你的句子我在下面写的代码中对每个键执行 arrayList 元素的总和，根据你的代码是不正确的。我会决定我想要实现的实际目标，然后如果我是你，我会尝试去做。