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 같이

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)); 

그것은 당신의 현재 구현 (요구 사항)와 비록 것은 잘 보인다 forEach의 결정적 행동에서 당신을 방지 할 수 있습니다.