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

前端之家收集整理的这篇文章主要介绍了java – 如何减少以下代码的“Cyclomatic Complexity”前端之家小编觉得挺不错的,现在分享给大家,也给大家做个参考。
我想知道如何减少以下代码的Cyclomatic Complexity,如果这是我应该担心的事情.

请参考方法ValuePojo.getSomething()
(请不要担心变量命名,为了清楚起见,这个问题已经重写了)

  1. public class ValuePojo
  2. {
  3. private ValueTypeEnum type;
  4.  
  5. private BigDecimal value1;
  6.  
  7. private BigDecimal value2;
  8.  
  9. private BigDecimal value3;
  10.  
  11. public ValuePojo()
  12. {
  13. super();
  14. }
  15.  
  16. /**
  17. * This method reports as "HIGH Cyclomatic Complexity"
  18. *
  19. * @return
  20. */
  21. public BigDecimal getSomething()
  22. {
  23. if (this.type == null)
  24. {
  25. return null;
  26. }
  27.  
  28. switch (this.type)
  29. {
  30. case TYPE_A:
  31. case TYPE_B:
  32. case TYPE_C:
  33. case TYPE_D:
  34. return this.value1;
  35.  
  36. case TYPE_E:
  37. case TYPE_F:
  38. case TYPE_G:
  39. case TYPE_H:
  40. return this.value2;
  41.  
  42. case TYPE_I:
  43. case TYPE_J:
  44. return this.value3;
  45. }
  46.  
  47. return null;
  48. }
  49. }

解决方法

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

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

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

猜你在找的Java相关文章