Làm cách nào để giảm "Độ phức tạp Cyclomatic" của mã sau đây

Question

Tôi tự hỏi làm thế nào để giảm Độ phức tạp Cyclomatic của mã sau đây và nếu điều này thậm chí là một cái gì đó mà tôi nên lo lắng về.

Vui lòng tham khảo ValuePojo.getSomething() phương pháp (Xin đừng lo lắng về việc đặt tên biến, điều này đã được tái bản cho rõ ràng trong câu hỏi này)

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

Các Cyclomatic phức tạp được xác định bởi số lượng nhánh thực hiện trong mã của bạn. if - else khối, switch báo cáo - tất cả tăng độ phức tạp của Cyclomatic của mã của bạn và cũng tăng số lượng các trường hợp kiểm tra bạn sẽ cần phải đảm bảo bảo hiểm mã thích hợp.

Để giảm độ phức tạp trong mã của bạn, tôi khuyên bạn nên xóa các tuyên bố case không có hành vi được xác định và thay thế bằng hành vi default trong tuyên bố switch của bạn.

Dưới đây là một question trên Stack Overflows khác giải quyết vấn đề này.

Answer 2

Nếu bạn thực sự cần giảm độ phức tạp của chu trình, bạn có thể xem xét sử dụng Bản đồ. Rõ ràng, trong quá trình thực hiện của bạn, nó chỉ nên được tạo và khởi tạo một lần.

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