[Refactoring 2판] Chapter 6-3 변수 추출하기

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

개념과 원칙

표현식에 이름을 붙이고 싶을 때 변수 추출을 한다. 현재 함수 안에서만 의미가 있다면 변수로 추출하는 것이 좋다. 그런데 함수를 벗어난 넓은 문맥에서까지 의미가 된다면 함수로 추출하는 것이 좋다. 그래야 import 해서 여기저기서 공통 로직을 사용해 중복을 줄일 수 있기 때문이다. (DRY)

코드 예시

리팩토링 전

  ...
  return order.quantity * order.itemPrice - 
    Math.max(0, order.quantity - 500) * order.itemPrice * 0.05 +
      Math.min(order.quantity * order.itemPrice * 0.1, 100);

리팩토링 후

  ...
  const basePrice = order.quantity * order.itemPrice;
  const quantityDiscount = Math.max(0, order.quantity - 500) * order.itemPrice * 0.05;
  const shipping = Math.min(order.quantity * order.itemPrice * 0.1, 100);
  return basePrice - quantityDiscount + shipping;

3줄짜리 코드가 4줄이 되었지만 코드 이해는 변수명으로 인해 훨씬 쉬워졌다. 👍

절차

1. 추출하려는 표현식에 부작용은 없는지 확인한다.
2. 불변 변수(const)를 하나 선언하고 이름을 붙이고 코드를 옮긴다.
3. 원본 표현식의 정의를 삭제한다.

클래스 안에서의 변수 추출 (메서드 추출)

클래스 문맥 안에서의 케이스를 살펴보자.

리팩토링 전

  class Order {
    constructor(aRecord) {
      this._data = aRecord
    }

    get quantity() { return this._data.quantity; }
    get itemPrice() { return this._data.itemPrice; }

    get price() {
      return this.quantity * this.itemPrice -
        Math.max(0, order.quantity - 500) * order.itemPrice * 0.05 +
          Math.min(order.quantity * order.itemPrice * 0.1, 100);
    }
  }

리팩토링 후

이 예시를 보면 클래스 안에서는 변수로 추출하려는 부분이 price 메서드 범위를 넘어서 Order 클래스 전체에 적용된다. 클래스 전체에 영향을 줄 때는 변수가 아닌 메서드로 추출하는 것을 권장한다.

  class Order {
    constructor(aRecord) {
      this._data = aRecord
    }

    get quantity() { return this._data.quantity; }
    get itemPrice() { return this._data.itemPrice; }

    get price() {
      return this.basePrice - this.quantityDiscount + this.shipping;
    }

    get basePrice()        { return this.quantity * this.itemPrice }
    get quantityDiscount() { return Math.max(0, this.quantity - 500) * this.itemPrice * 0.05; }
    get shipping()         { return Math.min(this.basePrice * 0.1, 100);}
  }

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

1. 코드를 짤 때는, 해당 코드가 속한 범위와 문맥까지 넓게 고민하자.
2. 객체는 특정 로직과 데이터를 외부와 공유하려 할 때 공유할 정보를 설명해주는 적당한 크기의 문맥이 되어준다. 덩치가 큰 클래스에서 공통 동작을 별도 이름으로 뽑아내서 추상화해두면 그 객체를 다룰 때 쉽게 활용할 수 있어 매우 유용하다.