3장

중복된 코드

한 클래스의 서로 다른 두 메소드 안에 같은 코드가 있는 경우 : Extract Method

동일한 수퍼클래스를 갖는 두 서브클래스에서 같은 코드가 있는 경우 : Extract Method 후 Pull Up Method

비슷하지만 같지는 않다면 : 비슷한 부분 Extract Method 후 Form Template Method 사용 가능한지 확인

같은 작업을 하지만 다른 알고리즘을 사용한다면 : 두 알고리즘 중 더 명확한 것 선택해서 Substitute Algorithm 사용

서로 관계 없는 두 클래스에서 중복된 코드가 있는 경우 : Extract Class 사용한 다음 양쪽에서 이 새로운 클래스를 사용

긴 메소드

• 어떤 것에 대해 주석을 달아야 할 필요를 느낄 때마다 대신 메소드를 작성하라

단지 한 줄의 코드라 할지라도 그것이 설명을 필요로 한다면 메소드로 뽑아낼 가치가 있다.

• 메소드 길이를 줄이기 위해선 Extract Method

• 메소드에 파라미터와 임시변수가 많다면

임시변수를 제거 하기 위해 Replace Temp with Query
긴 파라미터 리스트는 Introduce Parameter Object와 Preserve Whole Object로 짧게 할 수 있음
그래도 많다면 Replace Method with Method Object라는 중장비 동원해야 할 때

• 조건문과 루프 또한 메소드 추출이 필요하다는 신호 : Decompose Conditional

거대한 클래스

• 지나치게 많은 인스턴수 변수

Extract Class

새로 만들 클래스가 서브클래스로서 의미가 있으면 Extract Subclass

• 코드가 많은 클래스

클라이언트가 클래스를 어떻게 쓰게 할 지 결정하고 각각의 사용 방법에 대해 Extract Interface

긴 파라미터 리스트

대부분의 변경은 객체를 넘겨서 해결

이미 알고 있는 객체에 요청하여 파라미터의 데이터를 얻을 수 있으면

• Replace Parameter with Method

한 객체로부터 주워 모은 데이터 뭉치를 그 객체 자체로 바꾸기 위해

• Preserve Whole Object

객체와 관계 없는 여러 개의 데이터 아이템이 있는 경우

• Introduce Parameter Object

확산적 변경

한 클래스가 여러 종류의 변경 때문에 시달리는 경우

• Extract Class 사용해서 특정 원인에 대해 변해야 하는 것들을 묶어줌

산탄총 수술

확산적 변경과 반대

하나를 변경했을 때 많은 클래스를 고쳐야 하는 경우

• Move Method, Move Field를 사용해서 변경해야 할 부분을 모두 하나의 클래스로

• 기존의 클래스 중에서 메소드나 필드가 옮겨갈 후보가 적당히 없다면 새로 하나 만들어라

데이터 덩어리

• Extract Class

Switch 문

Switch문에 코드를 추가하려면 중복된 모든 switch문 찾아 바꿔줘야 함

Switch 보면 항상 다형성을 생각해야 함

만약 하나의 메소드에만 영향을 미치는 몇 개의 경우가 있다면, 굳이 바꿀 필요 없다.

• 이런 경우 다형성은 과하다

• Replace parameter with Explicit methods

• 만약 조건 중 null이 있는 경우라면 Intruduce Null Object

메시지 체인

• 클라이언트가 어떤 객체를 얻기 위해 다른 객체에 물어보고, 또 다른 객체에 물어보고 ...

• Hide Delegate

데이터 클래스

필드와 각 필드에 대한 get/set 메소드만 가지고있는 클래스

약간의 책임을 가질 필요가 있다.

get/set 메소드가 다른 클래스에서 사용되는지 찾아보고 동작을 데이터 클래스로 옮기기 위해

• Move Method 시도

• 메소드 전체 옮길 수 없을 땐 Extract Method

그 후에 get/set 메소드에 대해 Hide Method를 사용할 수 있다

거부된 유산

• 서브클래스가 부모 클래스로부터 주어진 것을 원치 않을 땐 단지 필요한 몇개만 고른다

• 새로운 형제 클래스 만들고 Push Down Method, Push Down Field 사용해서 사용되지 않는 메소드를 모두 형제 메소드로 옮겨라