-
리팩토링 : Consolidate Conditional Expression리팩토링 2012. 12. 3. 09:00
조건 :
같은 결과를 내는 조건문이 연속되어 있다면
적용 :
그것들을 하나의 조건식으로 합쳐서 뽑아내라.
적용전 :
double disabilityAmount() { if (_seniority < 2) return 0; if (_monthsDisabled > 12) return 0; if (_isPartTime) return 0; // compute the disability amount }
적용후 :
double disabilityAmount() { if (isNotEligableForDisability()) return 0; // compute the disability amount }
참조 :
http://www.refactoring.com/catalog/consolidateConditionalExpression.html
'리팩토링' 카테고리의 다른 글
리팩토링 : Convert Dynamic to Static Construction (0) 2012.12.05 리팩토링 : Consolidate Duplicate Conditional Fragments (0) 2012.12.04 리팩토링 : Collapse Hierarchy (0) 2012.12.02 리팩토링 : Change Value to Reference (0) 2012.12.01 리팩토링 : Change Bidirectional Association to Unidirectional (0) 2012.11.30 댓글