스프링 핵심원리 : 기본편 - 새로운 할인정책 개발과 적용, 그리고 문제점

"인프런의 스프링 핵심원리 - 기본편 강의를 듣고 작성한 글 입니다."

https://www.inflearn.com/course/%EC%8A%A4%ED%94%84%EB%A7%81-%ED%95%B5%EC%8B%AC-%EC%9B%90%EB%A6%AC-%EA%B8%B0%EB%B3%B8%ED%8E%B8/dashboard

스프링 핵심 원리 - 기본편 - 인프런 | 강의

 

새로운 할인정책의 개발

기존의 고정 금액 할인정책에 정률 할인 정책으로 요구사항이 변경 되었을 때, 지금까지 작성했던 코드들이 객체지향 원리가 잘 적용되어 구현체를 변경 시키기 편리한지, 그렇지 않은지 알아보자.

"계획을 따르기 보다 변화에 대응하기를" - 과연 지금의 예제는 변화에 대응하기 좋은 설계일까?

 

참고 - 애자일 소프트웨어 개발 선언 : https://agilemanifesto.org/iso/ko/manifesto.html

애자일 소프트웨어 개발 선언

 

정률 할인 정책을 위한 RateDiscountPolicy 를 추가해보자.

hello.core.discount 패키지에 다음과 같이 클래스를 만들고 코드를 작성한다.

- RateDiscountPolicy.java

public class RateDiscountPolicy implements DiscountPolicy{

    private int discountPercent = 10;

    @Override
    public int discount(Member member, int price) {
        // VIP 등급인 회원의 경우 가격 10% 할인
        if (member.getGrade() == Grade.VIP){
            return price * discountPercent / 100;
        }
        else{
            return 0;
        }
    }
}

위의 코드가 잘 동작하는지 확인하기 위해 테스트 코드를 작성해보자.

 

RateDiscountPolicy 클래스에서 ctrl + alt + t 를 누르면 테스트 코드를 작성할 클래스를 만들기 위한 옵션 창이 생성된다.(IntelliJ 기준)

옵션이 Junit5 로 되어있는 것을 확인 한 후 클래스 파일을 생성해주자.

그런 다음 아래와 같이 코드를 작성해보자.

- RateDiscountPolicyTest.java

 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
        Assertions.assertThat(discount).isEqualTo(1000);
    }
}

결과 : 테스트 통과

 

일단은 정률 할인 정책으로 만든 구현체를 통해 할인 기능이 잘 동작하는 것을 확인할 수 있다.

그런데 여기서 기능이 성공적으로 동작하는 것만 확인할 것이 아니라, 동작이 실패하는 경우 또한 테스트 해봐야 한다.

다음과 같이 할인 동작이 실패할 경우의 테스트 코드를 작성해보자.

- RateDiscountPolicyTest.java

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
        Assertions.assertThat(discount).isEqualTo(1000);
    }

    // 테스트를 할 때는 성공 뿐만이 아니라 실패 테스트도 반드시 만들어 봐야 한다.
    @Test
    @DisplayName("VIP 가 아니면 할인이 적용되지 않아야 한다.")
    void vip_x(){

        // given
        Member member = new Member(1L, "memberBASIC", Grade.BASIC);

        // when
        int discount = discountPolicy.discount(member, 10000);

        // then
        Assertions.assertThat(discount).isEqualTo(1000);
    }
}

위와 같이 코드를 작성한 후 vip_x 메소드를 실행시켜 보면 테스트가 다음과 같은 로그를 출력하면서 실패하는 것을 확인할 수 있다.

결과 :

org.opentest4j.AssertionFailedError: 
expected: 1000
 but was: 0
필요:1000
실제   :0

 

assertThat 을 통해 제공된 데이터가 1000이 나와야 하지만, 0 이 나왔으므로 테스트가 실패했다는 의미이다.

물론 0 이 나오는 것이 정상이다. 왜냐하면 회원 등급이 Basic 인 경우 할인 정책이 적용되서는 안되기 때문이다.

그럼 다음과 같이 코드를 수정하여 테스트를 통과 시켜보자.

- RateDiscountPolicyTest.java

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
        Assertions.assertThat(discount).isEqualTo(1000);
    }

    // 테스트를 할 때는 성공 뿐만이 아니라 실패 테스트도 반드시 만들어 봐야 한다.
    @Test
    @DisplayName("VIP 가 아니면 할인이 적용되지 않아야 한다.")
    void vip_x(){

        // given
        Member member = new Member(1L, "memberBASIC", Grade.BASIC);

        // when
        int discount = discountPolicy.discount(member, 10000);

        // then
        Assertions.assertThat(discount).isEqualTo(0);
        // 회원등급이 BASIC 인 경우 할인 정책이 적용되지 않기 때문에 discount 가 0 이 되는 것이 정상이다.
    }
}

위와 같이 코드를 수정 한 후 이번엔 테스트 클래스 전체를 실행시켜 보면 2개의 테스트가 모두 무사히 통과하는 것을 확인할 수 있다.

 

그렇다면 이번에 만든 정률 할인 정책을 실제 프로젝트에 적용시켰을 때 어떤일이 벌어지는지 한번 알아보자.

할인 정책의 변동사항을 적용하려면 먼저 할인 정책의 클라이언트인 OrderServiceImpl 클래스로 가서 할인 정책 객체를 생성하는 코드를 바꿔야 한다.

(이미 여기서 부터 OCP 원칙 위반이다. 구현 클래스에서 부터 코드를 바꿔야 한다.)

 

설계상의 문제점 발견

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

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

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

    -> 그렇게 보인다만 사실은 아니다.

- DIP 의 경우 주문 서비스 클라이언트 OrderServiceImpl 는 할인 정책에서 DiscountPolicy 인터페이스에 의존하면서 DIP 를 지킨것 같은데 어째서일까?

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

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

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

- OCP 의 경우 변경하지 않고 확장할 수 있다고 했다. 그런데 지금 코드는 기능을 확장해서 변경하면, 클라이언트 코드에 영향을 주게 된다.

따라서 OCP 를 위반한 것이다.

 

그렇다면 왜 클라이언트 코드를 변경해야 할까?

클래스 다이어그램 으로 의존관계를 분석해보자.

 

* 기대했던 의존관계

- 지금까지 단순히 DiscountPolicy 인터페이스만 의존한다고 생각했었다.

 

* 실제 의존관계

- 위의 다이어그램을 잘 보면 클라이언트인 OrderServiceImpl 이 DiscountPolicy 뿐만 아니라, FixDiscountPolicy 인 구체 클래스도 함께 의존하고 있다.

- 실제 코드를 보면 FixDiscountPolicy 클래스에 의존하고 있는것을 확인할 수 있다 즉, DIP 위반이다.

// 구현 클래스인 FixDiscountPolicy 객체 인스턴스를 생성하면서 해당 클래스에 의존하고 있다.
private final DiscountPolicy discountPolicy = new FixDiscountPolicy();

 

* 정책 변경

- FixDiscountPolicy 를 RateDiscountPolicy 로 변경하는 순간 OrderServiceImpl 의 소스 코드도 함께 변경해야 하게 된다.

- OCP 위반이다.

 

그렇다면 어떻게 이 문제를 해결할 수 있을까?

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

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

- DIP 위반인 현재 설계를 추상에만 의존하도록 변경시킨다. (인터페이스에만 의존하는 것으로 변경)

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

 

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

 

코드를 아래와 같이 변경하자.

- OrderServiceImpl.java

// private final DiscountPolicy discountPolicy = new RateDiscountPolicy();
// 위의 코드를 아래와 같이 바꾼다.
private DiscountPolicy discountPolicy;

위와 같이 코드를 짜면 인터페이스에만 의존하도록 코드를 변경할 수 있다.

그런데 지금 코드를 보면 구현체가 없는데 어떻게 코드를 실행할 수 있을까?

결국 이 코드를 실행시키면 NPE(Null Pointer Exception) 가 발생한다.

 

해결방안

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

(마치 스프링이 컴포넌트 스캔, 또는 직접 자바 코드로 스프링 컨테이너에 적재한 스프링 빈 객체를 @Autowired 어노테이션이 붙어있는 생성자를 통해 의존성을 주입 받는것 처럼 말이다.)