Объединение условий (Consolidate Conditional Expression)
Полезность страницы:
     
    
    0/100      
    
 
        
      
Есть последовательность проверок, возвращающих один результат.
Объединить их в одно условие и извлечь его.
double disabilityAmount() {
if (_seniority < 2) return 0;
if (_monthsDisabled > 12) return 0;
if (_isPartTime) return 0;
// something

double disabilityAmount() {
if (isNoEligableForDisability()) return 0;
// something
Этот рефакторинг является полезным, когда все проверки по смыслу являются частями одной. В этом случае после проведения рефакторинга можно применить Извлечение метода. Если же это действительно независимые условия, выражающие разные вещи, то не стоит их объединять.
Комментарии