[Refactoring 2판] Chapter 6-2 함수 인라인하기

리팩터링 2판을 보고 정리한 글로, 생략된 부분이 있거나 잘못 이해한 부분이 있을 수 있습니다.

개념과 원칙

함수 본문코드를 인라인해서 써버리고 함수 이름과 block 을 제거하는 것이 함수 인라인이다. 코드를 독립된 함수롤 추출하는 6-1 함수 추출과 반대되는 개념이다. 이 방법은 설명보다는 코드를 먼저보는게 이해가 빠르다.

코드 예시

리팩토링 전

  function getRating(driver) {
    return moreThanFiveLateDeliveries(driver) ? 2 : 1;
  }

  function moreThanFiveLateDeliveries(driver) {
    // 이 함수에서는 별도의 데이터 가공이 있는 것도 아니고 return 로직이 간단하다.
    // 이런 경우에는 함수로 이렇게 따로 추출하는게 비효율적이다.
    return driver.numberOfLateDeliveries > 5;
  }

리팩토링 후

  function getRating(driver) {
    return (driver.numberOfLateDeliveries > 5) ? 2 : 1;
  }

절차

1. 다형 메서드인지 확인한다.

   서브 클래스에서 오버라이드 하는 메서드는 인라인하면 안 된다. 다형성이란 같은 모양의 코드가 다른 행위를 하는 것을 의미한다. 부모 클래스와 부모 클래스 속성을 상속받은 서브 클래스가 있을 때, 서브 클래스들은 부모 클래스에 이미 있는 메서드와 똑같은 이름으로 메서드를 생성할 수 있는데, 이 경우 메서드 내 로직을 새롭게 정의할 수 있다. 이러한 메서드를 서브 클래스에서 오버라이드 하는 메서드라 한다. 이런 경우에는 메서드 정의를 유지해야 하므로 인라인하면 안되는 것이다.

2. 인라인할 함수를 호출하는 곳을 모두 찾는다.

3. 2 번에서 찾은 곳들은 모두 함수 본문으로 교체한다.

4. 함수 정의를 삭제한다.