2019 में C # परियोजनाओं में शीर्ष 10 कीड़े मिले

कीड़े के सभी प्रशंसकों के लिए हाय! नया साल जल्द ही आ रहा है, इसलिए निवर्तमान वर्ष का जायजा लेने का समय आ गया है। परंपरा से, हम पीवीएस-स्टूडियो टीम द्वारा 2019 में खुली सी # परियोजनाओं में पाई गई त्रुटियों की शीर्ष सूची पेश करने के लिए तैयार हैं। तैयार हैं? तो चलिए चलते हैं।

दसवां स्थान "मूर्ख हर कोई"

V3066 30 AdjustCellBorderStyle ’विधि: F isFirstDisplayedRow’ और DFirstDisplayedColumn ’में दिए गए तर्कों का संभावित गलत क्रम। DataGridViewComboBoxCell.cs 1934

protected override void OnMouseMove(DataGridViewCellMouseEventArgs e) { .... dgvabsEffective = AdjustCellBorderStyle( DataGridView.AdvancedCellBorderStyle, dgvabsPlaceholder, singleVerticalBorderAdded, singleHorizontalBorderAdded, isFirstDisplayedRow, // <= isFirstDisplayedColumn); // <= .... } 

लेख " WinForms: त्रुटियां, होम्स " से त्रुटि । विश्लेषक बताते हैं कि जब विधि के दो अंतिम तर्क मिश्रित होते हैं। चलो एडजस्टमेंट कंट्रोलर घोषणा को देखें:

 public virtual DataGridViewAdvancedBorderStyle AdjustCellBorderStyle( DataGridViewAdvancedBorderStyledataGridViewAdvancedBorderStyleInput, DataGridViewAdvancedBorderStyle dataGridViewAdvancedBorderStylePlaceholder, bool singleVerticalBorderAdded, bool singleHorizontalBorderAdded, bool isFirstDisplayedColumn, bool isFirstDisplayedRow) { .... } 

लगता है कि विश्लेषक सही है। अक्सर डेवलपर्स जानबूझकर उल्टे क्रम में कुछ तर्क पारित करते हैं, उदाहरण के लिए, चर के आसपास स्वैप करने के लिए। लेकिन ऐसा नहीं लगता कि ऐसा ही है। सबसे पहले, बूल प्रकार चर को मिलाया जाता है। दूसरे, कोई असामान्य तरीके के नाम नहीं हैं: कोई "स्वैप" या "रिवर्स" नहीं। इसके अलावा, इस तरह की त्रुटि करना मुश्किल नहीं है: लोग "लाइन / कॉलम" कोलाजेशन ऑर्डर को अलग तरह से समझते हैं।

नौवां स्थान "अनंत काल के करीब"

V3110 'TryValidateModel' पद्धति के अंदर संभावित अनंत पुनरावृत्ति। उपसर्गमूले उपपद.चक्षु 48

 public bool TryValidateModel(object model, string prefix) { return TryValidateModel(model, Prefix(prefix)); } 

लेख से त्रुटि " कीड़े के लिए ऑर्चर्ड सीएमएस के कोड को स्कैन करना "। एक गलती थी जो अनंत पुनरावृत्ति का कारण बनी। जिस तरह से त्रुटि की गई थी, उसे समझने के लिए, TryValidateModel विधि के ओवरलोडिंग पर विचार करना होगा :

 public bool TryValidateModel(object model) { return _updateModel.TryValidateModel(model); } 

यह संभव है कि पहला मामला भी इस तरह के कॉल का उपयोग करे:

 public bool TryValidateModel(object model, string prefix) { return _updateModel.TryValidateModel(model, Prefix(prefix)); } 

कोड सफलतापूर्वक संकलित किया गया, क्योंकि _updateModel IUpdateModel प्रकार का है और वर्तमान वर्ग भी IUpdateModel इंटरफ़ेस लागू करता है।

आठवां स्थान "यदि आप कर सकते हैं तो मुझे खोजें"

V3091 अनुभवजन्य विश्लेषण। यह संभव है कि एक टाइपो स्ट्रिंग शाब्दिक के अंदर मौजूद है: "प्रबंधन समूह आईडी।" 'ईद' शब्द संदिग्ध है। कॉन्स्टेंट.कैंस 36

 public class HelpMessages { public const string SubscriptionId = "Subscription Id of the subscription associated with the management"; public const string GroupId = "Management Group Id"; // <= public const string Recurse = "Recursively list the children of the management group"; public const string ParentId = "Parent Id of the management group"; public const string GroupName = "Management Group Id"; // <= public const string DisplayName = "Display Name of the management group"; public const string Expand = "Expand the output to list the children of the management group"; public const string Force = "Force the action and skip confirmations"; public const string InputObject = "Input Object from the Get call"; public const string ParentObject = "Parent Object"; } 

" Azure PowerShell: अधिकतर हानिकारक " लेख से त्रुटि। विश्लेषक ने समूहनाम को एक गलत स्ट्रिंग द्वारा आरंभ किए जाने के लिए निरंतर संदेह किया। "प्रबंधन समूह का नाम" जैसा कुछ होना चाहिए। इस त्रुटि की आलोचना अभी भी संदिग्ध है, लेकिन त्रुटि निश्चित रूप से दुर्लभ है और इसका पता लगाना मुश्किल है।

सातवीं जगह "बस नजरअंदाज"

V3078 'सॉर्टबाय' पद्धति के दोहराए जाने के बाद मूल छँटाई क्रम खो जाएगा। मूल छंटाई को संरक्षित करने के लिए 'तब' विधि का उपयोग करें। GridModel.Selection.cs 107

 internal partial class GridModel { private void BuildCellSelectionRegions(....) { .... this.MergeCellSelectionRegions(selectedItemsInView .OrderBy(c => c.Column.ItemInfo.LayoutInfo.Line) .OrderBy(c => c.RowItemInfo.LayoutInfo.Line)); } } 

लेख की त्रुटि " पीवीएस-स्टूडियो के साथ शुरुआत करने के तरीके के रूप में यूडब्ल्यूपी के लिए तेलरिक यूआई की जाँच ।" पहले से सॉर्ट किए गए संग्रह पर ऑर्डरबाय को बार-बार कॉल करने के कारण पिछली छंटनी का परिणाम खो जाएगा। एक को इस मामले में तब का उपयोग करना होगा:

 this.MergeCellSelectionRegions(selectedItemsInView .OrderBy(c => c.Column.ItemInfo.LayoutInfo.Line) .ThenBy(c => c.RowItemInfo.LayoutInfo.Line)); 

इस तरह की त्रुटियाँ असावधानी या अज्ञानता से होती हैं। मुझे लगता है कि कॉपी-पेस्ट यहां दोष देना है।

छठे स्थान पर "कोड का दस्तावेजीकरण किया गया है", उन्होंने कहा

V3009 यह विचित्र है कि यह विधि हमेशा एक और 'सही' के समान मान देती है। MaskedTextProvider.cs 1529

 public bool Remove(out int testPosition, out MaskedTextResultHint resultHint) { .... if (lastAssignedPos == INVALID_INDEX) { .... return true; // nothing to remove. } .... return true; } 

लेख से त्रुटि " पीवीएस-स्टूडियो स्टेटिक विश्लेषक द्वारा .NET कोर लाइब्रेरी सोर्स कोड की जाँच "। विधि हमेशा सही लौटेगी। हां, यह एक त्रुटि है, लेकिन एक और चीज है जो वास्तव में उत्सुक है। विधि विस्तृत टिप्पणी के बाद है:

स्वरूपित स्ट्रिंग से अंतिम वर्ण निकालता है। (वर्चुअल स्ट्रिंग में अंतिम वर्ण निकालें)। बाहर निकलने पर परम में वह स्थिति होती है जहां ऑपरेशन वास्तव में किया गया था। यह स्थिति परीक्षण स्ट्रिंग के सापेक्ष है। MaskedTextResultHint out param ऑपरेशन परिणाम के बारे में अधिक जानकारी देता है। सफलता पर सच लौटाता है , अन्यथा गलत ।

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

पांचवां स्थान "मुझे सूचकांक, अब!"

V3102 लूप के अंदर एक स्थिर सूचकांक द्वारा 'seq' ऑब्जेक्ट के तत्व तक संदिग्ध पहुंच। XmlQueryRuntime.cs 738

 public bool MatchesXmlType(IList<XPathItem> seq, ....) { .... for (int i = 0; i < seq.Count; i++) { if (!CreateXmlType(seq[0]).IsSubtypeOf(....)) return false; } return true; } 

लेख से त्रुटि " पीवीएस-स्टूडियो स्टेटिक विश्लेषक द्वारा .NET कोर लाइब्रेरी सोर्स कोड की जाँच "। जब लूप के लिए seq संग्रह को ट्रेस किया जाता है, तो डेवलपर गलती से सभी पुनरावृत्तियों ( i के बजाय इंडेक्स 0) पर अपने पहले तत्व तक पहुंच का उपयोग करता है।

चौथा स्थान "बस एक डॉलर कम"

V3138 स्ट्रिंग शाब्दिक में संभावित प्रक्षेपित अभिव्यक्ति होती है। निरीक्षण करने पर विचार करें: ई। SSPIHandleCache.cs 42

 internal static void CacheCredential(SafeFreeCredentials newHandle) { try { .... } catch (Exception e) { if (!ExceptionCheck.IsFatal(e)) { NetEventSource.Fail(null, "Attempted to throw: {e}"); } } } 

लेख से त्रुटि " पीवीएस-स्टूडियो स्टेटिक विश्लेषक द्वारा .NET कोर लाइब्रेरी सोर्स कोड की जाँच "। जाहिर है, स्ट्रिंग "फेंकने का प्रयास: {ई}" को प्रक्षेपित किया जाना चाहिए। $ वर्ण छूटने के कारण, ई अपवाद का लाइन प्रतिनिधित्व स्ट्रिंग में नहीं डाला जाएगा। नतीजतन, लाइन का उपयोग किया जाएगा "जैसा कि यह है।"

तीसरा स्थान "कोई रास्ता नहीं है"

V3008 [CWE-563] 'this.linker.s3.region' चर को क्रमिक रूप से दो बार मान दिया गया है। शायद यह एक गलती है। जाँच लाइनें: 116, 114. AWSSDK.DynamoDBv2.Net45 S3Link.cs 116

 public string Region { get { .... } set { if (String.IsNullOrEmpty(value)) { this.linker.s3.region = "us-east-1"; } this.linker.s3.region = value; } } 

लेख से त्रुटि " .NET के लिए अमेज़न वेब सेवा एसडीके स्रोत कोड में त्रुटियों की खोज "। इफ ब्लॉक के बॉडी में रिटर्न छूट गया था। परिणामस्वरूप, इस .linker.s3.region वैरिएबल को हमेशा मान मिलेगा , जिसमें एक खाली लाइन और नल शामिल है।

दूसरी जगह "सही पोशाक!"

'LOCALE_USER_DEFAULT' वैरिएबल को प्रारंभ करते समय V3070 Uninitialized चर 'LANG_USER_DEFAULT' का उपयोग किया जाता है। NativeMethods.cs 890

 internal static class NativeMethods { .... public static readonly int LOCALE_USER_DEFAULT = MAKELCID(LANG_USER_DEFAULT); public static readonly int LANG_USER_DEFAULT = MAKELANGID(LANG_NEUTRAL, SUBLANG_DEFAULT); .... } 

लेख " WinForms: त्रुटियां, होम्स " से त्रुटि । वर्ग क्षेत्रों का प्रारंभिक क्रम उलझन में है। LOCALE_USER_DEFAULT फ़ील्ड के मान की गणना करने के लिए, LANG_USER_DEFAULT फ़ील्ड का उपयोग किया जाता है, जो फ़िलहाल प्रारंभ नहीं किया गया है और 0. है। चर का उपयोग कोड में कहीं और नहीं किया जाता है। यह त्रुटि क्या होती है, यह जानने के लिए, WinForms कोड से विधियों वाला एक संपूर्ण परीक्षण कार्यक्रम लिखा गया था। उपयोग किए गए कुछ स्थिरांक के बजाय, उनके वास्तविक मूल्यों को सरलता के लिए प्रतिस्थापित किया गया था:

 internal static class NativeMethods { public static readonly int LOCALE_USER_DEFAULT = MAKELCID(LANG_USER_DEFAULT); public static readonly int LANG_USER_DEFAULT = MAKELANGID(0x00, 0x01); public static int MAKELANGID(int primary, int sub) { return ((((ushort)(sub)) << 10) | (ushort)(primary)); } public static int MAKELCID(int lgid) { return MAKELCID(lgid, 0x0); } public static int MAKELCID(int lgid, int sort) { return ((0xFFFF & lgid) | (((0x000f) & sort) << 16)); } } class Program { static void Main() { System.Console.WriteLine(NativeMethods.LOCALE_USER_DEFAULT); } } 

निष्पादन के परिणामस्वरूप हमारे पास निम्नलिखित होंगे: 0. अब LOCALE_USER_DEFAULT और LANG_USER_DEFAULT फ़ील्ड की घोषणा को स्वैप करके त्रुटि को ठीक करें। कार्यक्रम के निष्पादन का परिणाम: 1024।

पहले स्थान पर "पहले प्रयास करें, फिर विश्वास करें"

पहली जगह के साथ यह आसान नहीं है। यहां कुछ असाधारण और मनोरम होना चाहिए। प्रारंभ में, इस लेख के लिए मैंने बीस से अधिक दिलचस्प त्रुटियों का चयन किया, लेकिन उनमें से पहले स्थान के योग्य कुछ भी नहीं था। तभी मैंने अपने सहयोगी सर्गेई वासिलिव के लेख को याद किया। लेख केवल एक त्रुटि पर विस्तृत है। इस त्रुटि की सुंदरता यह है कि इसने हमारे विश्लेषक के काम को सीधे प्रभावित किया है। कैसे? आप इसे पहले ही लेख के शीर्षक से प्राप्त कर सकते हैं: " पीवीएस-स्टूडियो की लाइब्रेरी में उपयोग की जाने वाली त्रुटि ... पीवीएस-स्टूडियो " में किस तरह की त्रुटि मिली । यहां वह जगह है जहां मैंने त्रुटि का विवरण देने के लिए बहुत आलसी महसूस किया और इस तरह सुझाव दिया कि आप लिंक का पालन करें और विवरण का पता लगाएं। :) मैं गारंटी देता हूं कि यह इसके लायक है। इसके अलावा, लेख छोटा है।

निष्कर्ष

मुझे आशा है कि त्रुटियाँ आपके लिए बकाया थीं और लेख थकाने वाला नहीं था। सिर्फ रिकॉर्ड के लिए, आप अपने, सहकर्मियों और किसी भी टॉम, डिक या हैरी को खुश करने के लिए अपने और तीसरे पक्ष के प्रोजेक्ट में बग ढूंढने के लिए हमेशा पीवीएस-स्टूडियो विश्लेषक डाउनलोड कर सकते हैं। त्रुटियों को कम होने दें, और आत्म-सुधार के लिए समय - अधिक! :)

क्या आपने अंत तक पढ़ा? नए स्तर पर पहुंचने पर मेरी बधाई! हमारे ब्लॉग में हमारे upcomimg लेखों को याद न करें - 2019 में जावा और सी ++ परियोजनाओं में सर्वश्रेष्ठ बग।