重构if-else块以使用Map.ComputeIfAstant Map.ComputeIfPresent错误结果

Question

我有课

public class Gezana {
    private String name;
    private long value;
}

和列表,如：

List<Gezana> list = List.of(new Gezana("foo", 2), 
                            new Gezana("foo", 2),
                            new Gezana("foo", 2),
                            new Gezana("foo", 2), 
                            new Gezana("bar", 5),
                            new Gezana("bar", 5), 
                            new Gezana("bar", 5), 
                            new Gezana("doo", 9), 
                            new Gezana("doo", 9),
                            new Gezana("kee", 12));

我需要创建一个地图使用上面的列表名称作为键和值的总和作为值。我以前是这样做的：

Map<String, Long> oldMap = new HashMap<>(); 
for (Gezana gez : list) {            
    if (oldMap.containsKey(gez.getName())) {
        oldMap.put(gez.getName(), oldMap.get(gez.getName()) + gez.getValue());
    } else{
        oldMap.put(gez.getName(), gez.getValue());
    }          
}        
System.out.println("old map:" + oldMap);

我需要重构上面的内容，并考虑使用Map.ComputeIfAstent&Map.ComputeIfPresent，但没有得到预期的结果

Map<String, Long> map = new HashMap<>(); 
for (Gezana gez : list) {            
    map.computeIfAbsent(gez.getName(), k -> gez.getValue());     
    map.computeIfPresent(gez.getName(), (k, v)->  v + gez.getValue());              
}
System.out.println("new map:" + map);

输出：

old map:{bar=15, doo=18, foo=8, kee=12}
new map:{bar=20, doo=27, foo=10, kee=24}

看起来新的映射对于每添加一个键都有一个附加值，但是我看不出使用ComputeIfAstent&computeIfPresent方法有什么错误。有什么提示吗？

Answer 1

最好应用map::merge函数：

 Map<String, Long> oldMap = new HashMap<>();
for (Gezana gez : list) {
    oldMap.merge(gez.getName(), gez.getValue(), Long::sum);
}

System.out.println("old map:" + oldMap);

或者使用流APICollectors.GroupingBy和Collectors.SummingLong:

Map<String, Long> newMap = list.stream()
        .collect(Collectors.groupingBy(
            Gezana::getName, 
            Collectors.summingLong(Gezana::getValue)
        ));
System.out.println("new map:" + newMap);

输出：

old map:{bar=15, doo=18, foo=8, kee=12}
new map:{bar=15, doo=18, foo=8, kee=12}

Answer 2

问题是computeif缺席总是添加到映射中；然后，computeifpresent将更新computeif缺席可能刚刚添加的内容。

因此，当第一次遇到一个名称时，它的值会被添加两次--一次由ComputeIfAstent添加，然后由ComputeIfPresent添加。

您可以使用ComputeIfAstent/ComputeIfPresent执行此操作，如下所示：

map.computeIfAbsent(gez.getName(), k -> 0L);     
map.computeIfPresent(gez.getName(), (k, v)->  v + gez.getValue());

或compute:

map.compute(gez.getName(), (k, v) -> (v != null ? v : 0L) + get.getValue());

但合并会更容易：

oldMap.merge(gez.getName(), gez.getValue(), Long::sum);

或者，直接从列表中：

Map<String, Long> map = 
    list.stream().collect(groupingBy(Gezana::getName, summingLong(Gezana::getValue)))