Java 스트림이 sideeffect로 매핑되고 결과 목록을 수집하거나 foreach하고 채 웁니다.

Question

다음과 같은 코드 조각이 있습니다.

나는 이와 관련하여 두 가지 모순되는 (?) "규칙"을 읽었습니다.

• .map 부작용이없는 것이
• 그 .foreach해야하지 갱신 변경 가능한 변수 (I foreach는 사용하고 를 결과 목록을 채우는 리팩토링 경우 그래서, 그 고장이) http://files.zeroturnaround.com/pdf/zt_java8_streams_cheat_sheet.pdf
에서 언급 한 바와 같이

스트림을 사용하고 목록을 반환하기 위해 어떻게 해결할 수 있습니까? 아니면 단순히 스트림을 건너 뛸까요?

@Transactional 
public Collection<Thing> save(Collection<Thing> things) { 
    return things.stream().map(this::save).collect(Collectors.toList()); 
} 

@Transactional 
public Thing save(Thing thing) { 
    // org.springframework.data.repository.CrudRepository.save 
    // Saves a given entity. Use the returned instance for further operations as the save operation might have changed the entity instance completely. 
    Thing saved = thingRepo.save(thing); 
    return saved; 
} 

Answer 1

해당 종이가 가변 상태라고 말하지 않습니까? 귀하의 경우에는 메서드 내에서 목록을 선언 한 다음 forEach을 사용하면 모든 것이 정상입니다. 두 번째 대답 here에는 수행하려는 작업이 정확히 나와 있습니다.

Answer 2

는 전혀 변이하지 않는 경우 List 완전히 새로운를 수집하기 위해 아무 이유없이 거의있다. 게다가 사용 사례는 기본적으로 컬렉션의 모든 요소를 ​​반복하고 for-each을 사용하여 간단히 달성 할 수있는 것을 저장합니다. 어떤 이유로 hingRepo.save(thing) 당신은 여전히 ​​같은 모음을 반환 할 수 있습니다 개체를 변이합니다,하지만이 시점에서이 hingRepo.save(thing) 그 제안하지 않기 때문에 전혀 명확하게 보이지 않는 숨겨진 돌연변이가 있다면

.