लूप के बजाए एक सेमफोर का उपयोग करना। यह अच्छा है या बुरा है?

Question

मेरे पास एक प्रक्रिया है जो अपने स्वयं के धागे में चलती है और इसे अवरुद्ध किए बिना शुरू/बंद कर दिया जा सकता है। यह अंत में एक विंडोज सेवा में जाएगा, लेकिन मैं इसे अब तक कंसोल ऐप में स्थापित कर रहा हूं जब तक कि यह पूरी तरह से बाहर नहीं हो जाता है।

प्रारंभ करने के लिए कॉल के बाद(), मैं मुख्य प्रोग्राम थ्रेड को ब्लॉक-सी दबाए जाने तक अवरुद्ध करना चाहता हूं। मुझे पता है कि यह काम करेगा:

public static void Main(string[] args) 
{ 
    bool keepGoing = true; 

    var service = new Service(); 

    System.Console.TreatControlCAsInput = false; 

    System.Console.CancelKeyPress += delegate(object sender, ConsoleCancelEventArgs e) 
    { 
     e.Cancel = true; 
     service.Stop(); 
     keepGoing = false; // Break the while loop below 
    }; 

    service.Start(); 

    while(keepGoing) 
    { 
     Thread.Sleep(100); // 100 is arbitrary 
    } 

} 

हालांकि, मुझे ध्वज और मनमानी नींद मूल्य परेशान लगता है। मुझे पता है कि सीपीयू लागत व्यावहारिक रूप से 0 लूप में है, लेकिन मेरे पास एक "हार्ड" ब्लॉक होगा जो जैसे ही Ctrl-C हैंडलर किया जाता है। मैं नीचे खोजा गया, जब तक गुमनाम Ctrl-C हैंडलर किया जाता है ब्लॉक करने के लिए एक सेमाफोर का उपयोग कर:

public static void Main(string[] args) 
{ 
    var service = new Service(); 

    var s = new Semaphore(1, 1); 

    System.Console.TreatControlCAsInput = false; 

    System.Console.CancelKeyPress += delegate(object sender, ConsoleCancelEventArgs e) 
    { 
     e.Cancel = true; 
     service.Stop(); 
     s.Release(); // This will allow the program to conclude below 
    }; 

    service.Start(); 

    s.WaitOne(); // This will not block 
    s.WaitOne(); // This will block w/o CPU usage until the sempahore is released 

} 

यह एक बुरा डिजाइन है? क्या यह अधिक है? यह खतरनाक है?

संपादित करें:

मैं भी AppDomain.CurrentDomain.UnhandledException ऊपर हुक इस प्रकार है:

AppDomain.CurrentDomain.UnhandledException += delegate { 
    service.Stop(); 
    s.Release(); 
}; 

संपादित करें 2:

मैं नोट करना चाहिए यह महत्वपूर्ण है कि यह है कि Stop() विधि बाहर निकलने पर बुलाया जाता है। @ एडम राल्फ के पास हाइब्रिड कंसोल/सेवा के लिए एक बिल्कुल अच्छा पैटर्न है, लेकिन Q.

Answer 1

का जवाब देने पर यह जानकारी नहीं थी हमारे पास हमारे कुछ ऐप्स में एक समान आवश्यकता है। वे विंडोज सेवाएं हैं, लेकिन डिबगिंग के लिए हम अक्सर उन्हें कंसोल ऐप्स के रूप में चलाने के लिए चाहते हैं। इसके अलावा, हम आम तौर पर विंडोज़ सेवाओं के रूप में नए ऐप्स को कोड करते हैं, लेकिन अक्सर बाद में उन्हें एक सेवा के रूप में चलाने की ज़रूरत नहीं होती है, एक बार जब हम अवधारणा साबित कर लेते हैं, आदि

यह पैटर्न है उपयोग: -

using (var service = new Service()) 
{ 
    if (Environment.UserInterActive) 
    { 
     service.Start(); 
     Thread.Sleep(Timeout.Infinite); 
    } 
    else 
    { 
     ServiceBase.Run(service); 
    } 
} 

असीम सोने के लिए धागा बोलने अक्षम लग सकता है, लेकिन यह केवल डीबगिंग परिदृश्यों के लिए है और अनावश्यक धागा कोई CPU समय बस कुछ स्मृति लागत, जो ज्यादातर से बना है (1MB के बारे में) धागे को आवंटित ढेर अंतरिक्ष। प्रक्रिया को अभी भी Ctrl + C से या कमांड विंडो बंद करके बाहर किया जा सकता है।

- संपादित करें -

आप पाते हैं कि service.Dispose() जब Ctrl + C दबाया जाता है कहा जाता है नहीं किया जा रहा है (यानी एक अशिष्ट बीच में बंद करें होता है) और Dispose() करने के लिए कॉल महत्वपूर्ण है, तो मुझे लगता है कि आप स्पष्ट रूप से कर सकता है इस तरह तो: -

using (var service = new Service()) 
{ 
    if (Environment.UserInterActive) 
    { 
     Console.CancelKeyPress += (sender, e) => service.Dispose(); 
     service.Start(); 
     Thread.Sleep(Timeout.Infinite); 
    } 
    else 
    { 
     ServiceBase.Run(service); 
    } 
} 

ध्यान दें कि Stop()Dispose() में समाहित किया जाना चाहिए।