2019 के लिए सी # परियोजनाओं में शीर्ष 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: त्रुटियां, होम्स " लेख से त्रुटि। विश्लेषक इंगित करता है कि जब विधि के लिए तर्क पारित करते हैं, तो अंतिम दो तर्क उलट दिए गए थे। AdjustCellBorderStyle घोषणा पर एक नज़र डालें:

 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:" मूल रूप से हानिरहित है " लेख से थी। विश्लेषक को संदेह था कि GroupName स्थिरांक को एक त्रुटि स्ट्रिंग के साथ प्रारंभ किया गया था। संभवतः " प्रबंधन समूह का नाम " जैसा कुछ होना चाहिए। इस त्रुटि की महत्वपूर्णता का न्याय करना मुश्किल है, लेकिन यह विशिष्ट रूप से दुर्लभ और दुर्लभ है। मुश्किल का पता लगाने के लिए।

सातवीं जगह: "बस अनदेखी"

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)); } } 

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

 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 कोर पुस्तकालयों के स्रोत कोड को सत्यापित करना "। स्ट्रिंग "फेंकने का प्रयास: {e}" प्रक्षेपित प्रतीत होता है । $ चरित्र गायब होने के कारण, ई अपवाद के स्ट्रिंग प्रतिनिधित्व को स्ट्रिंग में प्रतिस्थापित नहीं किया जाएगा। नतीजतन, स्ट्रिंग "जैसा है" का उपयोग किया जाएगा।

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

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. है। LANG_USER_DEFAULT फ़ील्ड का उपयोग कोड में कहीं और नहीं किया गया है। यह समझने के लिए कि यह त्रुटि क्या है, 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।

पहला स्थान: "भरोसा करें, लेकिन सत्यापित करें"

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

निष्कर्ष

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

क्या आपने पढ़ा है? अगले स्तर पर जाने के लिए बधाई! हमारे ब्लॉग के निम्नलिखित लेखों में, जावा और सी ++ प्रोजेक्ट्स में 2019 के सर्वश्रेष्ठ बगों पर प्रकाश डाला गया है ।

यदि आप इस लेख को अंग्रेजी बोलने वाले दर्शकों के साथ साझा करना चाहते हैं, तो कृपया अनुवाद के लिंक का उपयोग करें: सर्गेई ख्रेनोव। 2019 में C # परियोजनाओं में शीर्ष 10 कीड़े मिले ।