Một mẫu thiết kế để tránh nhiều ifs

Question

Tôi có một lớp học thực sự đáng lo ngại với hai phương pháp bắt đầu hoặc dừng một số dịch vụ khi các dịch vụ đó có sẵn. Một cái gì đó như sau (họ không phải là if-elses, chỉ cần nếu):

void startServices() { 
if (service1 == true) { 
    start1(); 
} 
if (service2 == true) { 
    start2(); 
} if (serviceN == true) { 
    startN(); 
} 
} 


void stopServices() { 
if (service1 == true) { 
    stop1(); 
} 
if (service2 == true) { 
    stop2(); 
} 
if (serviceN == true) { 
    stopN(); 
} 

} 

Bạn có đề xuất cho tôi bất kỳ mẫu thiết kế nào để làm cho nó đẹp hơn không?

Cảm ơn!

Answer 1

Phụ thuộc; phản ứng đầu tiên của tôi là lưu trữ các dịch vụ trong một băm hoặc mảng. Mỗi dịch vụ thực hiện một giao diện với các phương thức start và stop. Bắt đầu hoặc dừng dịch vụ sau đó chỉ yêu cầu một khóa hoặc chỉ mục dịch vụ.

Nó vẫn còn hơi giòn, có lẽ, nhưng không biết nhiều hơn Tôi không chắc chắn cách "domainify" ut để nó trông giống như những gì bạn đang làm.

Answer 2

Bạn có thể sử dụng mẫu Chiến lược.

Ý tưởng là bạn nên biết bạn sẽ sử dụng chiến lược nào khi bạn khởi tạo lớp học của mình (hoặc bạn có thể thay đổi thành công tự động). Vì vậy, bạn có thể vượt qua trong chiến lược đó khi khởi tạo (và tùy chọn thay thế nó sau này).

public interface IStartupStrategy 
{ 
    void Start(); 
} 

public interface IStopStrategy 
{ 
    void Stop(); 
} 

public class MyClass 
{ 
    private readonly IEnumerable<IStartupStrategy> startupStrategies; 
    private readonly IEnumerable<IStopStrategy> stopStrategies; 

    public MyClass(IEnumerable<IStartupStrategy> startup, IEnumerable<IStopStrategy> stop) 
    { 
     this.startupStrategies = startup; 
     this.stopStrategies = stop; 
    } 

    public void Start() 
    { 
     foreach(var strategy in this.startupStrategies) 
     { 
      strategy.Start(); 
     } 
    } 

    public void Stop() 
    { 
     foreach(var strategy in this.stopStrategies) 
     { 
      strategy.Stop(); 
     } 
    } 
} 

Answer 3

Sử dụng đối tượng, nơi bạn có danh sách Dịch vụ mà bạn có thể lặp lại bằng cách tắt phương pháp được thừa kế().

public interface Service { 
    void start(); 
    void stop(); 
} 

public class TestService implements Service { 
    @Override 
    void start() { 
    } 

    @Override 
    void stop() { 
    } 
} 

Mỗi dịch vụ cũng có thể lưu trữ trạng thái của nó để chỉ tắt chúng nếu chúng bật.

Answer 4

Báo cáo chuyển đổi ít gây nhầm lẫn. Nếu được sử dụng cùng với mã enums trở nên rất dễ đọc.