메서드 올리기 (Pull Up Method)

Pull Up Method

반대 리팩터링: 메서드 내리기 (Pull Down Method)

참고

Form Template Method

절차

1. 똑같이 동작하는 메서드인지 면밀히 살펴본다.
2. 메서드 안에서 호출하는 다른 메서드와 참조하는 필드들을 슈퍼클래스에서도 호출하고 참조할 수 있는지 확인한다.
3. 메서드 시그니처가 다르다면 함수 선언 바꾸기로 슈퍼클래스에서 사용하고 싶은 형태로 통일한다. (Change Function Declaration)
4. 슈퍼클래스에 새로운 메서드를 생성하고, 대상 메서드의 코드를 복사해넣는다.
5. 정적 검사를 수행한다.
6. 서브클래스 중 하나의 메서드를 제거한다.
7. 테스트한다.
8. 모든 서브클래스의 메서드가 없어질 때 까지 다른 서브클래스의 메서드를 하나씩 제거한다.

예시

• Before

  class Employee {...}
  
  class Salesman extends Employee {
    get name() {...}
  }
  
  class Engineer extends Employee {
    get name() {...}
  }

• After

  class Employee {
    get name() {...}
  }
  
  class Salesman extends Employee {...}
  class Engineer extends Employee {...}

필드 올리기 (Pull Up Field)

Pull Up Field

반대 리팩터링: 필드 내리기 (Pull Down Field)

절차

1. 후보 필드들을 사용하는 곳 모두가 그 필드들을 똑같은 방식으로 사용하는지 면밀히 살핀다.
2. 필드들의 이름이 각기 다르다면 똑같은 이름으로 바꾼다. (Rename Field)
3. 슈퍼클래스에 새로운 필드를 생성한다.
   • 서브클래스에서 이 필드에 접근할 수 있어야한다.
   • protected 로 선언하면 된다.
4. 서브클래스의 필드들을 제거한다.
5. 테스트한다.

예시

• Before

  class Employee {...} // Java
  
  class Salesman extends Employee {
    private String name;
  }
  
  class Engineer extends Employee {
    private String name;
  }

• After

  class Employee {
    protected String name;
  }
  
  class Salesman extends Employee {...}
  class Engineer extends Employee {...}

생성자 본문 올리기 (Pull Up Constructor Body)

Pull Up Constructor Body

절차

1. 슈퍼클래스에 생성자가 없다면 하나 정의한다.
   • 서브클랫스의 생성자들에게서 이 생성자가 호출되는지 확인한다.
2. 문장 슬라이드하기로 공통 문장 모두를 super() 호출 직후로 옮긴다. (Slide Statements)
3. 공통 코드를 슈퍼클래스에 추가하고 서브클래스들에서는 제거한다.
   • 생성자 매개변수 중 공통 코드에서 참조하는 값들을 모두 super() 로 건넨다.
4. 테스트한다.
5. 생성자 시작 부분으로 옮길 수 없는 공통 코드에는 함수 추출하기와 메서드 올리기를 차례로 적용한다.

예시

• Before

  class Party {...}
  
  class Employee extends Party {
    constructor(name, id, monthlyCost) {
      super();
      this._id = id;
      this._name = name;
      this._monthlyCost = monthlyCost;
    }
  }

• After

  class Party {
    constructor(name){
      this._name = name;
    }
  }
  
  class Employee extends Party {
    constructor(name, id, monthlyCost) {
      super(name);
      this._id = id;
      this._monthlyCost = monthlyCost;
    }
  }

메서드 내리기 (Push Down Method)

Push Down Method

반대 리팩터링: 메서드 올리기 (Pull Up Method)

절차

1. 대상 메서드를 모든 서브클래스에 복사한다.
2. 슈퍼클래스에서 그 메서드를 제거한다.
3. 테스트한다.
4. 이 메서드를 사용하지 않는 모든 서브클래스에서 제거한다.
5. 테스트한다.

예시

• Before

  class Employee {
    get quota {...}
  }
  
  class Engineer extends Employee {...}
  class Salesman extends Employee {...}

• After

  class Employee {...}
  class Engineer extends Employee {...}
  class Salesman extends Employee {
    get quota {...}  
  }

필드 내리기 (Push Down Field)

Push Down Field

반대 리팩터링: 필드 올리기 (Pull Up Field)

절차

1. 대상 필드를 모든 서브클래스에 정의한다.
2. 슈퍼클래스에서 그 필드를 제거한다.
3. 테스트한다.
4. 이 필드를 사용하지 않는 모든 서브클래스에서 제거한다.
5. 테스트한다.

예시

• Before

  class Employee {        // Java
    private String quota;
  }
  
  class Engineer extends Employee {...}
  class Salesman extends Employee {...}

• After

  class Employee {...}
  class Engineer extends Employee {...}
  
  class Salesman extends Employee {
    protected String quota;
  }

⭐ 타입 코드를 서브클래스로 바꾸기 (Replace Type Code with Subclasses)

Replace Type Code with Subclasses

반대 리팩터링: 서브클래스 제거하기 (Remove Subclass)

절차

1. 타입 코드 필드를 자가 캡슐화한다.
2. 타입 코드 값 하나를 선택하여 그 값에 해당하는 서브클래스를 만든다.
   • 타입 코드 Getter 메서드를 오버라이드하여 해당 타입 코드의 리터럴 값을 반환하게 한다.
3. 매개변수로 받은 타입 코드와 방금 만든 서브클래스를 매핑하는 선택 로직을 만든다.

예시

• Before

  function createEmployee(name, type) {
    return new Employee(name, type);
  }

• After

  function createEmployee(name, type) {
    switch (type) {
      case "engineer": return new Engineer(name);
      case "salesman": return new Salesman(name);
      case "manager":  return new Manager (name);
    }

서브 클래스 제거하기 (Remove Subclass)

Remove Subclass

반대 리팩터링: 타입 코드를 서브클래스로 바꾸기 (Replace Type Code with Subclasses)

절차

1. 서브클래스의 생성자를 Factory 함수로 바꾼다. (Replace Constructor with Factory Function)
2. 서브클래스의 타입을 검사하는 코드가 있다면 그 검사 코드에 함수 추출하기와 함수 옮기기를 차례로 적용하여 슈퍼클래스로 옮긴다.
   • 하나 변경할 때마다 테스트한다.
3. 서브클래스의 타입을 나타내는 필드를 슈퍼클래스에 만든다.
4. 서브클래스를 참조하는 메서드가 방금 만든 타입 필드를 이용하도록 수정한다.
5. 서브클래스를 지운다.
6. 테스트한다.

예시

• Before

  class Person {
    get genderCode() {return "X";}
  }
  class Male extends Person {
    get genderCode() {return "M";}
  }
  class Female extends Person {
    get genderCode() {return "F";}
  }

• After

  class Person {
    get genderCode() {return this._genderCode;}
  }

슈퍼클래스 추출하기 (Extract Superclass)

Extract Superclass

대안 리팩터링: 클래스 추출하기 (Extract Class)

• 선택 기준은 중복 동작을 상속으로 해결하느냐 위임으로 해결하느냐에 달렸다.
• 나중에라도 슈퍼클래스 위임으로 바꾸기를 활용하면 된다.

절차

1. 빈 슈퍼클래스를 만든다. 원래 클래스들이 새 클래스를 상속하도록 한다.
2. 테스트한다.
3. 생성자 본문 올리기, 메서드 올리기, 필드 올리기를 차례로 적용하여 공통 원소를 슈퍼클래스로 옮긴다.
4. 서브클래스에 남은 메서드들을 검토한다.
   • 공통되는 부분이 있다면 함수로 추출한 다음 메서드 올리기를 적용한다.
5. 원래 클래스들을 사용하는 코드를 검토하여 슈퍼클래스의 인터페이스를 사용하게 할지 고민해본다.

예시

• Before

  class Department {
    get totalAnnualCost() {...}
    get name() {...}
    get headCount() {...}
  }
  
  class Employee {
    get annualCost() {...}
    get name() {...}
    get id() {...}
  }

• After

  class Party {
    get name() {...}
    get annualCost() {...}
  }
  
  class Department extends Party {
    get annualCost() {...}
    get headCount() {...}
  }
  
  class Employee extends Party {
    get annualCost() {...}
    get id() {...}
  }

계층 합치기 (Collapse Hierarchy)

Collapse Hierarchy

절차

1. 두 클래스 중 제거할 것을 고른다.
2. 필드 올리기와 메서드 올리기 혹은 필드 내리기와 메서드 내리기를 적용하여 모든 요소를 하나의 클래스로 옮긴다.
3. 제거할 클래스를 참조하던 모든 코드가 남겨질 클래스를 참조하도록 고친다.
4. 빈 클래스를 제거한다.
5. 테스트한다.

예시

• Before

  class Employee {...}
  class Salesman extends Employee {...}

• After

  class Employee {...}

⭐ 서브클래스를 위임으로 바꾸기 (Replace Subclass with Delegate)

Replace Subclass with Delegate

• 상속에는 단점이 있다.
  1. 무언가가 달라져야 하는 이유가 여러 개여도 상속에서는 그중 단 하나의 이유만 선택해 기준으로 삼을 수밖에 없다.
  2. 상속은 클래스들의 관계를 아주 긴밀하게 결합한다.

절차

1. 생성자를 호출하는 곳이 많다면 생성자를 Factory 함수로 바꾼다. (Replace Constructor with Factory Function)
2. 위임으로 활용한 빈 클래스를 만든다.
   • 이 클래스의 생성자는 서브클래스에 특화된 데이터를 전부 받아야 하며, 보통은 슈퍼클래스를 가리키는 역참조도 필요하다.
3. 위임을 저장할 필드를 슈퍼클래스에 추가한다.
4. 서브클래스 생성 코드를 수정하여 위임 인스턴스를 생성하고, 위임 필드에 대입해 초기화한다.
5. 서브클래스의 메서드 중 위임 클래스로 이동할 것을 고른다.
6. 함수 옮기기를 적용해 위임 클래스로 옮긴다. 원래 메서드에서 위임하는 코드는 지우지 않는다.
7. 서브클래스 외부에도 원래 메서드를 호출하는 코드가 있다면 서브클래스의 위임 코드를 슈퍼클래스로 옮긴다.
   • 이떄 위임이 존재하는지를 검사하는 보호 코드를 감싸야 한다.
   • 호출하는 외부 코드가 없다면 원래 메서드는 죽은 코드가 되므로 제거한다.
8. 테스트한다.
9. 서브클래스의 모든 메서드가 옮겨질 때까지 5~8 과정을 반복한다.
10. 서브클래스들의 생성자를 호출하는 코드를 찾아서 슈퍼클래스의 생성자를 사용하도록 수정한다.
11. 테스트한다.
12. 서브클래스를 삭제한다. (Remove Dead Code)

예시

• Before

  class Order {
    get daysToShip() {
      return this._warehouse.daysToShip;
    }
  }
  
  class PriorityOrder extends Order {
    get daysToShip() {
      return this._priorityPlan.daysToShip;
    }
  }

• After

  class Order {
    get daysToShip() {
      return (this._priorityDelegate)
        ? this._priorityDelegate.daysToShip
        : this._warehouse.daysToShip;
    }
  }
  
  class PriorityOrderDelegate {
    get daysToShip() {
      return this._priorityPlan.daysToShip
    }
  }

⭐ 슈퍼클래스를 위임으로 바꾸기 (Replace Superclass with Delegate)

Replace Superclass with Delegate

참고

bliki: TypeInstanceHomonym

절차

1. 슈퍼클래스 객체를 참조하는 필드를 서브클래스에 만든다.
   • 위임 참조를 새로운 슈퍼클래스 인스턴스로 초기화한다.
2. 슈퍼클래스의 동작 각각에 대응하는 전달 함수를 서브클래스에 만든다.
   • 서로 관련된 함수끼리 그룹으로 묶어 진행하며, 그룹을 하나씩 만들 때마다 테스트한다.
3. 슈퍼클래스의 동작 모두가 전달 함수로 오버라이드되었다면 상속 관계를 끊는다.

예시

• Before

  class List {...}
  class Stack extends List {...}

• After

  class Stack {
    constructor() {
      this._storage = new List();
    }
  }
  class List {...}