Giảm lệnh if-else trong Java

Question

Tôi có đoạn mã sau:

void f(String t) 
{ 
    if(t.equals("a")) 
    { 
    someObject.setType(ObjectType.TYPE_A); 
    } 
else if(t.equals("b")) 
    { 
    someObject.setType(ObjectType.TYPE_B); 
    } 

// 50 more similar code 

} 

Có cách nào đơn giản để ghi lại các điều kiện if-else như vậy là không có nhiều mã?

Answer 1

Bạn nên sử dụng thứ gì đó để loại bỏ sự lặp lại someObject.setType(ObjectType....)) Nếu ObjectType là enum, sau đó viết phương thức tương tự với valueOf để đạt được điều đó. Xem nếu bạn thích loại hình giải pháp:

void f(String t) { someObject.setType(ObjectType.byName(t)); } 

enum ObjectType { 
    TYPE_A, TYPE_B; 
    public static ObjectType byName(String name) { 
    return valueOf("TYPE_" + name.toUpperCase()); 
    } 
} 

Answer 2

Sử dụng Map (bạn sẽ phải điền) bản đồ từ String cho bất kỳ loại giá trị ObjectType.TYPE_x của bạn.

Answer 3

Nếu bạn có thể cấu trúc lại t thành một char, bạn có thể sử dụng switch thay vì (Java 6):

void f(char t) { 

    switch(t) { 

    case 'a`: 
     someObject.setType(ObjectType.TYPE_A); 
     break; 
    case 'b': 
     someObject.setType(ObjectType.TYPE_B); 
     break; 

    // ... 

    } 

} 

Như Marko chỉ ra, bạn có thể đi với String cũng bằng Java 7.

Nó không ngắn hơn, nhưng thanh lịch hơn. Hơn nữa, tôi nghĩ rằng nó có thể nhanh hơn, vì switch hoạt động gần với O(1) với các bảng nhảy (Ai đó có thể xác nhận điều này có đúng không?), Cho dù một số câu hỏi if là O(n).

Triển khai phức tạp hơn nhiều so với chỉ một đơn setType bạn cũng có thể nghĩ đến việc triển khai State Pattern.

Answer 4

tôi sẽ thêm này như là một chức năng của enum:

public enum ObjectType { 
    TYPE_A("a"), 
    TYPE_B("b"); 

    private String stringType; 

    private ObjectType(String stringType) { 
     this.stringType = stringType; 
    } 

    public String getStringType() { 
     return this.stringType; 
    } 

    public static ObjectType fromStringType(String s) { 
     for (ObjectType type : ObjectType.values()) { 
      if (type.stringType.equals(s)) { 
       return type; 
      } 
     } 
     throw new IllegalArgumentException("No ObjectType with stringType " + s); 
    } 
} 

... 

void f(String t) { 
    someObject.setType(ObjectType.fromStringType(t)); 
} 

Answer 5

1.You có thể đi cho Chuyển tuyên bố nếu bạn có số nếu có điều kiện hơn 3.

2. bạn có thể chuyển đổi các câu lệnh if else của mình thành ternary operations

Answer 6

Các đề xuất khác là tuyệt vời - đặc biệt là thông minh và bản đồ thông minh hơn. Nhưng việc cấu trúc lại cơ bản nhất đầu tiên tôi sẽ giải quyết ở đây là trích xuất một phương thức để trả về enum một cách trực tiếp và có người gọi thực hiện không có gì nhiều hơn setType cho giá trị trả về của phương thức đó.

void f(String t) { 
    final ObjectType type = findType(t); 
    if (type != null) 
    someObject.setType(type); 
    } 

ObjectType findType(String t) { 
    if (t.equals("a")) return ObjectType.TYPE_A; 
    if (t.equals("b")) return ObjectType.TYPE_B; 
    // 50 more similar code 
    } 

Trong một số trường hợp, điều này sẽ đầy đủ và tự nó; ở những người khác, phương pháp findType() có thể dẫn bạn đến một giải pháp dựa trên bản đồ hoặc dựa trên enum đơn giản.