inflearn logo
강의

강의

N
챌린지

챌린지

멘토링

멘토링

N
클립

클립

로드맵

로드맵

지식공유

실전! 스프링 부트와 JPA 활용1 - 웹 애플리케이션 개발

회원 기능 테스트

isEmpty(), NPE

157

최은영

작성한 질문수 5

0

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("이미 존재하는 회원입니다.");
    }
}

java spring 웹앱 spring-boot jpa

답변 1

0

y2gcoder

안녕하세요. 최은영님, 공식 서포터즈 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