학원에서는
member, board 별로 패키지를 생성했는데, 여기선 이렇게 용도별로 패키지를 생성하는 부분이 흥미롭다.
뭐가 더 나은지는 아직 모르겠다.
개발자 만의 스타일이 다른건지, 아니면 이런 방식이 좀더.. 요즘스러운 방식인지.
repository -- 좀더 개발자스러운 네이밍
service -- 비즈니스 로직을 구현하기 때문에 비즈니스 스러운 네이밍.
테스트케이스를 작성할 때, 번거롭게 테스트 클래스를 만드는 것 보다
클래스 이름 선택 + Command T 누르면 자동으로 생성됨.
package hello.hellospring.service;
import hello.hellospring.domain.Member;
import hello.hellospring.repository.MemberRepository;
import hello.hellospring.repository.MemoryMemberRepository;
import java.util.List;
import java.util.Optional;
public class MemberService {
// Service 경우, 비즈니스 로직을 처리하기 떄문에 네이밍할 때 비즈니스 용어를 써주는게 좋음
private final MemberRepository memberRepository;
// 의존성 주입
// 매번 new 로 생성하면 여러 DB를 생성하게 됨.. 메모리 낭비.
// 이렇게 하면 같은 DB를 쓸 수 있음.
public MemberService(MemberRepository memberRepository) {
this.memberRepository = memberRepository;
}
/**
* 회원가입
*/
public Long join(Member member) {
// 같은 이름이 있는 중복 회원 안됨
// commane option v -- 리턴 자동
// ctrl T -- Extract Method(Command option M) 메서드 추출
// 로직이 쭉 나올 경우 메서드로 뽑는게 좋음
validateDuplicateMember(member); // 중복 회원 검증
memberRepository.save(member);
return member.getId();
}
private void validateDuplicateMember(Member member) {
memberRepository.findByName(member.getName())
.ifPresent(m -> { // null 이 아닌 값이 있을 때 동작
throw new IllegalStateException("이미 존재하는 회원입니다.");
});
}
/**
* 전체 회원 조회
*/
public List<Member> findMembers() {
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.MemoryMemberRepository;
import org.junit.jupiter.api.AfterEach;
import org.junit.jupiter.api.Assertions;
import org.junit.jupiter.api.BeforeEach;
import org.junit.jupiter.api.Test;
import static org.assertj.core.api.Assertions.assertThat;
import static org.junit.jupiter.api.Assertions.*;
class MemberServiceTest {
// ctrl R -- 이전에 실행한 거를 그대로 실행해줌.
MemberService memberService;
MemoryMemberRepository memberRepository;
@BeforeEach
public void beforeEach() {
memberRepository = new MemoryMemberRepository();
memberService = new MemberService(memberRepository);
}
@AfterEach
public void afterEach() {
memberRepository.clearStore();
}
@Test
void join() {
//given -- 주어진 상황 (데이터)
Member member = new Member();
member.setName("spring");
//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);
IllegalStateException e = assertThrows(IllegalStateException.class, () -> memberService.join(member2));
assertThat(e.getMessage()).isEqualTo("이미 존재하는 회원입니다.");
// try{
// memberService.join(member2);
// fail();
// } catch (IllegalStateException e) {
// assertThat(e.getMessage()).isEqualTo("이미 존재하는 회원입니다.");
// }
//then
}
@Test
void findMembers() {
}
@Test
void findOne() {
}
}