इनलाइन अगर कथन - शॉर्ट सर्किटिंग

Question

जैसा कि मैं समझता हूं और पढ़ता हूं, यदि आप दूसरी स्थिति के लिए आग लगने के लिए यदि कथन (& & या ||) में शॉर्ट सर्किटिंग का उपयोग कर सकते हैं। और यदि आप दोनों हालत को आग लगाना चाहते हैं तो आप एकल ऑपरेंड (& या |) का उपयोग करेंगे।

तो कहते हैं कि अगर मैं इनलाइन अगर नीचे के रूप में बयान किया है:

var test = (MyObject != null || string.IsNullOrEmpty(MyObject.Property)) ? string.Empty : MyObject.Property; 

यह object reference त्रुटि फेंक देंगे अगर MyObject रिक्त है, जो मेरी राय में नहीं के रूप में मैं कम सर्किटिंग उपयोग कर रहा हूँ चाहिए। क्या कोई इसे समझा सकता है।

Answer 1

आप गलत स्थिति का उपयोग कर रहे हैं। यह हिस्सा:

MyObject != null || string.IsNullOrEmpty(MyObject.Property) 

होना चाहिए:

MyObject == null || string.IsNullOrEmpty(MyObject.Property) 

एक || की आरएचएस केवल कार्यान्वित करता है, तो बाएं हाथ झूठी है। आप इसे केवल निष्पादित करना चाहते हैं यदि MyObject शून्य है।

संपादित करें: बहुत हालांकि

var test = MyObject != null && !string.IsNullOrEmpty(MyObject.Property) 
     ? MyObject.Property : ""; 

नोट 2 और सशर्त ऑपरेटर 3 ऑपरेंड के उत्क्रमण: तुम सच में MyObject != null हिस्सा चाहते हैं, आप के लिए पूरी बात बदल सकता है।

Answer 2

आप एक == नहीं एक होना चाहिए =

var test = (MyObject == null || string.IsNullOrEmpty(MyObject.Property) ? string.Empty : MyObject.Property 

Answer 3

इस प्रयास करें:

var test = (MyObject == null || string.IsNullOrEmpty(MyObject.Property) 
      ? string.Empty : MyObject.Property 

Answer 4

MyObject != null || string.IsNullOrEmpty(MyObject.Property) 

यहाँ आप का कहना है।

यदि मेरी वस्तु शून्य नहीं है। या string.IsNullOrEmpty (MyObject.Property)

कि अगर MyObject रिक्त है वह दूसरे भाग पर अमल करने की कोशिश करेंगे जिसका मतलब है।

MyObject == null || string.IsNullOrEmpty(MyObject.Property) 

इस अशक्त अपवाद

Answer 5

होता है ऐसा इसलिए है क्योंकि MyObjectnull है और इसलिए पहली शर्त false तो दूसरे भाग हालत के पूरे पता करने के लिए मूल्यांकन किया जाना चाहिए है फेंक नहीं होंगे। यह करने के लिए लाइन में परिवर्तित करें:

MyObject != null && string.IsNullOrEmpty(MyObject.Property) 

Answer 6

आप के बजाय पठनीयता लाइन गिनती, जैसे पसंद करते हैं चाहिए:

string prop = string.Empty; 
if(MyObject != null && MyObject.Property != null) 
    prop = MyObject.Property; 

(अपने अपवाद के लिए कारण पहले से ही अन्य उत्तर में विस्तार से बताया गया था)