加入收藏 | 设为首页 | 会员中心 | 我要投稿 李大同 (https://www.lidatong.com.cn/)- 科技、建站、经验、云计算、5G、大数据,站长网!
当前位置: 首页 > 编程开发 > Java > 正文

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

发布时间:2020-12-15 00:01:08 所属栏目:Java 来源:网络整理
导读:我想知道如何减少以下代码的Cyclomatic Complexity,如果这是我应该担心的事情. 请参考方法ValuePojo.getSomething() (请不要担心变量命名,为了清楚起见,这个问题已经重写了) public class ValuePojo{ private ValueTypeEnum type; private BigDecimal value1
我想知道如何减少以下代码的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;
    }
}

解决方法

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

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

这是Stack Overflows上另一个解决此问题的question.

(编辑:李大同)

【声明】本站内容均来自网络,其相关言论仅代表作者个人观点,不代表本站立场。若无意侵犯到您的权利,请及时与联系站长删除相关内容!

    推荐文章
      热点阅读