코드 질문

작성한 질문수 1

0

좋은 질문입니다.

해당 코드의 'Collectors.toList();' 는 사실상 의미가 없는 코드입니다.

원래는 아래와 같이 작성되어 있었을 가능성이 높습니다.

.collect(Collectors.toList());

이후 'Collectors.toUnmodifiableList()'로 변경하는 과정에서 실수로 'Collectors.toList();'가 남아버린 것 같습니다.

죄송합니다.

참고로,

.collect(Collectors.toList())

는 수정 가능한(Mutable) List를 반환하고,

.collect(Collectors.toUnmodifiableList())

는 수정 불가능한(Unmodifiable) List를 반환합니다.

즉, 'toUnmodifiableList()'로 생성된 List는 요소를 추가하거나 삭제하려고 하면 예외가 발생합니다.

해당 예제는 조회 결과를 반환하는 용도이므로 외부에서 실수로 데이터를 변경하지 못하도록 'Collectors.toUnmodifiableList()'를 사용하는 것이 더 적절하다고 판단하여 변경했던 것으로 기억합니다.

질문해주셔서 감사합니다. 강의 코드 작성 과정에서 남아있는 불필요한 코드였습니다.
학습 과정에서 혼동을 드리게 된 점 양해 부탁드립니다.

0

안녕하세요, 인프런 AI 인턴입니다.
피플로마님, 질문해주신 코드에 대해 설명드리겠습니다.

코드에서

List<MemberDto> admins = memberEntities.stream()
                .map(MemberEntity::toDto)
                .collect(Collectors.toUnmodifiableList());Collectors.toList();

부분을 보면, Collectors.toUnmodifiableList() 다음에 쓰여진 Collectors.toList();는 의미가 없는 코드입니다. 이미 Collectors.toUnmodifiableList()를 사용하여 Stream 결과를 변경 불가능한 리스트로 수집하고 있습니다. Collectors.toList();는 그 뒤에 독립적으로 사용될 수 없는 상태입니다.

이는 자바 구문 오류를 발생시키며, 원하지 않는 결과를 초래할 수 있습니다. 단지 코드 가독성을 떨어뜨릴 뿐만 아니라 실행 시 에러를 유발할 가능성이 큽니다. 따라서 Collectors.toList(); 부분은 제거하시는 것이 좋습니다.

추가적으로 Spring Boot 관련하여 궁금하신 점이 있다면, 공식 문서를 참조하시거나, 관련 태그로 Stack Overflow에 질문을 남겨보시는 것도 좋은 방법입니다.

도움이 되셨길 바랍니다! 👍