Spring core 할인정책개발 (김영한의 스프링)

 

 

 

계획을 따르기보다 변화에 대응하기를😉

 

 

 

 

유두리 있게 할인정책에 변화를 주기 위해서 DiscountPolicy를 바꿉니다

VIP는 전체금액에서 얼마든 1000원을 할인해주는 고정 할인을 했었지만

그것이 옳지않다고 생각한 클라이언트에 요구로

VIP는 얼마를 계산하면 10%라는 할인이 적용되게 해달라고 요구했습니다!

그럼 class에 코드를 추가하겠습니다

 

 

 

▶ RateDiscountPolicy implements DiscountPolicy

10%할인 정책으로 이부분만 변경

 private int discountPercent = 10;  

package hello.core.discount;
import hello.core.member.Grade;
import hello.core.member.Member;
public class RateDiscountPolicy implements DiscountPolicy {
 private int discountPercent = 10; //10% 할인
 @Override
 public int discount(Member member, int price) {
 if (member.getGrade() == Grade.VIP) {
 return price * discountPercent / 100;
 } else {
 return 0;
 }
 }
}

vip_o()  :  VIP 회원에게 10,000원 상품을 구매할 때, 할인이 올바르게 적용되는지를 테스트합니다.

즉, VIP 회원에게 10% 할인이 적용되어 1,000원의 할인이 이뤄져야 합니다.

vip_x()  :  VIP 회원이 아닌 일반 BASIC 회원에게 10,000원 상품을 구매할 때,

할인이 적용되지 않는지를 테스트합니다. 즉, 할인 금액이 0원이어야 합니다.

 

 

package hello.core.discount;
import hello.core.member.Grade;
import hello.core.member.Member;
import org.junit.jupiter.api.DisplayName;
import org.junit.jupiter.api.Test;
import static org.assertj.core.api.Assertions.*;
import static org.junit.jupiter.api.Assertions.*;
class RateDiscountPolicyTest {
 RateDiscountPolicy discountPolicy = new RateDiscountPolicy();
 @Test
 @DisplayName("VIP는 10% 할인이 적용되어야 한다.")
 void vip_o() {
 //given
 Member member = new Member(1L, "memberVIP", Grade.VIP);
 //when
 int discount = discountPolicy.discount(member, 10000);
 //then
 assertThat(discount).isEqualTo(1000);
 }
 @Test
 @DisplayName("VIP가 아니면 할인이 적용되지 않아야 한다.")
 void vip_x() {
 //given
 Member member = new Member(2L, "memberBASIC", Grade.BASIC);
 //when
 int discount = discountPolicy.discount(member, 10000);
 //then
 assertThat(discount).isEqualTo(0);
 }
}

public class OrderServiceImpl implements OrderService {
 
 // private final DiscountPolicy discountPolicy = new FixDiscountPolicy();
 private final DiscountPolicy discountPolicy = new RateDiscountPolicy();
}

 

 

※ 문제점 발견

 

우리는 역할과 구현을 충실하게 분리했다. → OK

다형성도 활용하고, 인터페이스와 구현 객체를 분리했다. → OK

OCP, DIP 같은 객체지향 설계 원칙을 충실히 준수했다

        → 그렇게 보이지만 사실은 아니다.

DIP: 주문서비스 클라이언트( OrderServiceImpl )는 DiscountPolicy 인터페이스에 의존하면서 DIP를 지킨 것 같은데?

        → 클래스 의존관계를 분석해 보자. 추상(인터페이스) 뿐만 아니라 구체(구현) 클래스에도 의존하고 있다.

                ⚫︎추상(인터페이스) 의존: DiscountPolicy

                ⚫︎구체(구현) 클래스: FixDiscountPolicy , RateDiscountPolicy

OCP: 변경하지 않고 확장할 수 있다고 했는데!

        → 지금 코드는 기능을 확장해서 변경하면, 클라이언트 코드에 영향을 준다! 따라서 OCP를 위반한다.

 

 

 

 

 

잘보면 클라이언트인 OrderServiceImpl 이 DiscountPolicy 인터페이스 뿐만 아니라
FixDiscountPolicy 인 구체 클래스도 함께 의존하고 있다. 실제 코드를 보면 의존하고 있다! DIP 위반

실제 의존관계

 

중요 ! : 그래서 FixDiscountPolicy 를 RateDiscountPolicy 로 변경하는 순간 OrderServiceImpl 의 소스
코드도 함께 변경해야 한다! OCP 위반

 

어떻게 문제를 해결할 수 있을가?

 

클라이언트 코드인 OrderServiceImpl 은 DiscountPolicy 의 인터페이스 뿐만 아니라 구체 클래스도 함 께 의존한다.

그래서 구체 클래스를 변경할 때 클라이언트 코드도 함께 변경해야 한다.

DIP 위반 추상에만 의존하도록 변경(인터페이스에만 의존)

DIP를 위반하지 않도록 인터페이스에만 의존하도록 의존관계를 변경하면 된다

 

 

인터페이스에만 의존하도록 설계를 변경하자!!

 

원하는 구조

 

 

인터페이스에만 의존하도록 코드 변경

 

public class OrderServiceImpl implements OrderService {
 //private final DiscountPolicy discountPolicy = new RateDiscountPolicy();
 private DiscountPolicy discountPolicy;
}

 

인터페이스에만 의존하도록 설계와 코드를 변경했다.
그런데 구현체가 없는데 어떻게 코드를 실행할 수 있을까?
실제 실행을 해보면 NPE(null pointer exception)가 발생한다.

 

이 문제를 해결하려면 누군가가 클라이언트인 OrderServiceImpl 에 DiscountPolicy 의 구현 객체를 대
신 생성하고 주입해주어야 한다

 

 

 

▶ AppConfig 등장
애플리케이션의 전체 동작 방식을 구성(config)하기 위해, 구현 객체를 생성하고, 연결하는 책임을 가지는 별도의
설정 클래스를 만들자.

 

package hello.core;
import hello.core.discount.FixDiscountPolicy;
import hello.core.member.MemberService;
import hello.core.member.MemberServiceImpl;
import hello.core.member.MemoryMemberRepository;
import hello.core.order.OrderService;
import hello.core.order.OrderServiceImpl;
public class AppConfig {
 public MemberService memberService() {
 return new MemberServiceImpl(new MemoryMemberRepository());
 }
 public OrderService orderService() {
 return new OrderServiceImpl(
 new MemoryMemberRepository(),
 new FixDiscountPolicy());
 }
}

 

▶ MemberServiceImpl - 생성자 주입

 

⚫︎ 설계 변경으로 MemberServiceImpl 은 MemoryMemberRepository 를 의존하지 않는다!
⚫︎ 단지 MemberRepository 인터페이스만 의존한다.
⚫︎ MemberServiceImpl 입장에서 생성자를 통해 어떤 구현 객체가 들어올지(주입될지)는 알 수 없다.
⚫︎ MemberServiceImpl 의 생성자를 통해서 어떤 구현 객체를 주입할지는 오직 외부( AppConfig )에서 결정된
다.
⚫︎ MemberServiceImpl 은 이제부터 의존관계에 대한 고민은 외부에 맡기고 실행에만 집중하면 된다.

 

package hello.core.member;
public class MemberServiceImpl implements MemberService {
 private final MemberRepository memberRepository;
 public MemberServiceImpl(MemberRepository memberRepository) {
 this.memberRepository = memberRepository;
 }
 public void join(Member member) {
 memberRepository.save(member);
 }
 public Member findMember(Long memberId) {
 return memberRepository.findById(memberId);
 }
}

 

 

 

 

 

 

▶ OrderServiceImpl - 생성자 주입

 

설계 변경으로 OrderServiceImpl 은 FixDiscountPolicy 를 의존하지 않는다!

⚫︎ 단지 DiscountPolicy 인터페이스만 의존한다.

⚫︎ OrderServiceImpl 입장에서 생성자를 통해 어떤 구현 객체가 들어올지(주입될지)는 알 수 없다.

⚫︎ OrderServiceImpl 의 생성자를 통해서 어떤 구현 객체을 주입할지는 오직 외부( AppConfig )에서 결정한다.

⚫︎ OrderServiceImpl 은 이제부터 실행에만 집중하면 된다.

⚫︎ OrderServiceImpl 에는 MemoryMemberRepository , FixDiscountPolicy 객체의 의존관계가 주 입된다.

 

package hello.core.order;
import hello.core.discount.DiscountPolicy;
import hello.core.member.Member;
import hello.core.member.MemberRepository;
public class OrderServiceImpl implements OrderService {
 private final MemberRepository memberRepository;
 private final DiscountPolicy discountPolicy;
 public OrderServiceImpl(MemberRepository memberRepository, DiscountPolicy
discountPolicy) {
 this.memberRepository = memberRepository;
 this.discountPolicy = discountPolicy;
 }
 @Override
 public Order createOrder(Long memberId, String itemName, int itemPrice) {
 Member member = memberRepository.findById(memberId);
 int discountPrice = discountPolicy.discount(member, itemPrice);
 return new Order(memberId, itemName, itemPrice, discountPrice);
 }
}

 

 

 

▶ 사용 클래스 - MemberApp실행

 

package hello.core;
import hello.core.member.Grade;
import hello.core.member.Member;
import hello.core.member.MemberService;
public class MemberApp {
 public static void main(String[] args) {
 AppConfig appConfig = new AppConfig();
 MemberService memberService = appConfig.memberService();
 Member member = new Member(1L, "memberA", Grade.VIP);
 memberService.join(member);
 Member findMember = memberService.findMember(1L);
 System.out.println("new member = " + member.getName());
 System.out.println("find Member = " + findMember.getName());
 }
}

 

 

▶ 사용 클래스 - OrderApp

 

package hello.core;
import hello.core.member.Grade;
import hello.core.member.Member;
import hello.core.member.MemberService;
import hello.core.order.Order;
import hello.core.order.OrderService;
public class OrderApp {
 public static void main(String[] args) {
 AppConfig appConfig = new AppConfig();
 MemberService memberService = appConfig.memberService();
 OrderService orderService = appConfig.orderService();
 long memberId = 1L;
 Member member = new Member(memberId, "memberA", Grade.VIP);
 memberService.join(member);
 Order order = orderService.createOrder(memberId, "itemA", 10000);
 System.out.println("order = " + order);
 }
}

 

 

 

▶ 테스트 코드 오류 수정

테스트 코드에서 @BeforeEach 는 각 테스트를 실행하기 전에 호출된다

 

class MemberServiceTest {
 MemberService memberService;
 @BeforeEach
 public void beforeEach() {
 AppConfig appConfig = new AppConfig();
 memberService = appConfig.memberService();
 }
}
 
class OrderServiceTest {
 MemberService memberService;
 OrderService orderService;
 @BeforeEach
 public void beforeEach() {
 AppConfig appConfig = new AppConfig();
 memberService = appConfig.memberService();
 orderService = appConfig.orderService();
 }
}

정리

 

AppConfig를 통해서 관심사를 확실하게 분리했다. 배역, 배우를 생각해보자.

AppConfig는 공연 기획자다. AppConfig는 구체 클래스를 선택한다.

배역에 맞는 담당 배우를 선택한다.

애플리케이션이 어떻게 동작해야 할 지 전체 구성을 책임진다.

이제 각 배우들은 담당 기능을 실행하는 책임만 지면 된다.

OrderServiceImpl 은 기능을 실행하는 책임만 지면 된다.

 

 

 

 

▶ AppConfig를 중복을 제거하고, 역할에 따른 구현이 보이도록 리팩터링 하자.

new MemoryMemberRepository() 이 부분이 중복 제거되었다. 

이제 MemoryMemberRepository 를 다른 구현체로 변경할 때 한 부분만 변경하면 된다.
AppConfig 를 보면 역할과 구현 클래스가 한눈에 들어온다. 

애플리케이션 전체 구성이 어떻게 되어있는지 빠르게 파악할 수 있다.

 

package hello.core;
import hello.core.discount.DiscountPolicy;
import hello.core.discount.FixDiscountPolicy;
import hello.core.member.MemberRepository;
import hello.core.member.MemberService;
import hello.core.member.MemberServiceImpl;
import hello.core.member.MemoryMemberRepository;
import hello.core.order.OrderService;
import hello.core.order.OrderServiceImpl;
public class AppConfig {
 public MemberService memberService() {
 return new MemberServiceImpl(memberRepository());
 }
 public OrderService orderService() {
 return new OrderServiceImpl(
memberRepository(),
discountPolicy());
 }
 public MemberRepository memberRepository() {
 return new MemoryMemberRepository();
 }
 public DiscountPolicy discountPolicy() {
 return new RateDiscountPolicy();
 }
}

AppConfig의 등장으로 애플리케이션이 크게 사용 영역과, 객체를 생성하고 

구성(Configuration)하는 영역으로 분리되었다.

 

 

 

좋은 객체 지향 설계의 5가지 원칙의 적용

 

 

SRP 단일 책임 원칙 한 클래스는 하나의 책임만 가져야 한다

SRP 단일 책임 원칙을 따르면서 관심사를 분리함
구현 객체를 생성하고 연결하는 책임은 AppConfig가 담당
클라이언트 객체는 실행하는 책임만 담당

 

DIP 의존관계 역전 원칙 프로그래머는 “추상화에 의존해야지, 구체화에 의존하면 안된다.”

의존성 주입은 이 원칙을 따르는 방법 중 하나다.

새로운 할인 정책을 개발하고, 적용하려고 하니 클라이언트 코드도 함께 변경해야 했다. 왜냐하면 기존 클라이언
트 코드( OrderServiceImpl )는 DIP를 지키며 DiscountPolicy 추상화 

인터페이스에 의존하는 것 같았지만, FixDiscountPolicy 구체화 구현 클래스에도 함께 의존했다.
클라이언트 코드가 DiscountPolicy 추상화 인터페이스에만 의존하도록 코드를 변경했다.
하지만 클라이언트 코드는 인터페이스만으로는 아무것도 실행할 수 없다.
AppConfig가 FixDiscountPolicy 객체 인스턴스를 클라이언트 코드 대신 생성해서 

클라이언트 코드에 의존관계를 주입했다. 이렇게해서 DIP 원칙을 따르면서 문제도 해결했다.

 

OCP 소프트웨어 요소는 확장에는 열려 있으나 변경에는 닫혀 있어야 한다

다형성 사용하고 클라이언트가 DIP를 지킴
애플리케이션을 사용 영역과 구성 영역으로 나눔
AppConfig가 의존관계를 FixDiscountPolicy RateDiscountPolicy 로 

변경해서 클라이언트 코드에 주입하므로 클라이언트 코드는 변경하지 않아도 됨
소프트웨어 요소를 새롭게 확장해도 사용 영역의 변경은 닫혀 있다!