-
리팩토링 : Replace Temp with Query리팩토링 2013. 2. 18. 09:00
조건
표현식의 결과를 저장하기위해 임시 변수를 사용하고 있다면
적용
그 표현식을 메소드안으로 뽑아내고,
임시변수에 대한 모든 참조를 표현식으로 바꾸시오.
이렇게하면, 새로운 메소드는 다른 메소드에서 사용될 수 있음.
적용전
double basePrice = _quantity * _itemPrice; if (basePrice > 1000) return basePrice * 0.95; else return basePrice * 0.98;
적용후
if (basePrice() > 1000) return basePrice() * 0.95; else return basePrice() * 0.98; ... double basePrice() { return _quantity * _itemPrice; }
참조
http://www.refactoring.com/catalog/replaceTempWithQuery.html
'리팩토링' 카테고리의 다른 글
리팩토링 : Replace Type Code with State/Strategy (0) 2013.02.20 리팩토링 : Replace Type Code with Class (0) 2013.02.19 리팩토링 : Replace Subclass with Fields (0) 2013.02.17 리팩토링 : Replace Static Variable with Parameter (0) 2013.02.16 리팩토링 : Replace Recursion with Iteration (0) 2013.02.15 댓글