Spring / AppConfig를 이용한 기존의 OCP, DIP 문제 해결

public class OrderServiceImpl implements OrderService {
  //    private final DiscountPolicy discountPolicy = new FixDiscountPolicy();
      private final DiscountPolicy discountPolicy = new RateDiscountPolicy();
  }

기존의 FixDiscountPolicy를 RateDiscountPolicy로 변경하고자 할 때, 다음과 같은 문제점이 있었다.

 

1. DIP

OrderServiceImp는 DiscountPolicy 인터페이스에 의존하면서, 구현체인 FixDiscountPolicy에도 의존하고 있다.

DIP를 지키려면 오직 인터페이스에만 의존하도록 해야 한다.

 

2. OCP 

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

 

 

Expected Relationship

이렇게 보일 수 있지만 사실 구현체에도 의존을 하고 있기 때문에 

 

실제 의존관계는 다음과 같다.

 

그래서 FixDiscountPolicy를 RateDiscountPolicy로 변경하는 순간

OrderServiceImpl의 소스코드도 함께 변경해야 한다.  ▶︎ OCP 위반

 

 

문제점을 다시 한 번 짚자면 OrderServiceImp이 DiscountPolicy 인터페이스 뿐만 아니라 구체 클래스에도 함께 의존한다는 점.

DIP를 지키기 위해서는 오직 인터페이스에만 의존할 수 있도록 변경해야 한다.

 

인터페이스에만 의존하는 위의 Expected Relationship과 같은 그림으로 변경하기 위해 설계를 변경하자

  public class OrderServiceImpl implements OrderService {
      //private final DiscountPolicy discountPolicy = new RateDiscountPolicy();
      private DiscountPolicy discountPolicy;
}

이렇게 하면 우선 인터페이스에만 의존하도록 할 수 있다.

하지만 구현체가 없는데 어떻게 실행을 할 수 있을까???? 

단순하게 이렇게 실행하면 NPE(Null Pointer Exception) 발생함

 

 

Solution

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

 

---

관심사 분리하기

• 애플리케이션 = 하나의 공연
• 각각의 인터페이스 = 배우 역할
• 구현체 = 배우

라고 했을 때, 

이전 코드는 마치 배우 한명이 다른 배역의 배우를 섭외(초청)하는 것과 같다.

하지만 배우가 할 일은 배역을 수행하는 것에만 집중해야 하지 다른 배우를 섭외하는 것은 공연 기획자가 할 일이다.

 

관심사를 분리하자.

어떻게? 구현체(배우)는 연기에만 집중하고, 다른 배우를 초청한다던지, 그 외의 잡다한 일은 공연 기획자에게 맡기도록

구현체(배우)는 다른 배역에 어떠한 배우가 섭외되더라도 본인 역할에 충실할 수 있어야 한다.

( 할인정책이 Rate이던 Fix던 관계없이 잘 실행되어야 한다는 뜻 )

 

 

---

 AppConfig 

구현체를 생성하고 연결하는 책임을 가지는 별도의 설정 클래스를 만들자.

package hello.core;

import hello.core.discount.DiscountPolicy;
import hello.core.discount.FixDiscountPolicy;
import hello.core.member.MemberService;
import hello.core.member.MemberServiceImp;
import hello.core.order.OrderService;
import hello.core.order.OrderServiceImp;

public class AppConfig {

    public MemberService memberService()
    {
        return new MemberServiceImp(new MemoryMemberRepository());
    }

    public OrderService orderService()
    {
        return new OrderServiceImp(new MemoryMemberRepository(), new FixDiscountPolicy());
    }
}

AppConfig는 애플리케이션의 실제 동작에 필요한 구현 객체를 생성, 관리한다.

• MemberServiceImp
• MemoryMemberRepository
• OrderServiceImp
• FIxDiscountPolicy

 

AppConfig는 생성한 객체 인스턴스의 참조를 생성자를 통해서 주입(연결)해준다.

• MemberServiceImp -> MemoryMemberRepository
• OrderServiceImp -> MemoryMemberRepository, FixDiscountPolicy

 

 

MemberServiceImp

package hello.core.member;

import hello.core.MemberRepository;

public class MemberServiceImp implements MemberService{
    private final MemberRepository memberRepository;

    public MemberServiceImp(MemberRepository memberRepository) {
        this.memberRepository = memberRepository;
    }

    @Override
    public void join(Member member) {
        memberRepository.save(member);
    }

    @Override
    public Member findMember(Long memberId) {
        return memberRepository.findById(memberId);
    }
}

• 설계 변경으로 MemberServiceImpl 은 MemoryMemberRepository 를 의존하지 않는다! 단지 MemberRepository 인터페이스만 의존한다.
• MemberServiceImp 입장에서 생성자를 통해 어떤 구현 객체가 들어올지(주입될지)는 알 수 없다.
• MemberServiceImp 의 생성자를 통해서 어떤 구현 객체를 주입할지는 오직 외부( AppConfig )에서 결정된다.
• MemberServiceImp 은 이제부터 의존관계에 대한 고민은 외부에 맡기고 실행에만 집중하면 된다.

 

 

OrderService

package hello.core.order;

import hello.core.MemberRepository;
import hello.core.discount.DiscountPolicy;
import hello.core.member.Member;

public class OrderServiceImp implements OrderService{

    private final MemberRepository memberRepository;
    private final DiscountPolicy dp;

    public OrderServiceImp(MemberRepository memberRepository, DiscountPolicy dp) {
        this.memberRepository = memberRepository;
        this.dp = dp;
    }

    @Override
    public Order createOrder(Long memberId, String itemName, int itemPrice) {
        Member member = memberRepository.findById(memberId);
        int discountPrice = dp.discount(member,itemPrice);

        return new Order(memberId, itemName, itemPrice, discountPrice);
    }
}

• 설계 변경으로 OrderServiceImpl 은 FixDiscountPolicy 를 의존하지 않는다. 단지 DiscountPolicy 인터페이스만 의존한다.
• OrderServiceImpl 입장에서 생성자를 통해 어떤 구현 객체가 들어올지(주입될지)는 알 수 없다.
• OrderServiceImpl 의 생성자를 통해서 어떤 구현 객체을 주입할지는 오직 외부( AppConfig )에서 결정한다.
• OrderServiceImpl 은 이제부터 실행에만 집중하면 된다.

 

---

 

 

---

AppConfig에서 MemberServiceImp(구현체), MemoryMemberRepository(구현체) 생성해서 

생성한 객체 인스턴스의 참조를 생성자를 통해서 주입(연결)해준다.

DIP 완성 : MemberServiceImp은  MemberRepository인 추상에만 의존하면 된다.

• AppConfig 객체는 MemoryMemberRepository 객체를 생성하고 그 참조값을 memberServiceImpl 을 생성하면서 생성자로 전달한다.
• 클라이언트인 MemberServiceImp 입장에서 보면 의존관계를 마치 외부에서 주입해주는 것 같다고 해서 DI(Dependency Injection) 우리말로 의존관계 주입 또는 의존성 주입이라 한다.

---

TEST

MemberApp

package hello.core.member;

import hello.core.AppConfig;

public class MemberApp {
    public static void main(String[] args) {
        AppConfig appConfig = new AppConfig();
        MemberService memberService = appConfig.memberService();

        Member member = new Member(1L, "memberA", Grade.VIP);
        memberService.join(member);

        Member findMember = memberService.findMember(1L);
        System.out.println("findMember = " + member.getName());
        System.out.println("findMember = " + findMember.getName());
    }
}

 

OrderApp

package hello.core.order;

import hello.core.AppConfig;
import hello.core.member.Grade;
import hello.core.member.Member;
import hello.core.member.MemberService;
import hello.core.member.MemberServiceImp;

public class OrderApp {
    public static void main(String[] args) {
        AppConfig appConfig = new AppConfig();

        MemberService memberService = appConfig.memberService();
        OrderService orderService = appConfig.orderService();

        Member memberA = new Member(1L, "memberA", Grade.VIP);
        memberService.join(memberA);

        Order order = orderService.createOrder(1L,"itemA",10000);
        System.out.println("order = " + order);
        System.out.println(order.calculatePrice());
    }
}

 

MemberServiceTest

package hello.core;

import hello.core.member.Grade;
import hello.core.member.Member;
import hello.core.member.MemberService;
import hello.core.member.MemberServiceImp;
import org.assertj.core.api.Assertions;
import org.junit.jupiter.api.BeforeEach;
import org.junit.jupiter.api.Test;

public class MemberServiceTest {

//    AppConfig appConfig = new AppConfig();
//    MemberService memberService = appConfig.memberService();

    MemberService memberService;
    @BeforeEach
    public void beforeEach()
    {
        AppConfig appConfig = new AppConfig();
        memberService = appConfig.memberService();
    }


    @Test
    void join()
    {
        // given
        Member member = new Member(1L,"memberA", Grade.VIP);

        // when
        memberService.join(member);
        Member findMember = memberService.findMember(1L);

        // then
        Assertions.assertThat(member).isEqualTo(findMember);
    }

}

 

OrderServiceTest

package hello.core;

import hello.core.member.Grade;
import hello.core.member.Member;
import hello.core.member.MemberService;
import hello.core.member.MemberServiceImp;
import hello.core.order.Order;
import hello.core.order.OrderService;
import hello.core.order.OrderServiceImp;
import org.assertj.core.api.Assertions;
import org.junit.jupiter.api.BeforeEach;
import org.junit.jupiter.api.Test;

public class OrderServiceTest {

    MemberService memberService;
    OrderService orderService;

    @BeforeEach
    public void beforeEach()
    {
        AppConfig appConfig = new AppConfig();
        MemberService memberService = appConfig.memberService();
        OrderService orderService = appConfig.orderService();

    }
    @Test
    void createOrder()
    {
        // given
        Long memberId = 1L;
        Member member = new Member(memberId,"memberA", Grade.VIP);
        memberService.join(member);

        // when
        Order order = orderService.createOrder(memberId, "itemA", 100000);
        // then
        Assertions.assertThat(order.getDiscountPrice()).isEqualTo(1000);
    }

}

 

AppConfig 리펙터링

package hello.core;

import hello.core.discount.DiscountPolicy;
import hello.core.discount.FixDiscountPolicy;
import hello.core.member.MemberService;
import hello.core.member.MemberServiceImp;
import hello.core.order.OrderService;
import hello.core.order.OrderServiceImp;

public class AppConfig {

    public MemberService memberService()
    {
        return new MemberServiceImp(getMemberRepository());
    }

    private MemberRepository getMemberRepository() {
        return new MemoryMemberRepository();
    }

    public OrderService orderService()
    {
        return new OrderServiceImp(getMemberRepository(), getPolicy());
    }

    private DiscountPolicy getPolicy() {
        return new FixDiscountPolicy();
    }
}

 

• new MemoryMemberRepository() 중복 제거되었다. 이제 MemoryMemberRepository 를 다른 구현체로 변경할 때 한 부분만 변경하면 된다.
• AppConfig 를 보면 역할과 구현 클래스가 한눈에 들어온다. 애플리케이션 전체 구성이 어떻게 되어있는지 빠르게 파악할 수 있다.

리펙터링 Tip : 단축키 command + option + m

 

 

이제 FixDiscountPolicy를 RateDiscountPolicy로 변경하고자 할 때 AppConfig의 한 줄만 바꾸면 된다!

    private DiscountPolicy getPolicy() {
//        return new FixDiscountPolicy();
        return new RateDiscountPolicy();
    }

---

<정리>

• AppConfig를 통해서 관심사를 확실하게 분리했다.
• 배역, 배우를 생각해보자.
• AppConfig는 공연 기획자다.
• AppConfig는 구체 클래스를 선택한다. 배역에 맞는 담당 배우를 선택한다. 애플리케이션이 어떻게 동작해야 할지 전체 구성을 책임
• 다. 이제 각 배우들은 담당 기능을 실행하는 책임만 지면 된다. OrderServiceImp은 기능을 실행하는 책임만 지면 된다.

 

<참고자료>

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

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