[Refactoring 2판] Chapter 6-1 함수 추출하기

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

개념과 원칙

함수 추출 (function extraction) 이란, 코드를 독립된 함수로 묶는 것이다. 묶는 기준은 길이가 될 수도 있고, 재사용성을 기준으로 할 수도 있지만 리팩토링 2판의 지은이는 목적과 구현을 분리 하는 방식이 가장 합리적인 함수 추출 기준이라고 본다. 함수의 목적은 함수의 이름에서 바로 알 수 있어야 하며, 구현되는 부분은 함수 본문 코드에 넣는다.

절차

1. 함수를 새로 만들고 목적을 잘 드러내는 이름을 붙인다.'어떻게' 가 아닌 '무엇을' 하는 지가 드러나야 한다.

   이름이 쉽게 떠오르지 않는다면 함수로 추출하면 안 된다는 신호라니, 이런 경우 '함수로 따로 분리하는 게 최선인가?' 고민해보자. 중첩 함수를 지원하는 언어 (내가 쓰는 자바스크립트는 여기에 해당된다)를 사용한다면 추출한 함수를 원래 함수 안에 중첩시킨다. A 라는 함수 안에 중첩함수로 들어가는 b, c 함수간에는 범위를 공유하기 때문에 유효범위를 벗어난 변수를 처리하는 작업을 줄일 수 있다.

2. 추출할 코드를 원본 함수에서 복사하여 새 함수에 붙여넣는다.

3. 추출한 코드 중 원본 함수의 지역 변수를 참조하거나 추출한 함수의 유효범위를 벗어나는 변수는 없는지 검사하고, 있다면 새로 생성한 함수에 매개변수로 전달한다.

   추출한 코드에서만 사용하는 변수가 추출한 함수 밖에 선언되어 있다면 추출한 함수 안에서 선언하도록 수정한다. 추출한 코드 안에서 값이 바뀌는 변수가 하나이면 추출한 코드를 해당 변수에 새로운 값을 대입해주는 함수로 만들어 준다. 값이 바뀌는 지역변수가 너무 많고 복잡하다면, 한 변수당 역할 하나를 주는 변수 쪼개기 방법과 임시 변수를 질의 함수 (해당 변수에 새로운 값을 대입해주는 함수로 이해했다)로 바꾸는 리팩토링 방식을 사용해 코드를 단순하게 바꾼 후 함수 추출을 다시 시도한다.

4. 원본 함수에서 추출한 코드 부분은 새로 만든 함수로 호출한다.

5. 다른 코드에 방금 추출한 것과 똑같거나 비슷한 코드가 없는지 살피고 있다면 추출한 새 함수를 호출하도록 바꿀지 검토한다.

코드 예시

리팩토링 전

  function printOwing(invoice) {
   printBanner();
   let outstanding = 0;
   for (const order of invoice.orders) {
     outstanding += order.amount;
   }

   console.log('고객명: ${invoice.customer}');
   console.log('채무액: ${outstanding}');
  }

리팩토링 후

  function printOwing(invoice) {
   printBanner();
   const outstanding = calculateOutstanding(invoice);
   printDetails(outstanding);
  }

  function printDetails(outstanding) {
    console.log('고객명: ${invoice.customer}');
    console.log('채무액: ${outstanding}');
  }

  function calculateOutstanding(invoice) {
    let outstanding = 0;
    for (const order of invoice.orders) {
     outstanding += order.amount;
    }

    return outstanding;
  }

💡💡💡💡💡💡💡💡💡💡💡💡💡💡💡💡💡💡

1. 함수가 짧으면 캐싱하기가 더 쉬워서 컴파일러가 최적화하는데 오히려 유리하다.
2. 변수 선언문은 변수가 사용되는 코드 근처로 위치를 이동시킨다.