isEmpty(), NPE
157
작성한 질문수 5
private void validateDuplicateMember(Member member) {
//중복 회원 검증
List<Member> findMembers = memberRepository.findByName(member.getName());
if(!findMembers.isEmpty()){
throw new IllegalStateException("이미 존재하는 회원입니다.");
}
}중복 회원 검증 메서드에서 위의 코드는 test 돌릴 때 제대로 작동하는데 아래 코드처럼 if문의 조건을 바꾸면 test가 fail합니다..!
리스트가 null일 경우 isEmpty() 호출 시 NPE가 발생하기 때문에 가급적 null인지 여부도 함께 확인하라는 글을 본적이 있어서 아래처럼 작성한 것인데, 왜 에러가 나는 건지 궁금합니다!
private void validateDuplicateMember(Member member) {
//중복 회원 검증
List<Member> findMembers = memberRepository.findByName(member.getName());
if(!findMembers.isEmpty() || findMembers!=null){
throw new IllegalStateException("이미 존재하는 회원입니다.");
}
}
답변 1
0
안녕하세요. 최은영님, 공식 서포터즈 y2gcoder입니다.
조건에 오류가 있는 것으로 보입니다!
findMembers가 null이 아니고, findMembers 가 비어있지 않으면
을 검증해야 한다면
findMembers != null && !findMembers.isEmpty() 와 같이 null 인지를 먼저 판단하고, 리스트가 비어있지 않은지를 판단하는 게 맞고, 이를 AND 조건으로 묶어주셔야 합니다.
올려주신 코드 블록과 같이 한다면 findMembers 가 비어있거나 null일 때 해당 조건을 통과하므로 정상적인 체크를 할 수가 없습니다.
그리고 보통은 컬렉션 객체가 null인지 먼저 체크하고 그 후에 비어있는지 체크하는 것이 순서상 맞습니다:)
감사합니다.
sdk 설정 오류
0
53
2
오탈자 - @Transactional
0
56
1
src/test/resources 테스트 경로 문제
0
50
1
상품 등록후 H2 db 출력 순서 바꿀 수 있나요?
0
64
1
MemberRepositoryTest 실행오류
0
81
1
boot 4.x >>> trasasction rolled back log & p6spy(영한님, 수업 자료 업데이트 해주시면 감사하겠습니다!!)
1
184
2
강의 마지막 QueryDSL 사용 부분 질문있습니다
1
142
2
클라이언트에서 isbn과 author 수정 요청을 한 경우에 대해 질문드립니다.
0
51
1
도메인 모델 패턴 vs 트랜잭션 스크립트 패턴
0
71
1
기본 생성자
0
60
1
h2 DB 연결시 jdbc url 변경 이유가 궁금합니다.
0
103
1
멤버서비스테스트 부분에서 막힙니다.
0
165
4
실무에서도 EntityManager를 이용해서 많이 작업하는 편일까요?
0
116
1
초반에 h2 다운로드 과정 꼭 필요한가요?
0
120
2
자신 필드에도 get으로 접근하는 이유가 있을까요?
0
114
1
24분 27초 연관관계 편의 메서드 위치
0
113
1
단건 주문만 가능하게 한건 의도한 부분이신가요?
0
109
2
빌드 툴, Gradle
0
59
1
h2연결은 된 것 같은데 엔티티 테이블까지 작성 후 확인해보아도 테이블이 안보입니다
0
77
2
Repository에서 EntityManager 주입 방식 차이
0
90
1
롬복과 사용자 정의 setter 메서드
0
73
1
주문 목록 조회 fetch join 질문드립니다
0
82
1
dirty checking 질문드립니다.
0
83
1
동시성 관련 질문입니다
0
75
1





