Java 8ストリームはリストアイテムを縮小してマップに結合します

Question

name1/2合計値value1/2のキーでマップを作成する必要があります。

java 8ストリームを使用してこれを書き換える最もクリーンな方法は何でしょうか？

class Item { 

    private String name1; 
    private Integer value1; 
    private String name2; 
    private Integer value2; 

    public Item(final String name1, final Integer value1, final String name2, final Integer value2) { 
     this.name1 = name1; 
     this.value1 = value1; 
     this.name2 = name2; 
     this.value2 = value2; 
    } 
    //getters and setters 
} 

List<Item> list = Lists.newArrayList(
       new Item("aaa", 1, "bbb", 2), 
       new Item("bbb", 5, "ccc", 3), 
       new Item("aaa", 8, "bbb", 7), 
       new Item("bbb", 2, "aaa", 5)); 

Map<String, Integer> map = Maps.newHashMap(); 

for (Item item : list) { 
    map.merge(item.name1, item.value1, Integer::sum); 
    map.merge(item.name2, item.value2, Integer::sum); 
} 

System.out.println(map);//{aaa=14, ccc=3, bbb=16} 

Answer 1

可能な解決策は、2つのエントリによって行われたストリームに平坦マップに各アイテムである：各エントリには、名前および対応する値で構成されます。そして、これは同じキーを持つ値の値を合計することによってマップに集められます。両方の値を保持する組み込みのペアがないため、AbstractMap.SimpleEntryを使用できます。

Map<String, Integer> map = 
    list.stream() 
     .flatMap(i -> Stream.of(new AbstractMap.SimpleEntry<>(i.name1, i.value1), new AbstractMap.SimpleEntry<>(i.name2, i.value2))) 
     .collect(Collectors.toMap(Map.Entry::getKey, Map.Entry::getValue, Integer::sum)); 

また、カスタムcollectコールを使用する方が簡単かもしれません。アキュムレータは、forループ本体と同じように、各キーをマージします。トリッキーな部分はコンバイナです。この場合、2つのマップを結合し、2つ目のマップのエントリを繰り返して最初のマップにマージします。

Map<String, Integer> map = 
     list.stream() 
      .collect(
       HashMap::new, 
       (m, i) -> { 
        m.merge(i.name1, i.value1, Integer::sum); 
        m.merge(i.name2, i.value2, Integer::sum); 
       }, 
       (m1, m2) -> m2.forEach((k, v) -> m1.merge(k, v, Integer::sum)) 
      );