गुणों में लॉक, अच्छा दृष्टिकोण?

Question

मेरे मल्टीथ्रेडिंग एप्लिकेशन में मैं कुछ चर का उपयोग कर रहा हूं जिसे एक ही समय में कई उदाहरणों में बदला जा सकता है। यह अजीब है लेकिन उसने बिना किसी समस्या के ठीक काम किया है .. लेकिन निश्चित रूप से मुझे इसे थ्रेड-सुरक्षित बनाना होगा। मैं सिर्फ ताले से शुरुआत कर रहा हूं इसलिए मैं आपकी सलाह को अपमानित करूंगा:

जब क्लाइंट कनेक्ट होता है, क्लास क्लाइंट बनाया जाता है जहां प्रत्येक क्लाइंट का अपना "ए" चर होता है। अब उस के साथ जब तक

Client selectedClient SelectOtherClientClassByID(sentID); 

selectedClient.A=5; 

कोई समस्या नहीं है, तब भी जब 5 कक्षाएं एक ही समय (ThreadPool) कर रहे थे, लेकिन मैं एक गुणों के ताले जोड़ने के बारे में क्या सोच रहा था:

कभी कभी, ग्राहक ऐसे ही प्रणाली को बुलाती है ?

तरह:

A { 
    get { return mA; } 
    set { 
     // use lock here for settting A to some value 
    }  
} 

यह ठीक होगा?

Answer 1

आपको दोनों को लॉक और सेट में ताले का उपयोग करने की आवश्यकता है। यह ताला एक ही वस्तु होना चाहिए। उदाहरण के लिए:

private object mylock = new object(); 

public int A { 

    get { 
    int result; 
    lock(mylock) { 
    result = mA; 
    } 
    return result; 
    } 

    set { 
    lock(mylock) { 
     mA = value; 
    } 
    } 
} 

Answer 2

यह बहुत दुर्लभ है जब आपको केवल एक ही संपत्ति सेट करना है। अधिकतर selectedClient.A = 5 एक बहुत बड़े लॉजिकल ऑपरेशन का हिस्सा होगा, जिसमें कई असाइनमेंट/मूल्यांकन/आदि शामिल हैं। उस पूरे ऑपरेशन के दौरान आप लगातार राज्य में रहने के लिए selectedClient पसंद करेंगे और डेडलॉक्स/रेस स्थितियों को पेश नहीं करेंगे। इसलिए, यह काफी बेहतर हो अपने Client कक्षा में SyncRoot संपत्ति का खुलासा और पर ताला कि बुला कोड से करने के लिए होगा:

Client selectedClient = GetClient(...); 

lock(selectedClient.SyncRoot) 
{ 
    selectedClient.A = 5; 
    selectedClient.B = selectedClient.A * 54; 
} 

Answer 3

accessors के अंदर गुणों में पहुंच लॉकिंग फर्जी परिणाम हो सकता है। उदाहरण के लिए, निम्न कोड को देखो:

class C { 
    private object mylock = new object(); 

    public int A { 

     get { 
     int result; 
     lock(mylock) { 
     result = mA; 
     } 
     return result; 
     } 

     set { 
     lock(mylock) { 
      mA = value; 
     } 
     } 
    } 
} 
C obj = new C; 
C.A++; 

(हाँ, मैं यह पहली बार जवाब से कॉपी किया गया) एक रेस स्थिति यहाँ नहीं है! ऑपरेशन "सीए ++" को वास्तव में ए को दो अलग-अलग एक्सेस की आवश्यकता होती है, एक मूल्य प्राप्त करने के लिए और दूसरा अद्यतन मूल्य निर्धारित करने के लिए। कुछ भी सुनिश्चित नहीं करता है कि इन दो अभिगमों को उनके बीच संदर्भ स्विच के बिना एक साथ किया जाएगा। दौड़ की स्थिति के लिए शास्त्रीय परिदृश्य!

तो, सावधान रहें! ताले को एक्सेसर्स के अंदर रखना अच्छा नहीं है, ताले स्पष्ट रूप से प्राप्त किए जाने चाहिए, जैसा पिछला उत्तर सुझाता है (हालांकि इसे सिंक रूट्स के साथ नहीं होना चाहिए, कोई ऑब्जेक्ट करेगा)