inflearn logo
강의

강의

N
챌린지

챌린지

멘토링

멘토링

N
클립

클립

로드맵

로드맵

지식공유

스프링 입문 - 코드로 배우는 스프링 부트, 웹 MVC, DB 접근 기술

회원 리포지토리 테스트 케이스 작성

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

151

sjhyt0813

작성한 질문수 1

0

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

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

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

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

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

image.png

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

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

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

 

java spring mvc spring-boot

답변 2

0

y2gcoder

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

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

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

감사합니다.

0

인프런 AI 인턴

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

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

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

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

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

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

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

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

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

..

0

76

2

Unused property.....

0

102

2

project JDK is misconfigured

0

132

2

외부 API의 ID 타입(String/UUID)과 내부 도메인의 ID 타입(Long)이 불일치할 때의 설계 정석

0

83

2

단위/통합 테스트 버전충돌 문제

0

91

2

❗️springboot 4.0.2 버전 aspectj dependency 설정❗️

0

230

1

왜 컨트롤러는 변한게 없는데 새로 만든 html 파일이 뜨나요?

0

104

2

윈도우 build test오류 질문

1

103

2

테스트 관련 공부에 대한 조언을 얻고 싶습니다

0

101

2

테스트 실행 시 에러 질문

0

292

1

name을 통한 비교와 객체를 통한 비교

0

77

1

빌드 후 libs 없음

1

129

1

윈도우 gradlew.bat 에러

0

167

1

@PostMapping("/members/new")가 동작하지 않습니다

0

90

1

java static class와 kotlin class

0

79

1

스프링 DB연결

0

127

1

소요 시간

0

87

2

ddl.sql에 빨간 밑줄

1

102

2

welcome page 에러

0

183

3

잘 모르겠습니다.

0

164

2

fail을 똑같이 쳤는데 오류가 발생해요

0

157

2

index.html Welcome page

0

125

1

프로젝트 gradle-groovy ?

0

369

1

테스트코드 메서드명 한글

0

202

2