Switch 문장을 Polymorphic Dispatcher로 치환하기

switch 문장은 polymorhic dispatch를 사용해서 제거해야 case 문장에 존재하는 코드들로 부터의 영향을 최소화할 수 있다.

사전 준비

git co -b remove-switch-stmt origin/remove-switch-stmt

1. subclassing

Movie 클래스의 determineAmount 메소드는 type code에 의존하고 있다.
type code를 기반으로 Movie 클래스를 NewReleaseMovie, ChildrensMovie, RegularMovie로 subclassing한다.

1-1. red phase

테스트를 먼저 수정.
테스트의 입장으로 보면 변경된 부분은 polymorphic하다.
테스트 코드에서 new Movie를 new NewReleaseMovie 등의 서브클래스 생성으로 변경

1-2. green phase

hot fix를 이용해서 subclasses를 생성한다.
run test하여 제대로 동작하는지 확인

1-3. blue phase

remove duplication
new NewReleaseMovie("New Release 1", Movie.NEW_RELEASE)를 new NewReleaseMovie("New Release 1");

2. Push member down - determineAmount, determineFrequentRentalPoint

Movie 클래스에서 determineAmount, determineFrequentRentalPoint 2개의 메소드를 선택하여 push members down

2-1. green phase

push members down refactoring시 set action / leave abstract declaration을 선택한다.

2-2. blue phase

remove unused code
eclEmma의 code coverage를 수행하면서 반복적으로 각 클래스의 사용되지 않는 코드들을 제거한다.

3. 리팩토링 전후 비교