Làm thế nào để tôi loại bỏ câu lệnh Switch?

Question

Tôi có một điều khiển tab có 3 trang tab. Bên dưới tab này kiểm soát trên cùng một hình thức/xem, tôi có 3 điều khiển hình ảnh.

Dựa trên SelectedIndex của tab, tôi cần thay đổi độ mờ của dưới 3 hình ảnh.

Tính đến bây giờ tôi có một cái gì đó như thế này trong SelectionChanged sự kiện của Tab kiểm soát:

switch (Tab.SelectedIndex) 
{ 
    case 0: 
     img1.Opacity= 1; 
     img2.Opacity = 0.5; 
     img3.Opacity = 0.5; 
     break; 
    case 1: 
     img1.Opacity = 0.5; 
     img2.Opacity = 1; 
     img3.Opacity = 0.5; 
     break; 
    case 2: 
     img1.Opacity = 0.5; 
     img2.Opacity = 0.5; 
     img3.Opacity = 1; 
     break; 
} 

Làm thế nào để tôi loại bỏ lệnh switch này? Tôi sẽ sử dụng mẫu thiết kế nào ở đây?

Answer 1

Bạn có thể sử dụng mẫu thiết kế của Nhà nước được giải thích cho instace here. Bạn sẽ xác định vài trạng thái và sau đó theo điều kiện quyết định cái nào trong số chúng nên được sử dụng trong thời điểm này.

Ví dụ:

abstract class State 
{ 
    abstract vod Apply(Form context); 
} 

class StateOne : State 
{ 
    override void Apply(Form context) 
    { 
     img1.Opacity= 1; 
     img2.Opacity = 0.5; 
     img3.Opacity = 0.5; 
    } 
} 

Bạn cũng có thể kết hợp nó với các mô hình phương pháp thiết kế Nhà máy đó sẽ quyết định nhà nước để sử dụng.

static class StateFactory 
{ 
    static State GetState(condition) 
    { 
     if(condition == something) 
      return new StateOne(); 
     else ... 
    } 
} 

Điều này sẽ không xóa báo cáo chuyển đổi khỏi mã của bạn, nhưng ít nhất nó sẽ ở nơi hợp lý để thực hiện điều hợp lý.

Cách sử dụng:

StateFactory.GetState(condition).Apply(this); 

Answer 2

Tôi nghĩ rằng bạn có thể xử lý này trong XAML của bạn thân bằng cách sử dụng Triggers.

Ngoài ra nếu bạn muốn có đơn vị này kiểm chứng, bạn nên sử dụng MVVM pattern nơi bạn sẽ xác định properties for SelectedIndex, Opacities trong ViewModel của bạn và ràng buộc họ XAML

Answer 3

Nếu bạn có một loạt các dữ liệu tùy ý mà bạn cần phải vượt qua trong. .. Sau đó, có thực sự không phải là một cách để có được xung quanh báo cáo chuyển đổi dễ dàng (đó là giá trị nỗ lực, ít nhất). Tôi sẽ đề nghị, ít nhất là về khả năng đọc mã, để sử dụng một enum. Bạn có thể cấu trúc lại mã của mình như sau:

switch ((ImageTypes)Tab.SelectedIndex) 
    { 
     case ImageTypes.TypeOne: 
      img1.Opacity= 1; 
      img2.Opacity = 0.5; 
      img3.Opacity = 0.5; 
      break; 
     case ImageTypes.TypeTwo: 
      img1.Opacity = 0.5; 
      img2.Opacity = 1; 
      img3.Opacity = 0.5; 
      break; 
     case ImageTypes.TypeThree: 
      img1.Opacity = 0.5; 
      img2.Opacity = 0.5; 
      img3.Opacity = 1; 
      break; 
    } 
    public enum ImageTypes 
    { 
     TypeOne, 
     TypeTwo, 
     TypeThree 
    } 

Answer 4

Trích xuất và tiêm. Giải nén xem thay đổi logic (switch) đến lớp/phương pháp bên ngoài và tiêm nó vào xem của bạn:

public void HighlightImages(int selection, params Image[] images) 
{ 
    switch (selection) 
    { 
     case 0: 
      images[0].Opacity= 1; 
      images[1].Opacity = 0.5; 
      images[2].Opacity = 0.5; 
      break; 
     case 1: 
      images[0].Opacity = 0.5; 
      images[1].Opacity = 1; 
      images[2].Opacity = 0.5; 
      break; 
     case 2: 
      images[0].Opacity = 0.5; 
      images[1].Opacity = 0.5; 
      images[2].Opacity = 1; 
      break; 
    } 
} 

Trong lựa chọn thay đổi handler bạn chỉ cần ủy xử lý để phụ thuộc tiêm:

private void SelectedIndexChanged(object sender, EventArgs e) 
{ 
    this.highlighter.HighlightImages(Tab.SelectedIndex, img1, img2, img3); 
} 

Bằng cách này bạn sẽ có có thể kiểm tra logic thay đổi độ mờ một cách dễ dàng mà không cần phải điều khiển chế độ xem toàn diện.

Answer 5

Điều này sẽ không loại bỏ hoàn toàn, vì không có nhiều việc phải làm ngoài việc tạo ra một di sản hạng lớn, nhưng điều đó sẽ hơi quá mức cho tình huống này.

Thay vào đó, bạn có thể giảm kích thước của nó như thế này:

switch (Tab.SelectedIndex) 
{ 
    img1.Opacity = 0.5; 
    img2.Opacity = 0.5; 
    img3.Opacity = 0.5; 

    case 0: 
     img1.Opacity += 0.5; 
     break; 
    case 1: 
     img2.Opacity += 0.5; 
     break; 
    case 2: 
     img3.Opacity += 0.5; 
     break; 
} 

Và bạn có thể giảm mã dự phòng với các phương pháp, vì vậy bạn không cần phải thay đổi 0,5 ở khắp mọi nơi mỗi khi bạn muốn thay đổi độ mờ đục. (Đặt 0.5 trong một hằng số sẽ được tốt đẹp quá):

switch (Tab.SelectedIndex) 
{ 
    SetInitialOpacity(img1); 
    SetInitialOpacity(img2); 
    SetInitialOpacity(img3); 

    case 0: 
     IncreaseOpacity(img1); 
     break; 
    case 1: 
     IncreaseOpacity(img2); 
     break; 
    case 2: 
     IncreaseOpacity(img3); 
     break; 
} 

private void SetInitialOpacity(Image image) 
{ 
    image.Opacity = 0.5; 
} 


private void IncreaseOpacity(Image image) 
{ 
    image.Opacity += 0.5; 
} 

Answer 6

Đề cập đến những gì "jimmy-tâm" cho biết, đây là những gì tôi sẽ đi cho:

public static void HighlightImages(int selection, params Image[] images) 
    { 
     for (int img = 0; img < images.Length; img++) 
     { 
      images[img].Opacity = (img == selection ? 1 : 0.5); 
     } 
    }