오늘이라도

[III. 객체 지향 원리 적용] 2. 새로운 할인 정책 적용과 문제점 본문

인프런/스프링 핵심 원리 - 기본편 (김영한)

[III. 객체 지향 원리 적용] 2. 새로운 할인 정책 적용과 문제점

upcake_ 2022. 12. 13. 18:04
반응형

강의 링크

 

※ 코드는 저작권 상 올릴 수 없다고 합니다 ㅜㅜ


1. 할인 정책 적용과정에서의 문제점

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

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

 - OCP, DIP 같은 객체지향 설계 원칙을 충실히 준수했다. → 그렇게 보이지만 사실은 아니다.

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

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

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

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

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

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

 

2. 왜 클라이언트 코드를 변경해야 할까?

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

기대했던 의존관계

 - 지금까지 단순히 DiscountPolicy에만 의존한다고 생각했다.

 

실제 의존관계

 - 잘보면 클라이언트인 OrderServiceImpl이 DiscountPolicy 인터페이스 뿐만 아니라,

   FixDiscountPolicy인 구체 클래스도 함께 의존하고 있다. 실제 코드를 보면 의존하고 있으므로 DIP 위반!

 

정책 변경

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

 

3. 어떻게 문제를 해결할 수 있을까?

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

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

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

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

 

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

 - 인터페이스에만 의존하도록 코드를 변경했다.

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

 - 실제 실행을 해보면 NPE(NullPointerException)가 발생한다.

 

 * 해결방안

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

반응형