[Spring] SOLID 원칙을 잘 지키고 있는가

문제상황

할인 정책은 두 가지가 있다.

1. FixDiscountPolicy() : 고정 할인

2. RateDiscountPolicy() : 비율 할인

 

이때 우리는 할인 정책을 변경하기 위해 FixDiscountPolicy()에서 RateDiscountPolicy()으로 바꾸려고 한다.

 

여기서 문제 발생!

 

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

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

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

 

 

DIP: 주문서비스 클라이언트(OrderServiceImpl )는  DiscountPolicy   지킨 것 같은데?

클래스 의존관계를 분석해 보자.

추상(인터페이스) 뿐만 아니라 인터페이스에 의존하면서 DIP를 구체(구현) 클래스에도 의존하고 있다.

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

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

OCP: 변경하지 않고 확장할 수 있다고 했는데 지금 코드는 기능을 확장해서 변경하면 클라이언트 코드에 영향을 준다!

따라서 OCP를 위반한다.

 

 

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

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

 

우리가 기대했던 의존관계

 

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

 

실제 의존관계

 

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

 

 

 

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

 

 

 

 

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

클라이언트 코드인 께 의존한다.

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

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

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

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

 

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

 

 

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

 

 

인터페이스에만 의존하도록 설계와 코드를 변경했다. 그런데 구현체가 없는데 어떻게 코드를 실행할 수 있을까?

 

 

실제 실행을 해보면 NPE(null pointer exception)가 발생한다.

 

 

해결방안 

이 문제를 해결하러면 누군가가 클라이언트인 OrderServiceImpl에  DiscountPolicy의 구현 객체를 대신 생성하고

주입해주어야 한다.

 

구체적인 해결방안은 다음 관심사의 분리에서 다루겠다.