목차
새로운 할인 정책 개발
새로운 할인 정책을 확장해보자!
•
악덕 기획자 : 서비스 오픈 직전에 할인 정책을 지금처럼 고정 금액 할인이 아니라 좀 더 합리적인 주문 금액 당 할인하는 정률% 할인으로 변경하고 싶어요. 예를 들어서 기존 정책은 VIP가 10000원을 주문하든 20000원을 주문하든 항상 1000원을 할인했는데, 이번에 새로 나온 정책은 10%로 지정해두면 고객이 10000원 주문시 1000원을 할인해주고, 20000원 주문시에 2000원을 할인해주는 거에요!
•
순진 개발자 : 제가 처음부터 고정 금액 할인은 아니라고 했잖아요;;
•
악덕 기획자 : 애자일 소프트웨어 개발 선언 몰라요? “계획을 따르기보다 변화에 대응하기를”
•
순진 개발자 : ... (하지만 난 유연한 설계가 가능하도록 객체지향 설계 원칙을 준수했지 후후)
순진 개발자가 정말 객체지향 설계 원칙을 잘 준수 했는지 확인해보자.
이번에는 주문한 금액의 %를 할인해주는 새로운 정률 할인 정책을 추가하자.
RateDiscountPolicy 추가
RateDiscountPolicy 코드 추가
package hello.core.discount;
import hello.core.member.Grade;
import hello.core.member.Member;
public class RateDiscountPolicy implements DiscountPolicy {
private int discountPercent = 10;
@Override
public int discount(Member member, int price) {
if (member.getGrade() == Grade.VIP) {
return price * discountPercent / 100;
} else {
return 0;
}
}
}
Java
복사
Test 작성
package hello.core.discount;
import hello.core.member.Grade;
import hello.core.member.Member;
import org.assertj.core.api.Assertions;
import org.junit.jupiter.api.DisplayName;
import org.junit.jupiter.api.Test;
import static org.assertj.core.api.Assertions.*;
import static org.junit.jupiter.api.Assertions.*;
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
assertThat(discount).isEqualTo(1000);
}
@Test
@DisplayName("VIP가 아니면 할인이 적용되지 않아야 한다.")
void vip_x() {
// given
Member member = new Member(2L, "memberBASIC", Grade.BASIC);
// when
int discount = discountPolicy.discount(member, 10000);
// then
assertThat(discount).isEqualTo(0);
}
}
Java
복사
새로운 할인 정책 적용과 문제점
할인 정책 적용
•
할인 정책을 변경하려면 클라이언트인 OrderServiceImpl 코드를 고쳐야 한다.
public class OrderServiceImpl implements OrderService {
// private final DiscountPolicy discountPolicy = new FixDiscountPolicy();
private final DiscountPolicy discountPolicy = new RateDiscountPolicy();
}
Java
복사
문제점 발견
•
우리는 역할과 구현을 충실하게 분리했다. → 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를 위반하지 않도록 인터페이스에만 의존하도록 의존관계를 변경하면 된다.
public class OrderServiceImpl implements OrderService {
// private final DiscountPolicy discountPolicy = new FixDiscountPolicy();
private DiscountPolicy discountPolicy;
}
Java
복사
•
인터페이스에만 의존하도록 설계와 코드를 변경했다.
•
그런데 구현체가 없는데 어떻게 코드를 실행할 수 있을까?
•
실제 실행을 해보면 NPE(null pointer exception)가 발생한다.
•
이 문제를 해결하려면 누군가가 클라이언트인 OrderServiceImpl에 DiscountPolicy의 구현 객체를 대신 생성하고 주입해주어야 한다.
관심사의 분리
ex) 배우는 배역만을 수행해야지, 대본을 짜고, 자신이 직접 상대 배우를 선택하는 등의 역할을 해서는 안된다. 공연 기획자를 만들고, 공연 기획자의 책임을 확실히 분리해야한다.
AppConfig
•
애플리케이션의 전체 동작 방식을 구성(config)하기 위해, 구현 객체를 생성하고, 연결하는 책임을 가지는 별도의 설정 클래스를 만들자.
AppConfig
package hello.core;
import hello.core.discount.FixDiscountPolicy;
import hello.core.member.MemberService;
import hello.core.member.MemberServiceImpl;
import hello.core.member.MemoryMemberRepository;
import hello.core.order.OrderService;
import hello.core.order.OrderServiceImpl;
public class AppConfig {
public MemberService memberService() {
return new MemberServiceImpl(new MemoryMemberRepository());
}
public OrderService orderService() {
return new OrderServiceImpl(new MemoryMemberRepository(), new FixDiscountPolicy());
}
}
Java
복사
AppConfig.java
•
AppConfig는 애플리케이션의 실제 동작에 필요한 구현 객체를 생성한다.
◦
MemberServiceImpl
◦
MemoryMemberRepository
◦
OrderServiceImpl
◦
FixDiscountPolicy
•
AppConfig는 생성한 객체 인스턴스의 참조(레퍼런스)를 생성자를 통해서 주입(연결)해준다.
◦
MemberServiceImpl → MemoryMemberRepository
◦
OrderServiceImpl → MemoryMemberRepository, FixDiscountPolicy
MemberServiceImpl - 생성자 주입
package hello.core.member;
public class MemberServiceImpl implements MemberService{
private final MemberRepository memberRepository;
public MemberServiceImpl(MemberRepository memberRepository) {
this.memberRepository = memberRepository;
}
@Override
public void join(Member member) {
memberRepository.save(member);
}
@Override
public Member findMember(Long memberId) {
return memberRepository.findById(memberId);
}
}
Java
복사
•
설계 변경으로 MemberServiceImpl 은 MemoryMemberRepository 를 의존하지 않는다!
•
단지 MemberRepository 인터페이스만 의존한다.
•
MemberServiceImpl 입장에서 생성자를 통해 어떤 구현 객체가 들어올지(주입될지)는 알 수 없다.
•
MemberServiceImpl 의 생성자를 통해서 어떤 구현 객체를 주입할지는 오직 외부( AppConfig )에서 결정된다.
•
MemberServiceImpl은 이제부터 의존관계에 대한 고민은 외부에 맡기고 실행에만 집중하면 된다.
클래스 다이어그램
•
객체의 생성과 연결은 AppConfig가 담당한다.
•
DIP 완성 : MemberServiceImpl은 MemberRepository인 추상에만 의존하면 된다. 이제 구체 클래스를 몰라도 된다.
•
관심사의 분리 : 객체를 생성하고 연결하는 역할과 실행하는 역할이 명확히 분리되었다.
회원 객체 인스턴스 다이어그램
•
appConfig 객체는 memoryMemberRepository 객체를 생성하고 그 참조값을 memberServiceImpl 을 생성하면서 생성자로 전달한다.
•
클라이언트인 memberServiceImpl 입장에서 보면 의존관계를 마치 외부에서 주입해주는 것 같다고 해서
DI(Dependency Injection) 우리말로 의존관계 주입 또는 의존성 주입이라 한다.
OrderServiceImpl - 생성자 주입
package hello.core.order;
import hello.core.discount.DiscountPolicy;
import hello.core.discount.FixDiscountPolicy;
import hello.core.discount.RateDiscountPolicy;
import hello.core.member.Member;
import hello.core.member.MemberRepository;
import hello.core.member.MemoryMemberRepository;
public class OrderServiceImpl implements OrderService {
private final MemberRepository memberRepository;
private final DiscountPolicy discountPolicy;
public OrderServiceImpl(MemberRepository memberRepository, DiscountPolicy discountPolicy) {
this.memberRepository = memberRepository;
this.discountPolicy = discountPolicy;
}
@Override
public Order createOrder(Long memberId, String itemName, int itemPrice) {
Member member = memberRepository.findById(memberId);
int discountPrice = discountPolicy.discount(member, itemPrice);
return new Order(memberId, itemName, itemPrice, discountPrice);
}
}
Java
복사
•
설계 변경으로 OrderServiceImpl 은 FixDiscountPolicy 를 의존하지 않는다!
•
단지 DiscountPolicy 인터페이스만 의존한다.
•
OrderServiceImpl 입장에서 생성자를 통해 어떤 구현 객체가 들어올지(주입될지)는 알 수 없다.
•
OrderServiceImpl 의 생성자를 통해서 어떤 구현 객체을 주입할지는 오직 외부( AppConfig )에서 결정한다.
•
OrderServiceImpl 은 이제부터 실행에만 집중하면 된다.
•
OrderServiceImpl 에는 MemoryMemberRepository , FixDiscountPolicy 객체의 의존관계가 주입된다.
AppConfig 실행
사용 클래스 - MemberApp
package hello.core;
import hello.core.member.Grade;
import hello.core.member.Member;
import hello.core.member.MemberService;
import hello.core.member.MemberServiceImpl;
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("new member = " + member.getName());
System.out.println("find Member = " + findMember.getName());
}
}
Java
복사
사용 클래스 - OrderApp
package hello.core;
import hello.core.member.Grade;
import hello.core.member.Member;
import hello.core.member.MemberService;
import hello.core.member.MemberServiceImpl;
import hello.core.order.Order;
import hello.core.order.OrderService;
import hello.core.order.OrderServiceImpl;
public class OrderApp {
public static void main(String[] args) {
AppConfig appConfig = new AppConfig();
MemberService memberService = appConfig.memberService();
OrderService orderService = appConfig.orderService();
Long memberId = 1L;
Member member = new Member(memberId, "memberA", Grade.VIP);
memberService.join(member);
Order order = orderService.createOrder(memberId, "itemA", 10000);
System.out.println("order = " + order);
System.out.println("order.calculatePrice = " + order.calculatePrice());
}
}
Java
복사
테스트 코드 오류 수정
public class MemberServiceTest {
MemberService memberService;
@BeforeEach
public void beforeEach() {
AppConfig appConfig = new AppConfig();
memberService = appConfig.memberService();
}
}
Java
복사
public class OrderServiceTest {
MemberService memberService;
OrderService orderService;
@BeforeEach
public void beforeEach() {
AppConfig appConfig = new AppConfig();
memberService = appConfig.memberService();
orderService = appConfig.orderService();
}
}
Java
복사
@BeforeEach 는 각 테스트를 실행하기 전에 호출된다.
정리
•
AppConfig를 통해서 관심사를 확실하게 분리했다.
•
배역, 배우를 생각해보자.
•
AppConfig는 공연 기획자다.
•
AppConfig는 구체 클래스를 선택한다. 배역에 맞는 담당 배우를 선택한다. 애플리케이션이 어떻게 동작해야 할지 전체 구성을 책임진다.
•
이제 각 배우들은 담당 기능을 실행하는 책임만 지면 된다.
•
OrderServiceImpl 은 기능을 실행하는 책임만 지면 된다.
AppConfig 리팩터링
Java
복사