인프런 영문 브랜드 로고
인프런 영문 브랜드 로고

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

hyunchal1098님의 프로필 이미지
hyunchal1098

작성한 질문수

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

회원 서비스 테스트

강의내용 17분쯤? 질문있어욤

작성

·

420

3

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

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

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

[질문 내용]

강의내용중 이해가 안가는 부분이 있습니다

회원서비스테스트 17분 쯤인데요

영상말씀하신것을 들어보면 memberRepository를 하나만 사용해서 같은 memberRepository로 데이터 CRUD를 하려하신다고 했는데

보면 @BeforeEach에서 new memberRepository를 하고 new memberService(memberRepository) 해주고 있는데요

이렇게 된다면 매번 새로운 memberRepository를 생성해서 매번 새로운 memberService를 생성하는 것 아닌가요??

이렇게 되면 더 좀 뭐랄까 좀더 비효율적이지 않나요??

그리고 코딩목적과도 약간 다르지않나요?

제가 코드를 잘못이해한것인가욤??

ㅠㅠ진짜 이해가 안가서 질문드립니다.

죄송합니다

답변 1

1

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

강의 수강 중 강의 관련하여 궁금한 사항은 언제든 질문 남겨주셔도 됩니다^^

아래와 같이 변경하거나

class MemberServiceTest {
    MemberService memberService = new MemberService(new MemoryMemberRepository());
    MemoryMemberRepository memberRepository;

    @BeforeEach
    public void beforeEach() {
        memberRepository = new MemoryMemberRepository();
    }

혹은 필드에서 생성까지 마칠수 있기도 합니다.

MemberService memberService = new MemberService(new MemoryMemberRepository());
MemoryMemberRepository memberRepository = new MemoryMemberRepository();

 

개인적으로 비효율적이라기 보다는강의처럼 작성하는 것이 각 테스트 실행과 종료에 인스턴스 생성과 할당 해제를 명확히 보여주는 장점이 더 도드라져 보이네요.

그리고 실제 테스트 시간을 비교해보시면 아시겠지만, 유의미한 시간차이는 발생하지 않습니다. 1~4ms 정도 차이가 나긴하지만 이정도로 비효율적이라고 보기는 힘들 것 같아요.

말씀하신 코딩목적은 어떠한 목적을 말씀하신 건지 잘 모르겠습니다.

조금 더 풀어서 설명해주시면 좋을 것 같아요.


.
감사합니다.

hyunchal1098님의 프로필 이미지
hyunchal1098

작성한 질문수

질문하기