Tái cấu trúc cây If else else

Question

Tôi có một cây khác sẽ phát triển khi tôi thêm các mục bổ sung để duy trì và tôi đang tìm cách tốt nhất để viết nó để bảo trì Tôi bắt đầu với mã này

private void ControlSelect() 
{ 

    if (PostingType == PostingTypes.Loads && !IsMultiPost) 
    { 
     singleLoadControl.Visible = true; 
      singleTruckControl.Visible = false; 
      multiTruckControl.Visible = false; 
      multiLoadControl.Visible = false; 
    } 
    else if (PostingType == PostingTypes.Trucks && !IsMultiPost) 
    { 
     singleLoadControl.Visible = false; 
      singleTruckControl.Visible = true; 
      multiTruckControl.Visible = false; 
      multiLoadControl.Visible = false; 
    } 
    else if (PostingType == PostingTypes.Loads && IsMultiPost) 
    { 
     singleLoadControl.Visible = false; 
      singleTruckControl.Visible = false; 
      multiTruckControl.Visible = false; 
      multiLoadControl.Visible = true; 
    } 
    else if (PostingType == PostingTypes.Trucks && IsMultiPost) 
    { 
     singleLoadControl.Visible = false; 
     singleTruckControl.Visible = false; 
      multiTruckControl.Visible = true; 
     multiLoadControl.Visible = false; 
    } 
} 

và tư duy tái Sacombank nó một cái gì đó như thế này

private void ControlSelect() 
{ 
    List<UserControl> controlList = GetControlList(); 

     string visableControl = singleLoadControl.ID; 
     if (PostingType == PostingTypes.Loads && !IsMultiPost) 
     { 
     visableControl = singleLoadControl.ID; 
     } 
     else if (PostingType == PostingTypes.Trucks && !IsMultiPost) 
     { 
     visableControl = singleTruckControl.ID; 
     } 
     else if (PostingType == PostingTypes.Loads && IsMultiPost) 
     { 
     visableControl = multiLoadControl.ID; 
     } 
     else if (PostingType == PostingTypes.Trucks && IsMultiPost) 
     { 
     visableControl = multiTruckControl.ID; 
     } 

     foreach (UserControl userControl in controlList) 
     { 
     userControl.Visible = (userControl.ID == visableControl); 
     } 
} 

private List<UserControl> GetControlList() 
{ 
    List<UserControl> controlList = new List<UserControl> 
     { 
     singleLoadControl, 
      multiTruckControl, 
      singleTruckControl, 
      multiLoadControl 
     }; 
     return controlList; 
} 

tôi đi biểu diễn hit nhưng tôi có thể quản lý tất cả các điều khiển của tôi là một nơi duy nhất

wa suy nghĩ khác của tôi s để làm cho mỗi khối chọn kiểm soát nó phương pháp riêng, một cái gì đó như thế này

private void SetSingleLoadControlAsSelected() 
{ 
     singleLoadControl.Visible = true; 
     singleTruckControl.Visible = false; 
     multiTruckControl.Visible = false; 
     multiLoadControl.Visible = false; 
} 

tôi không đi biểu diễn hit nhưng tôi duy trì các điều khiển trong nhiều vị trí

Tôi đang nghiêng về lựa chọn cho một chỉ vì tôi thích khía cạnh bảo trì của nó.

Answer 1

gì về

singleLoadControl.Visible = 
     PostingType == PostingTypes.Loads && !IsMultiPost;  
singleTruckControl.Visible = 
     PostingType == PostingTypes.Trucks && !IsMultiPost;  
multiTruckControl.Visible = 
     PostingType == PostingTypes.Loads && IsMultiPost;  
multiLoadControl.Visible = 
     PostingType == PostingTypes.Trucks && IsMultiPost; 

nếu bạn muốn khả năng để làm cho nhiều điều khiển có thể nhìn thấy (hoặc thêm giá trị liệt kê hơn) trang trí enum với [Flags] thuộc tính như sau:

[Flags] 
public enum PostTyp { None=0, IsMultiPost = 1, Loads = 2, Trucks = 4 } 

và sửa đổi luật như sau:

singleLoadControl.Visible = 
     ((PostingType & (PostTyp.Loads | ~PostTyp.MultiCast)) 
     == PostingType);  
singleTruckControl.Visible = 
     ((PostingType & (PostTyp.Trucks | ~PostTyp.MultiCast)) 
     == PostingType);   
multiTruckControl.Visible = 
     ((PostingType & (PostTyp.Loads | PostTyp.MultiCast)) 
     == PostingType);   
multiLoadControl.Visible = 
     ((PostingType & (PostTyp.Trucks | PostTyp.MultiCast)) 
     == PostingType);  

Answer 2

Điều gì về điều này:

 singleLoadControl.Visible = false; 
    singleTruckControl.Visible = false; 
    multiTruckControl.Visible = false; 
    multiLoadControl.Visible = false; 

    if (PostingType == PostingTypes.Loads && !IsMultiPost) 
    { 
      singleLoadControl.Visible = true; 
    } 
    else if (PostingType == PostingTypes.Trucks && !IsMultiPost) 
    { 
      singleTruckControl.Visible = true; 
    } 
    else if (PostingType == PostingTypes.Loads && IsMultiPost) 
    { 
     multiLoadControl.Visible = true; 
    } 
    else if (PostingType == PostingTypes.Trucks && IsMultiPost) 
    { 
      multiTruckControl.Visible = true; 
} 

Answer 3

Có vẻ như bạn đang sử dụng một điều tra, tôi khuyên bạn nên chuyển đổi với trường hợp mặc định để đối phó với các giá trị không xác định. Tôi tin rằng cách tiếp cận này làm cho những ý định rõ ràng hơn là làm tất cả việc kiểm tra trong bài tập.

switch (PostingType) 
{ 
case PostingTypes.Loads: 
    singleLoadControl.Visible = !IsMultiPost; 
    multiTruckControl.Visible = IsMultiPost; 
    singleTruckControl.Visible = false; 
    multiTruckLoadControl.Visible = false; 
    break; 

case PostingTypes.Trucks: 
    singleLoadControl.Visible = false; 
    multiTruckControl.Visible = false; 
    singleTruckControl.Visible = !IsMultiPost; 
    multiLoadControl.Visible = IsMultiPost; 
    break; 

default: 
    throw InvalidOperationException("Unknown enumeration value."); 
} 

Answer 4

Nếu bạn đang mong đợi để có thêm rất nhiều thông số khác nhau, bạn có thể tạo một mảng đa chiều của các đối tượng

Arr[0][0] = singleLoadControl 
Arr[0][1] = singleTruckControl 
Arr[1][0] = multiLoadControl 
Arr[1][1] = multiTruckControl 

này là khá đáng sợ, nhưng làm cho đơn giản hơn nếu phát biểu. Nếu bạn sẽ có vô số tham chiếu đến các điều khiển dù sao đi nữa, tôi muốn sử dụng một biểu diễn dựa trên mã của những gì các tải đó là gì. một mảng như vậy có thể được bọc trong một lớp học để cho phép bạn truy cập vào các yếu tố sử dụng một cái gì đó giống như:

ControlClassInstance.single.truck 

Bạn muốn có mã như thế này:

p1 = IsMultiPost ? ControlClassInstance.multi : ControlClassInstance.single 
p2 = p1[PostingType] //(this call would mean adding an indexer) 

Loại giải pháp này là cách quá phức tạp trừ khi bạn mong đợi mọi thứ trở nên phức tạp ... và cũng có thể nghèo nàn.

Answer 5

singleLoadControl.Visible = false; 
    singleTruckControl.Visible = false; 
    multiTruckControl.Visible = false; 
    multiLoadControl.Visible = false; 


    singleLoadControl.Visible = (PostingType == PostingTypes.Loads && !IsMultiPost); 
    singleTruckControl.Visible = (PostingType == PostingTypes.Trucks && !IsMultiPost); 
    multiLoadControl.Visible = (PostingType == PostingTypes.Loads && IsMultiPost); 
    multiTruckControl.Visible = (PostingType == PostingTypes.Trucks && IsMultiPost); 

Answer 6

Nếu nó sẽ khác hợp lý (ví dụ, nếu những đã là điều khiển tùy chỉnh tên miền cụ thể), bạn có thể đóng gói logic bên trong điều khiển tự (Replace Conditional with Polymorphism).Có lẽ tạo ra một giao diện như thế này:

public interface IPostingControl { 
    void SetVisibility(PostingType postingType, bool isMultiPost); 
} 

Sau đó, mỗi kiểm soát sẽ chịu trách nhiệm cho các quy tắc tầm nhìn riêng của mình:

public class SingleLoadControl: UserControl, IPostingControl { 

    // ... rest of the implementation 
    public void SetVisibility(PostingType postingType, bool isMultiPost) { 
     this.Visible = postingType == PostingType.Load && !isMultiPost); 
    } 
} 

Cuối cùng, trong trang của bạn, chỉ cần lặp qua IPostingControls của bạn và gọi SetVisibility(postingType, isMultiPost).

Answer 7

private void ControlSelect() 
{ 
    if (PostingType == PostingTypes.Loads && !IsMultiPost) 
    { 
     singleLoadControl.Visible = true; 
     singleTruckControl.Visible = false; 
     multiTruckControl.Visible = false; 
     multiLoadControl.Visible = false; 
     return; 
    } 
    if (PostingType == PostingTypes.Trucks && !IsMultiPost) 
    { 
     singleLoadControl.Visible = false; 
     singleTruckControl.Visible = true; 
     multiTruckControl.Visible = false; 
     multiLoadControl.Visible = false; 
     return; 
    } 
    if (PostingType == PostingTypes.Loads && IsMultiPost) 
    { 
     singleLoadControl.Visible = false; 
     singleTruckControl.Visible = false; 
     multiTruckControl.Visible = false; 
     multiLoadControl.Visible = true; 
     return; 
    } 
    if (PostingType == PostingTypes.Trucks && IsMultiPost) 
    { 
     singleLoadControl.Visible = false; 
     singleTruckControl.Visible = false; 
     multiTruckControl.Visible = true; 
     multiLoadControl.Visible = false; 
     return; 
    } 
} 

Answer 8

Bạn đã xem số State Pattern chưa?