एकाधिक ifs

Question

से बचने के लिए एक डिज़ाइन पैटर्न मेरे पास वास्तव में एक भयानक कक्षा है जिसमें दो विधियां हैं जो कि सेवाएं उपलब्ध होने पर कुछ सेवाओं को शुरू या बंद कर देती हैं। निम्नलिखित की तरह कुछ (वे कर रहे हैं नहीं तो-elses, बस अगर):

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

} 

तुम मुझे यह खूबसूरत बनाने के लिए किसी भी डिजाइन गपशप सुझाव देते हैं?

धन्यवाद!

Answer 1

निर्भर करता है; मेरी पहली प्रतिक्रिया हैश या सरणी में सेवाओं को स्टोर करना है। प्रत्येक सेवा प्रारंभ और बंद विधियों के साथ एक इंटरफ़ेस लागू करती है। सेवा शुरू करना या रोकना, फिर केवल एक सेवा कुंजी या अनुक्रमणिका की आवश्यकता होती है।

यह अभी भी थोड़ा भंगुर है, लेकिन बिना किसी ज्ञात के मुझे यकीन नहीं है कि "डोमेनइफ़" कैसे करें, इसलिए यह आपके जैसा काम करता है।

Answer 2

आप रणनीति पैटर्न का उपयोग कर सकते हैं।

विचार यह है कि आपको पता होना चाहिए कि जब आप अपनी कक्षा को तुरंत चालू करते हैं तो आप किस रणनीति का उपयोग करने जा रहे हैं (या आप इसे गतिशील रूप से latrr बदल सकते हैं)। इसलिए, आप तत्कालता पर उस रणनीति को पारित कर सकते हैं (और वैकल्पिक रूप से इसे बाद में बदल दें)।

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

उपयोग वस्तुओं, जहाँ आप सेवा जो आप उन्हें एक विरासत विधि विराम के साथ बंद करने के माध्यम से पुनरावृति कर सकते हैं की एक सूची है()।

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

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

    @Override 
    void stop() { 
    } 
} 

प्रत्येक सेवा अपने राज्य को भी स्टोर कर सकती है ताकि वे चालू होने पर उन्हें बंद कर सकें।

Answer 4

स्विच स्टेटमेंट कम भ्रमित हैं। यदि enums कोड के साथ एक साथ उपयोग किया जाता है तो बहुत पठनीय हो जाता है।