Внешний вид сайта:

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

 

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

Дополнить страницу Вы можете помочь другим людям дополнив эту страницу вашими знаниями по данному вопросу!
10:14

Комментарии

Нет комментариев. Ваш будет первым!