java8 스트림은이 코드를 클래스 멤버

Question

을 변경하지 않습니다.

stream::map 뒤에 어떻게이 항목이 namesSet에 추가되지 않습니까?

Set<String> namesSet; 

@Override 
public boolean isValid(List<String> input) { 
    namesSet = new HashSet<>(); 
    input.stream().forEach(item -> namesSet.add(item)); 
    return namesSet.size() == input.size(); 
} 

Answer 1

스트림 작업이 게으른 당신은 forEach 또는 collect 같은 터미널 작업을 종료하지 않는 한 실행되지 않습니다

갱신 여기 내 수정합니다.

이 경우 map 대신 forEach을 사용하십시오. map은 중간 작업입니다.

그런데, isValid 메서드가 필드를 수정하는 매우 불쾌한 부작용처럼 들립니다. 한 번 호출하여 true을 얻은 경우 두 번째 호출에서 false이 표시됩니다. 어쨌든, 나는이 방법을 이해할 수 없다. (한번 고치면) 첫 번째 호출에서는 항상 true을 반환하고 모든 후속 호출에서는 false을 반환하기 때문이다. 유일한 예외는 빈 목록입니다.이 경우 항상 true을 반환합니다.

편집 : 당신은 그냥 목록에 중복 포함되어 있는지 여부를 찾으려면

당신은 다만 수, 당신의 방법에 대해 자세히 살펴 갖는 here에서

input.stream().distinct().count() == input.size() 

Answer 2

합니다.

"중간 작업은 항상 게으르며, filter()와 같은 중간 작업을 실행하면 실제로는 필터링이 수행되지 않고 통과 할 때 초기 요소를 포함하는 새로운 스트림이 생성됩니다 파이프 라인 소스의 순회는 파이프 라인의 터미널 작업이 실행될 때까지 시작되지 않습니다. "

그래서 터미널 작업을 호출해야합니다.

Answer 3

문제를 해결하기 위해 Stream API를 사용하면 이 필요하지 않습니다.

그냥 당신이 List와 비교 크기 (HashSet의 copy constructor 사용)의 내용으로 Set을 만듭니다

@Override 
public boolean isValid(List<String> input) { 
    Set<String> namesSet = new HashSet<>(input); // copies the content of 'input' 
    return namesSet.size() == input.size(); 
}