मुझे स्टाइलकॉप नियम SA1503 (CurlyBracketsMustNotBeOmitted) के साथ कठिन समय हो रहा है।स्टाइलकॉप नियम प्राप्त करना SA1503 CurlyBracketsMustNotBe अधिक लचीला होने के लिए सबमिट किया गया
public void SomeFunction(string someArg)
{
if (string.IsNullOrEmpty(someArg)) throw new ArgumentNullException("someArg");
// rest of the function here
}
इस के पीछे तर्क है जब एक ही तर्क और/या तर्क पर चेक पर एकाधिक सत्यापन जांच कर खड़ी स्थान बचाने के लिए है:
मेरी कोड में मैं अक्सर एक पैटर्न इस प्रकार है। इस तरह के चेक में तर्क आमतौर पर सरल और संक्षेप में होता है और इसी तरह अपवाद के लिए भी फेंक दिया जाता है।
हालांकि, मैं करूंगा कभी नहीं लिखने
if (someConditional)
DoSomeStuff();
मैं हमेशा
if (someConditional)
{
DoSomeStuff();
}
तो लिखते थे सारांश में:
- उपयोग कर्ली कोष्ठक अगर अगर बयान में विभाजित किया गया है एकाधिक लाइनें
- साधारण तर्क सत्यापन आदि के लिए घुंघराले ब्रैकेट का उपयोग न करें जो आसानी से (और पठनीय रूप से) एक लाइन
स्टाइलकॉप यहां मेरी सहायता कर सकता है?
@ रिचर्ड ई: अंततः आप क्या कर रहे थे? मैं नियम को भंग नहीं करना चाहता, लेकिन मैं आपके पूर्ववर्ती पैटर्न लिखना चाहता हूं जैसे आप वर्णन करते हैं। –
@Lieven: उस समय के लिए हमने इस नियम को छोड़ दिया है और हमारे एक लाइन सशर्त पर घुंघराले ब्रैकेट का उपयोग करने का फैसला किया है। इस अंत तक हमने स्टाइलकॉप नियम SA1501 को बंद कर दिया। –
ऑफटॉपिक: अगर परिवर्तनीय खाली स्ट्रिंग है तो ArgumentNullException को फेंक न दें, आपको इसे केवल नल पैरामीटर के लिए उपयोग करना चाहिए –