Appconfig 리팩터링

 

현채 AppConfig는 중복이 있고, 역할에 따른 구현이 잘 안보인다.

public class AppConfig {

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

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

}

 

아래와 같이 역할과 구현이 분명히 보이도록 수정해 주자.

• MemberRepository가 어떤 구현체를 사용할지 분명하게 보임 --> 변경 용이
• DiscountPolicy가 어떤 구현체를 사용할지 분명하게 보임 --> 변경 용이
• new MemoryMemberRepository()중복이 제거되었다.
• 설계에 대한 그림이 Config에 분명히 드러나야 한다.

public class AppConfig {

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

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

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

    private FixDiscountPolicy discountPolicy() {
        return new FixDiscountPolicy();
    }

 

*GitHub : 210725, Refactoring AppConfig

---

 

GitHub - bsh6463/SpringCoreFunction