[Spring] 입문_백엔드 개발(3)

1. 회원 서비스 개발

package hello.hellospring.service;

import hello.hellospring.domain.Member;
import hello.hellospring.repository.MemberRepository;

import java.util.List;
import java.util.Optional;

public class MemberService {

    private final MemberRepository memberRepository;

	// 외부에서 넣어줌
    public MemberService(MemberRepository memberRepository) {
        this.memberRepository = memberRepository;
    }

    // 회원가입
    public Long join(Member member) {
        
        // 이름 중복 여부 확인
        validateDuplicateMember(member);
        memberRepository.save(member);  // 이름중복여부에서 통과하면 저장

        return member.getId();
    }

    private void validateDuplicateMember(Member member) {
        memberRepository.findByName(member.getName())
                 .ifPresent(m -> {
                     throw new IllegalStateException("이미 존재하는 회원입니다!");
                 });
    }

    // 전체회원 조회
    public List<Member> findMembers() {
        return memberRepository.findAll();
    }

	// 특정 회원 조회
    public Optional<Member> findOne(Long memberId) {
        return memberRepository.findById(memberId);
    }
}

• 레포지토리 이슈 -> 밑에 작성

• 이름 중복 회원 확인방법 -> 아래 코드를 validateDuplicateMember(Member member)라는 하나의 함수로 생성
  (Shift+Ctrl+Alt+T(리팩토링 관련) -> extend method)

Optional<Member> result = memberRepository.findByName(member.getName());
// result.get();  // 이런식으로 바로 꺼내도 괜찮은데 이왕이면 사용X
result.ifPresent(m -> {
    throw new IllegalStateException("이미 존재하는 회원입니다!");
});  // Optional로 감쌌기 때문에 예외처리 가능!

---

2. 회원 서비스 테스트

[ 회원가입 ]

@Test
void 회원가입() {
    // given
    Member member = new Member();
    member.setName("hello");  // 만약 hello가 아닌 spring이면 회원가입에서 터진다 -> clear해줘야함

    // when
    Long saveId = memberService.join(member);  // join return값이 id

    // then
    // repository에 있는 값이 맞아?를 찾기 위해서는 repository 꺼내야함
    Member findMember = memberService.findOne(saveId).get();  // 값 바로가지고옴
    Assertions.assertThat(member.getName()).isEqualTo(findMember.getName());  // hello와 동일한 이름을 가지고있는지 확인
}

• 성공
  

[ 중복회원예약 ]

@Test
public void 중복회원예약() {
    // given
    Member member1 = new Member();
    member1.setName("spring");

    Member member2 = new Member();
    member2.setName("spring");

    // when
    memberService.join(member1);  // 성공

	// then
    // NullPointerException이면 test실패
    IllegalStateException e = assertThrows(IllegalStateException.class, () -> memberService.join(member2));
    assertThat(e.getMessage()).isEqualTo("이미 존재하는 회원입니다!");  // join에서 에러날때 
}

• 좋은 예외처리는 아니지만 이렇게도 가능

// 중복되면 validateDuplicateMember에 걸려서 예외가 터짐!
try {
	memberService.join(member2);
	fail();
} catch (IllegalStateException e) {
	Assertions.assertThat(e.getMessage()).isEqualTo("이미 존재하는 회원입니다!");
}

• 중복회원 가입한 상황이라면
  
• 중복회원 가입하지 않은 상황이라면
  

[ 레포지토리 이슈 및 기타 ]

// MemberService
private final MemberRepository memberRepository;

// 외부에서 넣어줌
public MemberService(MemberRepository memberRepository) {
    this.memberRepository = memberRepository;
}

// MemberServiceTest
class MemberServiceTest {

    MemberService memberService;
    MemoryMemberRepository memberRepository;

    @BeforeEach
    public void beforeEach() {
        // 동작 전에 외부에서 넣어주는거로 바꾸기
        memberRepository = new MemoryMemberRepository();
        memberService = new MemberService(memberRepository);
    }

    @AfterEach
    public void afterEach() {
        memberRepository.clearStore();
    }
}

• 레포지토리
  • 기존방법 : 처음에는 MemberService에 private final MemberRepository memberRepository = new MemoryMemberRepository(); 이렇게 작성하여 회원 서비스가 메모리 회원 리포지토리를 직접 생성하게 했다.
  • 문제점 : 지속적으로 다른 레포지토리 객체가 생성이 되면 각자 다른 인스턴스기 때문에 데이터가 적재되는것이 아니라서 내용이 달라질 수 있음 -> 다른 레포지토리 데이터로 test하는 것임
  • 즉, memberService에서의 memoryMemberRepository랑 memberServiceTest의 memoryMemberRepository가 서로 다른 인스턴스(레포지토리)임
  • 지금은 MemberRepository static이라 문제가 없지만, 만약 static이 없으면 문제가 생김
  • 해결책 : new로 다른레포지토리 생성이 되어 -> memberService의 final MemberRepository 로 두고 constructor만들기(동작 전에 외부에서 넣어주는거로 바꾸기) -> BeforeEach 함수 생성
  • 결론 : 회원 서비스 코드를 DI 가능하게 변경한다.
• AfterEach
  • 전체 실행을 한다면, test는 순서대로 test되지않기 때문에 테스트 하나 끝나고 나면 clear해줘야함