• 변경할 수 있는 데이터를 최대한 줄이도록 노력해야한다.
• 계산해서 알아낼 수 있는 변수는 제거할 수 있다.
  • 계산 자체가 데이터의 의미를 잘 표현한 경우도 있다.
  • 해당 변수가 어디선가 잘못된 값으로 수정될 수 있는 가능성 제기
• 계산에 필요한 데이터가 변하지 않는 값이라면, 계산의 결과에 해당하는 데이터 역시 불변 데이터이기 때문에 해당 변수는 그대로 유지 가능

코드 예제

public class Discount {

    private double discountedTotal;
    private double discount;

    private double baseTotal;

    public Discount(double baseTotal) {
        this.baseTotal = baseTotal;
    }

    public double getDiscountedTotal() {
        return this.discountedTotal;
    }

    public void setDiscount(double number) {
        this.discount = number;
        this.discountedTotal = this.baseTotal - this.discount;
    }
}

@Test
    void discount() {
        Discount discount = new Discount(100);
//        assertEquals(100, discount.getDiscountedTotal());

        discount.setDiscount(10);
        assertEquals(90, discount.getDiscountedTotal());
    }

• assertEquals(100, discount.getDiscountedTotal());값을 설정하면 당연히 100이 나올줄 알았겠지만 0이나왔다. 왜냐면 baseTotal쪽에 값을 100을 집어넣었기때문에 total은 값이 존재하지 않기 때문이다.

public class Discount {
    
    private double discount;

    private double baseTotal;

    public Discount(double baseTotal) {
        this.baseTotal = baseTotal;
    }

    public double getDiscountedTotal() {
        return this.baseTotal - this.discount;
    }

    public void setDiscount(double number) {
        this.discount = number;
    }
}

파생 함수를 지우고 질의 함수로 변경하여 깔끔하게 리팩토링이 되었다.

ProductionPlan