[Spring] 스프링 입문 - 회원 서비스 개발

회원 서비스 개발

: 회원 리포지토리와 도메인을 활용하여 실제 비즈니스 로직을 작성

 

1. MemberService 클래스 생성

💡 service 패키지를 새로 만들어서, 그 안에 생성합니다

 

[단축키]

Command + Option + V : return 형식으로 변수 추출

 

 

2. 회원가입 메서드 생성

public class MemberService {
    private final MemberRepository memberRepository = new MemoryMemberRepository();

    /**
     * 회원 가입
     */
    public Long join(Member member) {
        // 같은 이름이 있는 중복 회원은 안되는 조건
        Optional<Member> result = memberRepository.findByName(member.getName());
        result.ifPresent(m -> { // return에 값이 있으면
            throw new IllegalStateException("이미 존재하는 회원입니다.");
        });

        memberRepository.save(member);
        return member.getId();
    }
}

Optional은 null을 판단하므로, if (null) 과 같은 조건문을 따로 추가할 필요가 없다는 점
Optional로 변수를 감싸주었기 때문에 isPresent 활용 가능, 바로 isPresent에서 중복 이름이 있는지 없는지를 판별한다

But! Optional을 바로 반환하는 건 좋지 않다

// 같은 이름이 있는 중복 회원은 안되는 조건
        memberRepository.findByName(member.getName())
                .ifPresent(m -> { // return에 값이 있으면
                  throw new IllegalStateException("이미 존재하는 회원입니다.");
              });

findByName의 결과가 Optional이므로, 바로 isPresent를 사용해주는 방법

코드가 훨씬 간결해진다

 

메서드 추출 단축키를 활용하자

controll + T → 메서드 추출,

command + option + M 으로 바로 추출할 수도 있다

 

3. 전체 회원 조회 메서드 생성

💡 Tip!
repository는 단순히 데이터를 저장하고 내보내는 듯한 느낌이라면, service는 비즈니스적인 로직 느낌
service 메서드를 네이밍 할 때, 비즈니스에 가깝게 하자

 

/**
     * 전체 회원 조회
     */

    public List<Member> findMembers() {
        return memberRepository.findAll();
    }

findAll의 반환 타임이 List<Member>이므로 반환만 해주면 된다

 

4. 개인 회원 조회 메서드 생성

public Optional<Member> findOne(Long memberId) {
        return memberRepository.findById(memberId);
    }

 

회원 서비스 테스트

[단축키] 테스트 생성

command + shift + T

 

💡 테스트는 한글로 적어도 괜찮다! 실제 코드에 포함되지 않기 때문에

given, when, then 문법

given : 어떤 것이 주어졌을 때

when : 이 코드를 실행 했을 때

then : 결과는 이것이 나와야 한다

 

회원가입 테스트 코드 작성

class MemberServiceTest {

    MemberService memberService = new MemberService();

    @Test
    void 회원가입() {
        // given
        Member member = new Member();
        member.setName("hello");

        // when
        Long saveId = memberService.join(member);

        // then
        Member findMember = memberService.findOne(saveId).get();
        assertThat(member.getName()).isEqualTo(findMember.getName());
    }

이 코드는 단순한 코드

예외 플로우를 고려하지 않은 반쪽짜리 테스트

 

중복 회원 검증 로직에서 예외 처리까지 테스트 해야 한다!

 

 

 

중복 회원이 있을 때 예외를 테스트하는 메서드 작성

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

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

        // when
        memberService.join(member1);
        try {
            memberService.join(member2);
            fail();
        } catch (IllegalStateException e) {
            assertThat(e.getMessage()).isEqualTo("이미 존재하는 회원입니다.");
        }

        // then
    }

member1과 2가 같은 이름이다.

이 때, 둘다 회원가입을 하면 에러가 발생해야 함.

try에서 member2를 가입시키면서 에러를 발생시키고, IllegalStateException에서 error를 캐치한다.

만약, catch로 넘어가지 않으면 fail() 실행

에러 메시지가 같은지 확인하여 테스트 성공과 실패 여부를 판단한다

 

 

try-catch 말고 좋은 문법은 없을까?

// when
        memberService.join(member1);
        assertThrows(IllegalStateException.class, () -> memberService.join(member2));        assertThrows(IllegalStateException.class, () -> memberService.join(member2));

assertThrows(IllegalStateException.class, () -> memberService.join(member2));

• () -> memberService.join(member2) 위의 로직을 실행했을 때, IllegalStateException 예외가 터져야 한다는 뜻

 

 

memberService.join(member1);
        IllegalStateException e = assertThrows(IllegalStateException.class, () -> memberService.join(member2));

        assertThat(e.getMessage()).isEqualTo("이미 존재하는 회원입니다.");

e라는 변수로 받아서, assertThat으로 에러 메시지 검증하기!

 

 

이제 테스트 순서에 영향받지 않도록, 테스트 메서드가 끝나면 clear 해주자!

MemoryMemberRepository memberRepository = new MemoryMemberRepository();

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

repository를 불러와서, clearStore 해주기!

 

 

MemberService의 memberRepository와, MemberServiceTest의 memberRepository를 하나로 통일하기

[단축키] 생성자 constructor command + N

private final MemberRepository memberRepository;

    public MemberService(MemberRepository memberRepository) {
        this.memberRepository = memberRepository;
    }

MemberService의 memberRepository를 직접 new로 생성하는 것이 아닌 외부에서 생성하도록 변경!

class MemberServiceTest {

    MemberService memberService;
    MemoryMemberRepository memberRepository;

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

실행하기 전에 BeforeEach를 먼저 실행!

각 테스트를 실행하기 전에 memberRepository를 하나로 통일하여, 같은 메모리의 memberRepository가 사용된다

memberService 입장에서, memberRepository를 외부에서 넣어준다 → dependency injection(di)

 

 

@참고자료

[인프런] 김영한 - 스프링 입문 (무료강의)