728x90
반응형
간단한 수식을 대입받는 임시변수를 사용할때는 임시변수 참조 부분을 수식으로 치환
변경 전
double basePrice = anOrder.basePrice();
return (basePrice > 1000);
변경 후
return (anOrder.basePrice() > 10000);
변경 전
double basePrice = _quantity * _itemPrice;
if (basePrice > 1000)
return basePrice * 0.5;
else
return basePrice * 0.9;
변경 후 A
if (getBasePrice() > 1000)
return getBasePrice() * 0.5;
else
return getBasePrice() * 0.9;
double getBasePrice()
{
return _quantity * _itemPrice;
}
변경 후 B
if (double basePrice = getBasePrice() > 1000)
return basePrice * 0.5;
else
return basePrice * 0.9;
double getBasePrice()
{
return _quantity * _itemPrice;
}
728x90
반응형
'IT > 리팩토링' 카테고리의 다른 글
[리팩토링] 메소드 삽입 (0) | 2019.10.25 |
---|---|
[리팩토링] 메소드 추출 (0) | 2019.10.25 |
리팩토링 - 코드 개선 방법 (0) | 2019.10.25 |
[리팩토링] 클래스 멤버변수 이동 (0) | 2019.10.25 |
[리팩토링] 객체간 메소드 이동 (0) | 2019.10.25 |