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

jy9922·2022년 8월 5일
0

Spring

목록 보기
5/34
post-thumbnail

✔ 회원 서비스

회원가입 메서드

  • 기본 회원가입 메서드
/* 회원가입 메서드 */
public Long join(Member member){
        // 같은 이름이 있는 중복 회원 제거
        Optional<Member> result = memberRepository.findByName(member.getName());
        // ifPresent => result가 값이 있으면, 동작하는 메서드
        result.ifPresent(m -> {
            throw new IllegalAccessError("이미 존재하는 회원입니다.");
        });
        memberRepository.save(member);
        return member.getId();
    }
  • Optional 감싸기 제거
// Optional로 바로 반환하는 것은 좋지 않다.
// 아래와 같은 코드로 변경할 수 있다.
public Long join(Member member){
        // 같은 이름이 있는 중복 회원 제거
        memberRepository.findByName(member.getName())
            .ifPresent(m -> {
                throw new IllegalAccessError("이미 존재하는 회원입니다.");
        });
        memberRepository.save(member);
        return member.getId();
    }
  • 메서드 추출 ( ctrl + alt + m )
// 메서드 추출하기
public Long join(Member member){
        validateDuplicatedMember(member); // 중복회원 검증
        memberRepository.save(member);
        return member.getId();
    }
    // ctrl + alt + m 으로 메서드 밖으로 추출하기
    private void validateDuplicatedMember(Member member) {
        memberRepository.findByName(member.getName())
            .ifPresent(m -> {
                throw new IllegalAccessError("이미 존재하는 회원입니다.");
        });
    }

전체 회원 조회

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

    public Optional<Member> findDre(Long memberID){
        return memberRepository.findById(memberID);
    }

✔ 회원 서비스 테스트

  • 클래스 선택하고 ctrl + shift + t로 자동 테스트 케이스 만들기
  • 테스트는 한글로 바꿔도 된다 (한글로도 많이 적음)

    테스트 케이스
    아래와 같은 패턴으로 작성하는 것읗 권장해요!

    // given : 무엇인가 주었는데
    // when : 이것을 실행했을 때
    // then : 아래의 결과가 나와야 한다

회원가입 테스트

@Test
    void 회원가입() {
        // given : 무엇인가 주었는데
        Member member = new Member();
        member.setName("hello");
        // when : 이것을 실행했을 때
        Long saveId = memberService.join(member);
        // then : 아래의 결과가 나와야 한다
        Member findMember = memberService.findOne(saveId).get();
        Assertions.assertThat(member.getName()).isEqualTo(findMember.getName());
    }

✅ 중복회원예외( ) 1

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

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


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

테스트 통과

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

테스트 오류

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

✅ 중복회원예외( ) 2 - try-catch 제거

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

// 메시지 내용으로 확인
memberService.join(member1);
        IllegalAccessError e = assertThrows(IllegalAccessError.class, () -> memberService.join(member2));
        Assertions.assertThat(e.getMessage()).isEqualTo("이미 존재하는 회원입니ㄴ다.");

✅ clear 메서드 추가

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

여기서 잠깐! ✋

DI, Dependency Injection (의존관계 주입)

코드를 보면 서로 다른 리포지토리를 테스트하고 있다.
이는 공유 데이터(store)가 static이 아니라면 문제가 될 여지가 있다.

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

따라서, 같은 인스턴스를 사용할 수 있게 코드를 수정해주는 것이 좋다.

private final MemberRepository memberRepository = new MemberRepository();

MemberService에서 위 코드를 다음과 같이 바꾸면 된다.

private final MemberRepository memberRepository;

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

MemberService 생성자를 생성해 MemberService를 외부에서 넣을 수 있도록 바꿔준다.
다시, MemberServiceTest로 와서 코드를 다음과 같이 수정한다.

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

@BeforeEach로 인해 테스트 실행시 생성해 주는데,
먼저 memoryMemberRepository를 만들고 해당 객체를 memberService에 넣어준다.
이로인해, 같은 리포지토리를 사용할 수 있게 된다.

memberService 입장에서 직접 new 하지 않고, 외부에서 memberRepository를 넣어준다.
이를 Dependency Injection (의존관계 주입) 이라고 한다.

✔ 전체 코드

리포지토리 구현 코드

package hello.hellospring.repository;

import hello.hellospring.domain.Member;

import java.util.*;

public class MemoryMemberRepository implements MemberRepository{

    private static Map<Long, Member> store = new HashMap<>();
    private static long sequence = 8L; // 0,1,2 와 같은 key값을 생성해줌

    @Override
    public Member save(Member member) {
        member.setId(++sequence);
        store.put(member.getId(), member);
        return member;
    }

    @Override
    public Optional<Member> findById(Long id) {
        return Optional.ofNullable(store.get(id));
        // 결과가 null인 경우, Optional.ofNullable로 감싸서 반환
    }

    @Override
    public Optional<Member> findByName(String name) {
        return store.values().stream()
                .filter(member -> member.getName().equals(name)) // 같은 경우 filter
                .findAny(); // 돌리다가 있으면 찾은 것을 반환
                            // 만약 없으면 Optional에 null을 포함해서 반환
    }

    @Override
    public List<Member> findAll() {
        return new ArrayList<>(store.values());
    }
}

회원 서비스 코드

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){
        validateDuplicatedMember(member); // 중복회원 검증
        memberRepository.save(member);
        return member.getId();
    }

    // ctrl + alt + m 으로 메서드 밖으로 추출하기
    private void validateDuplicatedMember(Member member) {
        memberRepository.findByName(member.getName())
            .ifPresent(m -> {
                throw new IllegalAccessError("이미 존재하는 회원입니다.");
        });
    }

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

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

회원 서비스 테스트 코드

package hello.hellospring.service;

import hello.hellospring.domain.Member;
import hello.hellospring.repository.MemberRepository;
import hello.hellospring.repository.MemoryMemberRepository;
import org.assertj.core.api.Assertions;
import org.junit.jupiter.api.AfterEach;
import org.junit.jupiter.api.BeforeEach;
import org.junit.jupiter.api.Test;

import static org.junit.jupiter.api.Assertions.assertThrows;
import static org.junit.jupiter.api.Assertions.fail;


class MemberServiceTest {
    // 서로 두개의 다른 instance가 나오게 된다. ( 이 방법은 비추 )
    // 같은 리포지토리를 테스트 하는 것이 좋아요!

    //
    MemberService memberService;
    MemoryMemberRepository memberRepository;

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

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

    @Test
    void 회원가입() {
        // given : 무엇인가 주었는데
        Member member = new Member();
        member.setName("hello");
        // when : 이것을 실행했을 때
        Long saveId = memberService.join(member);
        // then : 아래의 결과가 나와야 한다
        Member findMember = memberService.findOne(saveId).get();
        Assertions.assertThat(member.getName()).isEqualTo(findMember.getName());
    }

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

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

        memberService.join(member1);
        IllegalAccessError e = assertThrows(IllegalAccessError.class, () -> memberService.join(member2));
        Assertions.assertThat(e.getMessage()).isEqualTo("이미 존재하는 회원입니다.");

    }
    @Test
    void findMember() {
    }

    @Test
    void findDre() {
    }
}

0개의 댓글