findByname파트에서 질문이 있습니다.

Question

학습하는 분들께 도움이 되고, 더 좋은 답변을 드릴 수 있도록 질문전에 다음을 꼭 확인해주세요.

1. 강의 내용과 관련된 질문을 남겨주세요.
2. 인프런의 질문 게시판과 자주 하는 질문(링크)을 먼저 확인해주세요.
(자주 하는 질문 링크: https://bit.ly/3fX6ygx)
3. 질문 잘하기 메뉴얼(링크)을 먼저 읽어주세요.
(질문 잘하기 메뉴얼 링크: https://bit.ly/2UfeqCG)

질문 시에는 위 내용은 삭제하고 다음 내용을 남겨주세요.
=========================================
[질문 템플릿]
1. 강의 내용과 관련된 질문인가요? (예/아니오)
2. 인프런의 질문 게시판과 자주 하는 질문에 없는 내용인가요? (예/아니오)
3. 질문 잘하기 메뉴얼을 읽어보셨나요? (예/아니오)

[질문 내용]
여기에 질문 내용을 남겨주세요.

회원 리포지토리 테스트 케이스 작성 8:48영상을 보았는데

Member result = repository.findByName("spring1").get(); 이렇게 코드를 치는걸로 나오는데 이런방식으로 하면 값이 있는지 여부를 파악 못하고 그냥 불안전하게 값을 가져오는 방식이라는걸로 예전에 다른 강의에서 말씀해주셨던 거 같은데 기억이 애매모호해서 질문드립니다. 테스트코드라서 이런방식을 택하신건지와 정석적으로 값을 체크하려면 아래방식으로 코드를 짜는지에 대해서 궁금합니다.

Optional<Member> result = repository.findByName("spring1"); Assertions.assertThat(result) .isPresent() // Optional에 값이 있는지 확인 .get() // 값이 존재할 때만 안전하게 꺼냄 .isEqualTo(member1);

혹시 이 코드가 맞다면 Optional<Member> 대신에 Member 클래스를 넣으면 정석인 코드가 되는지 알고 싶습니다.

y2gcoder · Answer

안녕하세요. sjhyt0813님, 공식 서포터즈 y2gcoder입니다.

말씀하신 것과 같이 간단한 테스트 코드이고, 바로 전에 찾고자하는 엔티티가 그대로 저장되는 것이 보이는 등의 이유로 편의성을 위해 바로 get()으로 꺼내신 것으로 보입니다!

실제로는 복잡한 서비스 로직 간의 테스트 코드라면 말씀하신 것처럼 optional 값에 대한 선 검증을 하시는 편이 좋습니다 🙂

감사합니다.

인프런 AI 인턴 · Answer

안녕하세요, 인프런 AI 인턴입니다. sjhyt0813님.

스프링과 자바의 Optional<Member>에 관한 질문을 주셨네요. Optional 객체를 사용하면 NullPointerException을 방지하고, 메서드에서 값이 없음을 명시적으로 처리할 수 있습니다. 설정하신 코드에서 Optional<Member> result = repository.findByName("spring1") 패턴을 사용하는 이유는, 값이 정말 있는지를 확인하고 안전하게 꺼낼 수 있도록 하는 것입니다.

기존에 제시한 Member result = repository.findByName("spring1").get();의 경우, Optional이 비어있을 때 NoSuchElementException을 발생시킬 수 있습니다. 따라서 Optional의 isPresent() 메서드를 사용하여 값이 존재하는지 확인한 후에 get() 메서드를 사용하여 값을 꺼내는 것이 안전합니다. 아래의 코드는 이러한 안전한 접근 방식을 보여줍니다:

Optional<Member> optionalResult = repository.findByName("spring1");

if (optionalResult.isPresent()) {
    Member memberResult = optionalResult.get();
    Assertions.assertThat(memberResult).isEqualTo(member1);
}

Assertions.assertThat(result).isPresent()을 추가한다면, Optional 객체가 비어있지 않음을 확인합니다. 이 패턴은 프로덕션 코드에서도 널 체크를 확실하게 하고 싶을 때 유용하게 쓸 수 있습니다.

관련해서 유사한 질문과 답변들이 인프런에서 다루어진 바 있는데, 아래 링크에서 비슷한 논의를 참고할 수 있습니다:

이 내용들이 도움이 되었기를 바라며, 추가적인 궁금증이 있으시다면 언제든지 더 질문해 주세요. 곧 지식공유자께서도 도움을 주실 것입니다.

저는 질문자님의 이해를 돕고자 지식공유자님의 기존 답변을 참고하여 유사한 답변 링크를 추천해드리고 있습니다.
현재 베타 기능으로 답변이 만족스럽지 않을 수 있는 점 양해 부탁드립니다. 🙏
추가적으로 궁금한 점이 있으시면, 이어서 질문해 주세요. 곧 지식공유자께서 답변해 주실 것입니다.

인프런 커뮤니티 질문&답변

findByname파트에서 질문이 있습니다.