메서드 기능이 너무 단순해서 메서드명만 봐도 너무 뻔할 땐,
그 메서드의 기능을 호출하는 메서드에 넣어버리고 그 메서드는 삭제하자
특징
- 메서드명에 모든 기능이 반영될 정도로 메서드 기능이 지나치게 단순하면, 그 메서드는 없애야한다
- 단순 위임 기능을 하는 메서드들 때문에 코드가 복잡해질 수 있다
- 리팩토링의 핵심은 한눈에 파악할 수 있는 직관적인 메서드를 만드는 것과 메서드를 간결하게 만드는 것이긴 하지만!
- 잘못 쪼개진 메서드에도 적용할 수 있다
- 잘못 쪼개진 메서드의 내용을 다시 큰 메서드에 직접 삽입한 후, 다시 작은 메서드로 추출한다
방법
- 메서드가 재정의 되어있지 않는지 확인하자
- 당연한 소리지만 하위클래스에서 재정의하고 있는 메서드를 삭제하면 안된다
- 그 메서드를 호출하는 부분을 모두 찾는다
- 각 호출 부분을 메서드 내용으로 교체한다
- 테스트를 실시하고, 메서드 정의를 삭제한다
참고 : 마틴 파울러, 『리팩토링』, 김지원 옮김, 한빛미디어(2012)