foreachをラムダ式に変更する

Question

foreachをラムダ式に変更したかったのです。私はそれを正しかったか？

Map<String, String> countriesToChooseAsMainCountry = new LinkedHashMap<String, String>(); 

のForeach：

for (Country country : project.getCountries()) { 
    countriesToChooseAsMainCountry.put(Long.toString(country.getId()), country.getName()); 
} 

ラムダ：私はそれが正しい行った場合

project.getCountries() 
     .forEach(country -> countriesToChooseAsMainCountry.put(Long.toString(country.getId()), country.getName())); 

、私は何とかこれの可読性を向上させることができますか？ありがとう。

Answer 1

としてあなたはCollectors.toMapを使用することができます：それは罰金だあなたの現在の実装（要件）といえforEachの非決定的な振る舞いを防ぎます

project.getCountries() 
     .stream() 
     .collect(Collectors.toMap(country -> Long.toString(country.getId()), // key 
           Country::getName, //value 
           (a, b) -> b, // function to determine the value in case of same country ids (which is essentially overriding the value) 
           LinkedHashMap::new)); 

。