🌱서비스 부분 로직 바꾸기 ( OCP, DIP 무시 버전)

1. 애자일 소프트웨어 개발 선언 (계획을 따르기보다 변화에 대응 순응하기를!)

개발을 하다보면, 애자일 방식을 사용할 수 밖에 없다.

사실 기획자도 자신이 뭘 하고 싶은지 정확하게 모를 때가 많다! 그때마다, 개발자는 만들어서 보여주고 기획자에게 역으로 방법을 재시한다던가, 아님 기획자가 중간에 자신이 깨닫지 못한 부분을 추가하고 싶을 때, 변화게 순응해서 또 그렇게 만들어줘야한다.

왜냐면... 기획이란 그런거고... 개발자란 그걸 실행시키는 역할이니까... 어쩔 수 없다.

2. 다시 바뀐 할인 정책 이야기로 돌아가자

고정 금액 할인에서
고정 퍼센트 할인으로 바꾼다. 

 

🥑 RateDiscountPolicy(%할인율)

package hello.core.discount;

import hello.core.member.Grade;
import hello.core.member.Member;

public class RateDiscountPolicy implements DiscountPolicy {

    private int discountPercent = 10;

    @Override
    public int discount(Member member, int price) {
       if(member.getGrade()== Grade.VIP){
           return price*discountPercent/100;
       }else{
           return 0;
        }
    }
}

🥑 OrderServiceImpl(%할인율)

package hello.core.order;

import hello.core.discount.DiscountPolicy;
import hello.core.discount.FixDiscountPolicy;
import hello.core.discount.RateDiscountPolicy;
import hello.core.member.Member;
import hello.core.member.MemberRepository;
import hello.core.member.MemoryMemberRepository;

public class OrderServiceImpl implements OrderService{

    private final MemberRepository memberRepository=new MemoryMemberRepository();
    //private final DiscountPolicy discountPolicy=new FixDiscountPolicy();
    private final DiscountPolicy discountPolicy=new RateDiscountPolicy();
    @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);
    }
}

🥑 TEST

package hello.core.discount;

import hello.core.member.Grade;
import hello.core.member.Member;
import org.assertj.core.api.Assertions;
import org.junit.jupiter.api.DisplayName;
import org.junit.jupiter.api.Test;

import static org.junit.jupiter.api.Assertions.*;

class RateDiscountPolicyTest {

    RateDiscountPolicy discountPolicy=new RateDiscountPolicy();

    @Test
    @DisplayName("VIP는 10% 할인이 적용되야 한다.")
    void vip_맞음(){
        //given
        Member member=new Member(1L,"memberVIP", Grade.VIP);
        //when
        int discount=discountPolicy.discount(member,9000);
        //then
        Assertions.assertThat(discount).isEqualTo(900);
    }

    @Test
    @DisplayName("VIP가 아니면 10% 할인이 적용되지말아야 한다.")
    void vip_아님(){
        //given
        Member member=new Member(1L,"memberBasic", Grade.BASIC);
        //when
        int discount=discountPolicy.discount(member,10000);
        //then
        Assertions.assertThat(discount).isEqualTo(0);
    }

}

3. 왜 해당 코드는 OCP, DIP 무시된 것 일까?

 

 DIP: 의존관계 역전 원칙 (Dependency inversion principle)

프로그래머는 추상화에 의존해야지 구체화에 의존하면 안된다.

---

OrderServiceImpl 경우 해당 클래스를 클라이언트라고 생각하면, 

추상(인터페이스) 의존의 경우: DiscountPolicy에 의존하고 있고
구체(구현클레스)의존의 경우: FixDiscountPolicy() -> RateDiscountPolicy()에 의존하고 있다.

---

• OCP: 개방-폐쇄 원칙 (Open/closed principle)

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

---

FixDiscountPolicy() -> RateDiscountPolicy()로 코드를 확장해서 변경하면
클라이언트 코드를 바꿔줘야한다!! 

//private final DiscountPolicy discountPolicy=new FixDiscountPolicy();
private final DiscountPolicy discountPolicy=new RateDiscountPolicy();

 

---

.

.

.

그럼 어떻게 하면..DIP 와 OCP를 지키는 프로그램을 설계할 수 있을까...? 다음글에서 이어 쓰도록 하겠다..!!