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. (编辑:李大同) 【声明】本站内容均来自网络,其相关言论仅代表作者个人观点,不代表本站立场。若无意侵犯到您的权利,请及时与联系站长删除相关内容! |