Обзор темы [Рефакторинг кода]

Объединение условий (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

 

Этот рефакторинг является полезным, когда все проверки по смыслу являются частями одной. В этом случае после проведения рефакторинга можно применить Извлечение метода. Если же это действительно независимые условия, выражающие разные вещи, то не стоит их объединять.