암초보

※ 루프 안에 있는 변수나 collecting temporary variable도 아닌 임시변수에 값을 여러 번 대입하는 경우에는, 각각의 대입에 대해서 따로따로 임시변수를 만들어라.

※ 하나의 임시변수를 두 가지 이상의 용도로 사용하면 코드를 보는 사람이 매우 혼란스러움

※ 절차

double temp = 2 * (_height + _width); System.out.println(temp); temp = _height * _width; System.out.println(temp);

After

final double perimeter = 2 * (_height + _width); System.out.println(perimeter); final double area = _height * _width; System.out.println(area);

//////////////////////////////////////////////////////////////////////////////////////////
출처 : 마틴 파울러의 리팩토링
//////////////////////////////////////////////////////////////////////////////////////////

※ 복잡한 수식이 있는 경우에는, 수식의 결과나 또는 수식의 일부에 자신의 목적을 잘 설명하는 이름으로 된 임시변수를 사용하라.

※ 보통 Extract Method로 해결 - 임시변수는 한 메소드의 컨텍스트 내에서만 유용하지만 메소드는 객체의 모든 부분, 다른 객체에서도 유용
※ 지역변수 때문에 Extract Method의 사용이 어려운 경우 Introduce Explaining Variable 사용

※ 절차

1. final 변수를 선언하고, 복잡한 수식의 일부를 이 변수에 대입
2. 원래의 복잡한 수식에서, 임시변수에 대입한 수식을 임시변수로 바꾼다.
3. 컴파일 및 테스트
4. 수식의 다른 부분에 대해서도 위의 작업 반복

Before

double price() {    // price = (base price) - (quantity discount) + (shipping);    return _quantity * itemPrice -       Math.max(0, _quantity - 500) * _itemPrice * 0.05 +       Math.min(_quantity * itemPrice * 0.1, 100.0); }

Step 1.

double price() {    // price = (base price) - (quantity discount) + (shipping);    final double basePrice = _quantity * itemPrice;    return basePrice -       Math.max(0, _quantity - 500) * _itemPrice * 0.05 +       Math.min(_quantity * itemPrice * 0.1, 100.0); }

Step 2.

double price() {    // price = (base price) - (quantity discount) + (shipping);    final double basePrice = _quantity * itemPrice;    return basePrice -       Math.max(0, _quantity - 500) * _itemPrice * 0.05 +       Math.min(basePrice * 0.1, 100.0); }

Step 3. (동일하게 모두 적용)

double price() {    // price = (base price) - (quantity discount) + (shipping);    final double basePrice = _quantity * itemPrice;    final double quantityDiscount = Math.max(0, _quantity - 500) * _itemPrice * 0.05;    final double shipping = Math.min(basePrice * 0.1, 100.0);    return basePrice - quantityDiscount + shipping; }

☞ Extract Method를 사용한 경우
Before

double price() {    // price = (base price) - (quantity discount) + (shipping);    return _quantity * itemPrice -       Math.max(0, _quantity - 500) * _itemPrice * 0.05 +       Math.min(_quantity * itemPrice * 0.1, 100.0); }

After

double price() {    // price = (base price) - (quantity discount) + (shipping);    return basePrice() - quantityDiscount() + shipping(); } private double quantityDiscount() {    return Math.max(0, _quantity - 500) * _itemPrice * 0.05 ; } private double shipping() {    return Math.min(basePrice() * 0.1, 100.0); } private double basePrice() {    return _quantity * itemPrice; }

//////////////////////////////////////////////////////////////////////////////////////////
출처 : 마틴 파울러의 리팩토링
//////////////////////////////////////////////////////////////////////////////////////////

※ 어떤 수식의 결과값을 저장하기 위해서 임시변수를 사용하고 있다면, 수식을 뽑아내서 메소드로 만들고, 임시변수를 참조하는 곳을 찾아 모두 메소드 호출로 바꾼다. 새로 만든 메소드에서도 사용될 수 있다.

※ Extract Method 하기 전의 필수 단계

※ 절차

double getPrice() {    int basePrice = _quantity * _itemPrice;    double discountFactor;    if (basePrice > 1000) discountFactor = 0.95;    else discountFactor = 0.98;    return basePrice * discountFactor; }

Step 1.

double getPrice() {    final int basePrice = _quantity * _itemPrice;    final double discountFactor;    if (basePrice > 1000) discountFactor = 0.95;    else discountFactor = 0.98;    return basePrice * discountFactor; }

Step 2.

double getPrice() {    final int basePrice = basePrice();    final double discountFactor;    if (basePrice > 1000) discountFactor = 0.95;    else discountFactor = 0.98;    return basePrice * discountFactor; } private int basePrice() {    return _quantity * _itemPrice; }

Step 3.

double getPrice() {    final int basePrice = basePrice();    final double discountFactor;    if (basePrice() > 1000) discountFactor = 0.95;    else discountFactor = 0.98;    return basePrice * discountFactor; } private int basePrice() {    return _quantity * _itemPrice; }

Step 4.

double getPrice() {    final double discountFactor;    if (basePrice() > 1000) discountFactor = 0.95;    else discountFactor = 0.98;    return basePrice() * discountFactor; } private int basePrice() {    return _quantity * _itemPrice; }

Step 5.

double getPrice() {    final double discountFactor = discountFactor();    return basePrice() * discountFactor; } private int basePrice() {    return _quantity * _itemPrice; } private double discountFactor() {  if (basePrice() > 1000) return 0.95;    else return 0.98; }

Step 6.

double getPrice() {    return basePrice() * discountFactor(); } private int basePrice() {    return _quantity * _itemPrice; } private double discountFactor() {  if (basePrice() > 1000) return 0.95;    else return 0.98; }

//////////////////////////////////////////////////////////////////////////////////////////
출처 : 마틴 파울러의 리팩토링
//////////////////////////////////////////////////////////////////////////////////////////