ReplaceTempwithQuery以查询取代临时变量.PPTVIP

Replace Nested Conditional with Guard Clauses double getPayAmount() { double result; if (_isDead) result = deadAmount(); else { if (_isSeparated) result = separatedAmount(); else { if (_isRetired) result = retiredAmount(); else result = normalPayAmount(); }; } 　　　　return result; }; double getPayAmount() { if (_isDead) return deadAmount(); if (_isSeparated) return separatedAmount(); if (_isRetired) return retiredAmount(); return normalPayAmount(); }; Split Loop - 1 void printValues() { double averageAge = 0; double totalSalary = 0; for (int i = 0; i people.length; i++) { averageAge += people[i].age; totalSalary += people[i].salary; } averageAge = averageAge / people.length; System.out.println(averageAge); System.out.println(totalSalary); } void printValues() { double totalSalary = 0; for (int i = 0; i people.length; i++) { totalSalary += people[i].salary; } double averageAge = 0; for (int i = 0; i people.length; i++) { averageAge += people[i].age; } averageAge = averageAge / people.length; System.out.println(averageAge); System.out.println(totalSalary); } Split Loop - 2 void printValues() { System.out.println(averageAge()); System.out.println(totalSalary()); } private double averageAge() { double result = 0; for (int i = 0; i people.length; i++) { result += people[i].age; } return result / people.length; } private double totalSalary() { double result = 0; for (int i = 0; i people.length; i++) { result += people[i].salary; } return result; } 简化条件表达式 分解表达式 if(date.before(SUMMER_START) || date.after(SUMMER_END)) charge = quantity * _winterRate + _winterServiceCharge; else charge = quantity * _summerRate if(notSummer(date)) charge = winterCharge(quantity); else charge = summerCharge(quantity); Remove Double Negative if ( !item.isNotFound() ) if ( item.isFound() ) 在对象之间搬移特性 搬移函数（Move Method) 搬移值域(Move Field) 提炼类(Extract Class) 将类内联化(Inline Class) 隐藏[委托关系](Hide Delegate) 移除中间人(Remove Middle Man) 引入外加函数