Question

我想知道如何降低以下代码的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;
    }
}

Answer 1

如果您确实需要降低圈复杂度，可以考虑使用Map。显然，在您的实现中，它只应该被创建并初始化一次。

  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);
  }

Answer 2

Cyclomatic Complexity由代码中的执行分支数决定。 if - else块，switch语句 - 都会增加代码的Cyclomatic复杂性，并且还会增加确保适当代码覆盖所需的测试用例数。

为了降低代码的复杂性，我建议您删除没有定义行为的case语句，并将其替换为default语句中的switch行为。

这是Stack Overflows上的另一个question来解决这个问题。

如何减少以下代码的“Cyclomatic Complexity”

2 个答案: