Уменьшение цикломатической сложности оператора Switch — Sonar

Я считаю, что это Sonar предупреждение. Я думаю, что Sonar предупреждения — это не обязательные правила, а просто рекомендации. Ваш блок кода READABLE и MAINTAINABLE как есть. Это уже просто, но если вы действительно хотите изменить это, вы можете попробовать эти два подхода ниже и посмотреть, станет ли сложность ниже:

Примечание: сейчас у меня нет с собой компилятора, поэтому могут быть ошибки, заранее извините.

Первый подход:

Map<String, String> multipliers = new HashMap<String, Float>();
    map.put("country", country);
    map.put("exchange", exchange);
    map.put("ccp", ccp);
    map.put("tenant", tenant);

Затем мы можем просто использовать карту, чтобы получить нужный элемент.

    return map.get(type) == null ? name : map.get(type).getName() + HOLIDAY_CALENDAR;

2-й подход:

Все ваши объекты имеют один и тот же метод, поэтому вы можете добавить в него метод Interface с getName() и изменить подпись метода, например:

getCalendarName(YourInterface yourObject){
    return yourObject == null ? name : yourObject.getName() + HOLIDAY_CALENDAR;
}

Если ваша первая цель состоит только в том, чтобы уменьшить цикломатическую сложность, вы должны создать методы для каждого способа получения имени, как показано ниже.

 public String getCalenderName() {
    switch (type) {
    case COUNTRY:
        return getCountryName();
    case CCP:
        return getCcpName();
    case EXCHANGE:
        return getExchangeName();
    case TENANT:
        return getTenantName();
    default:
        return name;
    }
}

private String getCountryName() {
    return country == null ? name : country.getName() + HOLIDAY_CALENDAR;
}

private String getCcpName() {
    return ccp == null ? name : ccp.getName() + " CCP" + HOLIDAY_CALENDAR;
}

private String getExchangeName() {
    return exchange == null ? name : getName.toString() + HOLIDAY_CALENDAR;
}

private String getTenantName() {
    return tenant == null ? name : getName.toString() + HOLIDAY_CALENDAR;
}

Обратите внимание, что в вашем конкретном примере я полагаю, что у вас есть 1 класс, который собирает (по крайней мере) 4 очень похожих поведения. Рефакторинг, безусловно, имел бы больше смысла, чтобы иметь, например, одну базовую реализацию (абстрактную или нет) и 4 других унаследованных класса.

Насколько мне известно, не используйте оператор return в операторе switch. Примените эту логику после оператора switch, используя переменную. Создайте метод для проверки нулевого значения и вызовите этот метод из переключателя, после чего вы сможете уменьшить цикломатическую сложность.

Я думаю, вы можете снизить сложность, просто убедившись, что что-то еще исправлено в вашем коде.

Возьмем кейс:

case COUNTRY:
  return country == null ? name : country.getName() + HOLIDAY_CALENDAR;

Это означает, что если тип календаря COUNTRY, страна, с которой связан календарь, может быть null. Это то, что вы должны предотвратить по дизайну, потому что я не вижу причины, по которой это может быть допустимой ситуацией. Зачем вам страновой календарь без страны?

Поэтому убедитесь, что с календарем связан ненулевой объект, как только вы назначите ему type. Таким образом, ваши случаи будут похожи на

case COUNTRY:
  return country.getName() + HOLIDAY_CALENDAR;

что снижает вашу цикломатическую сложность до 5.

Вы можете удалить все нулевые сравнения и проверить их перед переключением регистра. В этом случае сложность уменьшится на 4 или может быть больше.

Если ваши объекты: страна, cpp, биржа и арендатор используют один и тот же интерфейс, например. ObjectWithGetName, вы можете реорганизовать свой код следующим образом:

  public String getCalenderName() {
    ObjectWithGetNameMethod calendarType = null;
    switch (type) {
        case COUNTRY:
           calendarType = country;
           break;
        case CCP:
           calendarType = cpp;
           break;
        case EXCHANGE:
           calendarType = exchange;
           break;
        case TENANT:
           calendarType = tenant;
           break;
        default:
           calendarType = null;
    }
    return (calendarType != null ? (calendarType.getName() + HOLIDAY_CALENDAR) : name);
  }

Также я думаю, что было бы неплохо переместить переключатель в отдельный метод, так как похоже, что что-то будет использоваться во многих разных местах.

public String getName() {
    if (type == null) {
        return name;
    }
    if (type == BusinessCalendarType.COUNTRY) {
        return country == null ? name : country.getName() + HOLIDAY_CALENDAR;
    } else if (type == BusinessCalendarType.CCP) {
        return ccp == null ? name : ccp.getName() + " CCP" + HOLIDAY_CALENDAR;
    } else if (type == BusinessCalendarType.EXCHANGE) {
        return exchange == null ? name : exchange.getName() + HOLIDAY_CALENDAR;
    } else if (type == BusinessCalendarType.TENANT)  {
        return tenant == null ? name : tenant.getName() + HOLIDAY_CALENDAR;
    } else {
        return name;
    }
}

это сработало для меня