Палец вверх 6
Перевод
Перевод

Как уменьшить «цикломатическую сложность» следующего кода

Мне интересно, как уменьшить Cyclomatic Complexity следующего кода, и если это даже то, что я должен беспокоиться.

Пожалуйста, обратитесь к методу ValuePojo.getSomething () (Пожалуйста, не беспокойтесь о именовании переменных, это было переписано для ясности в этом вопросе)

public class ValuePojo
{
    private ValueTypeEnum type;

    private BigDecimal    value1;

    private BigDecimal    value2;

    private BigDecimal    value3;

    public ValuePojo()
    {
        super();
    }

    /**
     * This method reports as "HIGH Cyclomatic Complexity"
     * 
     * @return
     */
    public BigDecimal getSomething()
    {
        if (this.type == null)
        {
            return null;
        }

        switch (this.type)
        {
            case TYPE_A:
            case TYPE_B:
            case TYPE_C:
            case TYPE_D:
                return this.value1;

            case TYPE_E:
            case TYPE_F:
            case TYPE_G:
            case TYPE_H:
                return this.value2;

            case TYPE_I:
            case TYPE_J:
                return this.value3;
        }

        return null;
    }
}
java code-complexity
задан Corey Scott 9 окт. 2012 г., 8:17:29
источник

2 ответа

Решение 5
Перевод
Перевод

Цикломатическая сложность определяется количеством ветвей исполнения в вашем коде. if - else блокирует, операторы switch - все это увеличивает Cyclomatic сложность вашего кода, а также увеличивает количество тестовых случаев, которые вам понадобятся для обеспечения надлежащего покрытия кода.

Чтобы уменьшить сложность в вашем коде, я бы предложил вам удалить операторы case которые не имеют определенного поведения, и заменить его поведением по default в вашем операторе switch .

Вот еще один вопрос о переполнении стека, который решает эту проблему.

ответ дан Luhar 9 окт. 2012 г., 8:32:07
источник
Палец вверх 8
Перевод
Перевод

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

  public BigDecimal getSomething() {
      if (this.type == null) {
          return null;
      }
      Map<Type,BigDecimal> map = new HashMap<Type,BigDecimal>();
      map.put(TYPE_A, value1);
      map.put(TYPE_B, value1);
      map.put(TYPE_C, value1);
      map.put(TYPE_D, value1);
      map.put(TYPE_E, value2);
      map.put(TYPE_F, value2);
      map.put(TYPE_G, value2);
      map.put(TYPE_H, value2);
      map.put(TYPE_I, value3);
      map.put(TYPE_J, value3);

      return map.get(type);
  }
ответ дан Kim Stebel 9 окт. 2012 г., 9:01:19
источник