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