Wie reduziere ich die "Cyclomatic Complexity" des folgenden Codes

Question

Ich frage mich, wie man die Cyclomatic Complexity des folgenden Codes reduzieren kann und ob das überhaupt etwas ist, um das ich mir Sorgen machen sollte.

Bitte beachten Sie die Methode ValuePojo.getSomething() (Bitte nicht über die Variablennamen Sorge, hat dies aus Gründen der Klarheit neu geschrieben worden, in dieser Frage)

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

Die zyklomatische Komplexität bestimmt wird durch die Anzahl der Ausführungszweige in Ihrem Code. if - else Blöcke, switch Anweisungen - alle erhöhen die Cyclomatic-Komplexität Ihres Codes und erhöhen auch die Anzahl der Testfälle, die Sie benötigen, um eine angemessene Codeabdeckung sicherzustellen.

Um die Komplexität in Ihrem Code zu reduzieren, würde ich Ihnen die case Aussagen deutet darauf hin, entfernen Sie das in Ihrer switch Anweisung mit einem default Verhalten nicht ein definiertes Verhalten hat und ersetzen.

Hier ist eine weitere question auf Stack Overflows, die dieses Problem anspricht.

Answer 2

Wenn Sie wirklich die zyklomatische Komplexität verringern müssen, können Sie eine Karte verwenden. Offensichtlich sollte es in Ihrer Implementierung nur einmal erstellt und initialisiert werden.

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