#3-3. [스프링 핵심 원리-기본편] - 리펙토링

[전체 구조 다이어그램]

[AppConfig]

(중복을 제거하고, 한눈에 각각의 역할이 보일 수 있도록 분리하자.)

package hello.core.order;

import hello.core.discount.DiscountPolicy;
import hello.core.discount.FixDiscountPolicy;
import hello.core.member.MemberService;
import hello.core.member.MemberServiceImpl;
import hello.core.member.MemoryMemberRepository;

public class AppConfig {

    public MemberService memberService() {
        return new MemberServiceImpl(memberRepository());
    }

    private MemoryMemberRepository memberRepository() {
        return new MemoryMemberRepository();
    }

    public OrderService orderService() {
        return new OrderServiceImpl(memberRepository(), discountPolicy());
    }

    public DiscountPolicy discountPolicy() {
        return new FixDiscountPolicy();
    }
}

 

[새로운 구조와 할인 정책 적용]

(FixDiscountPolicy -> RateDiscountPolicy)

기존에 코드에서는 할인 정책이 변경되면 OrderServiceImpl코드도 변경되어야 했다. 그러나 지금은!!

역할이 분리가 되었기에, DiscountPolicy를 생성하는 AppConfig 코드만 변경해주면 된다!

// 할인 정책을 변경해야 한다면??
// AppConfig 중에서...
public DiscountPolicy discountPolicy() {
    // return new FixDiscountPolicy();
    // 변경
    return new RateDiscountPolicy();
    
}

 

[실제 SOLID원칙이 어떻게 적용되었는가]

SRP 단일 책임 원칙

한 클래스는 하나의 책임만 가져야 한다.

 

클라이언트 객체는 직접 구현 객체를 생성하고, 연결하고, 실행하는 다양한 책임을 가지고 있음

- SRP 단일 책임 원칙을 따르면서 관심사를 분리함

- 구현 객체를 생성하고 연결하는 책임은 AppConfig가 담당

- 클라이언트 객체는 실행하는 책임만 담당

 

DIP 의존관계 역전 원칙

프로그래머는 “추상화에 의존해야지, 구체화에 의존하면 안된다.” 의존성 주입은 이 원칙을 따르는 방법 중 하나다.

 

- 새로운 할인 정책을 개발하고, 적용하려고 하니 클라이언트 코드도 함께 변경해야 했다.왜냐하면 기존 클라이언트 코드( OrderServiceImpl )는 DIP를 지키며 DiscountPolicy 추상화 인터페이스에 의존하는 것 같았지만, FixDiscountPolicy 구체화 구현 클래스에도 함께 의존했다.

- 클라이언트 코드가 DiscountPolicy 추상화 인터페이스에만 의존하도록 코드를 변경했다.

- 하지만 클라이언트 코드는 인터페이스만으로는 아무것도 실행할 수 없다.

- AppConfig가 FixDiscountPolicy 객체 인스턴스를 클라이언트 코드 대신 생성해서 클라이언트 코드에 의존관계를 주입했다. 이렇게해서 DIP 원칙을 따르면서 문제도 해결했다.

 

OCP 개방 폐쇄 원칙

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

 

- 다형성 사용하고 클라이언트가 DIP를 지킴

- 애플리케이션을 사용 영역과 구성 영역으로 나눔

- AppConfig가 의존관계를 FixDiscountPolicy RateDiscountPolicy 로 변경해서 클라이언트 코드에 주입하므로 클라이언트 코드는 변경하지 않아도 됨

- 소프트웨어 요소를 새롭게 확장해도 사용 영역의 변경은 닫혀 있다!