जंगली पश्चिम में सबसे तेज रिपोर्ट। और इसके अलावा मुट्ठी भर कीड़े ...

न केवल माइक्रोसॉफ्ट ने हाल ही में अपनी परियोजनाओं का ओपन सोर्स कोड पोस्ट किया है - अन्य कंपनियां भी इस प्रवृत्ति का अनुसरण कर रही हैं। हमारे लिए, पीवीएस-स्टूडियो डेवलपर्स, यह फिर से विश्लेषक का परीक्षण करने का एक शानदार तरीका है, देखें कि यह कौन सी दिलचस्प चीजें पा सकता है और परियोजना के लेखकों को इसके बारे में सूचित कर सकता है। आज हम फास्ट रिपोर्ट्स प्रोजेक्ट के अंदर देखते हैं।

क्या जाँच हुई?

FastReport एक रिपोर्ट जनरेटर है जिसे फास्ट रिपोर्ट्स द्वारा विकसित किया गया है। C # में लिखा गया है और .NET मानक 2.0+ के साथ संगत है। परियोजना का स्रोत कोड हाल ही में GitHub पर पोस्ट किया गया था, जहां इसे आगे के विश्लेषण के लिए डाउनलोड किया गया था।

रिपोर्टें पाठ, छवियों, रेखाओं, आकृतियों, रेखाचित्र, तालिकाओं, बारकोड आदि के उपयोग का समर्थन करती हैं। वे डेटा सहित, एक कवर और एक बैक पेज सहित एकल-पृष्ठ और बहु-पृष्ठ हो सकते हैं। डेटा स्रोत XML, CSV, Json, MS SQL, MySql, Oracle, Postgres, MongoDB, Couchbase, RavenDB, SQLite हो सकते हैं।

रिपोर्ट टेम्पलेट बनाने के कई तरीके हैं: कोड से; एक xml फ़ाइल के रूप में; एक ऑनलाइन डिजाइनर या FastReport डिजाइनर समुदाय संस्करण का उपयोग कर।

यदि आवश्यक हो, पुस्तकालयों को NuGet संकुल के रूप में डाउनलोड किया जा सकता है ।

आप प्रोजेक्ट के GitHub पृष्ठ पर उत्पाद सुविधाओं के बारे में अधिक पढ़ सकते हैं।

परियोजना की विधानसभा के साथ कोई समस्या नहीं थी - मैंने इसे विज़ुअल स्टूडियो 2017 से इकट्ठा किया, जहां से बाद में पीवीएस-स्टूडियो प्लगइन द्वारा इसकी जांच की गई।

पीवीएस-स्टूडियो एक स्थिर विश्लेषक है जो सी, सी ++, सी #, जावा कोड में त्रुटियों की तलाश करता है। C # कोड का विश्लेषण करते समय, आप PVS-Studio प्लग-इन का उपयोग करके Visual Studio IDE विश्लेषक का उपयोग कर सकते हैं, या आप कमांड लाइन से परियोजनाओं की जांच कर सकते हैं, जिसके लिए PVS-Studio_Cmd.exe कमांड-लाइन उपयोगिता का उपयोग किया जाता है। यदि वांछित है, तो आप बिल्ड सर्वर पर विश्लेषण को कॉन्फ़िगर कर सकते हैं या विश्लेषण के परिणामों को सोनारक्यूब तक पहुंचा सकते हैं ।

खैर, आइए देखें कि इस बार क्या दिलचस्प था।

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

चश्मा और अधिक

निम्नलिखित विधि से मुलाकात करें:

public override string ToString() { if (_value == null) return null; return this.String; } 

PVS-Studio चेतावनी : V3108 यह 'ToSting ()' विधि से 'null' वापस करने के लिए अनुशंसित नहीं है। वेरिएंट .cs 1519

हां, एक ओवरराइड ToString () विधि से अशक्त लौटना अपने आप में कोई त्रुटि नहीं है, लेकिन फिर भी यह खराब शैली है। यह Microsoft से प्रलेखन में अन्य बातों के अलावा संकेत दिया गया है: आपका ToString () ओवरराइड खाली या अशक्त स्ट्रिंग नहीं लौटना चाहिए । डेवलपर्स जो एक स्थिर वापसी की उम्मीद नहीं करते हैं, जैसे कि ToString () का रिटर्न मान अप्रिय हो सकता है जब एक ArgumentNullException को नीचे दिए गए कोड के निष्पादन के दौरान फेंक दिया जाता है (बशर्ते कि एक्सटेंशन विधि IEnumeros <T> के लिए कहा जाता है)।

 Variant varObj = new Variant(); varObj.ToString().Contains(character); 

आप इस तथ्य से गलती पा सकते हैं कि उदाहरण सिंथेटिक है, लेकिन इसका सार नहीं बदलता है।

इसके अलावा, इस कोड की टिप्पणी इस प्रकार है:

 /// <summary> /// Returns <see cref="String"/> property unless the value on the right /// is null. If the value on the right is null, returns "". /// </summary> /// <returns></returns> 

उफ़। इसके बजाय "" शून्य हो जाता है ।

चलिए जारी रखते हैं।

लाइब्रेरी में फास्टस्ट्रिंग क्लास है। विवरण: StringBuilder का तेजी से विकल्प । दरअसल, इस वर्ग में StringBuilder का एक क्षेत्र होता है। FastString क्लास के कंस्ट्रक्टर्स Init मेथड को कॉल करते हैं, जो संबंधित फील्ड को इनिशियलाइज़ करता है।

कंस्ट्रक्टरों में से एक का कोड:

 public FastString() { Init(initCapacity); } 

और इनिट विधि का कोड:

 private void Init(int iniCapacity) { sb = new StringBuilder(iniCapacity); //chars = new char[iniCapacity]; //capacity = iniCapacity; } 

यदि वांछित है, तो आप स्ट्रिंग फ़ील्ड को StringBuilder संपत्ति के माध्यम से एक्सेस कर सकते हैं:

 public StringBuilder StringBuilder { get { return sb; } } 

कुल मिलाकर, FastString में 3 निर्माता हैं:

 public FastString(); public FastString(int iniCapacity); public FastString(string initValue); 

मैंने पहले डिजाइनर के शरीर को पहले ही दिखाया है कि वे शेष दो अनुमान लगाते हैं, मुझे लगता है, यह भी मुश्किल नहीं है। और अब, ध्यान। लगता है कि निम्नलिखित कोड क्या उत्पादन करेगा:

 FastString fs = new FastString(256); Console.WriteLine(fs.StringBuilder.Capacity); 

ध्यान दें, उत्तर:

अप्रत्याशित रूप से? आइए संगत निर्माता के शरीर को देखें:

 public FastString(int iniCapacity) { Init(initCapacity); } 

हमारे लेख के नियमित पाठकों को पहले से ही यहाँ एक समस्या खोजने के लिए एक आँख होनी चाहिए। नेत्र विश्लेषक (गंध, तर्क, इसे कॉल करें जो आप चाहते हैं) निश्चित रूप से सुन्न है, और उन्होंने एक समस्या पाई: V3117 कंस्ट्रक्टर पैरामीटर 'iniCapacity' का उपयोग नहीं किया जाता है। FastString.cs 434

क्या संयोग है कि वर्ग कोड में एक निरंतर फ़ील्ड initCapacity होता है , जिसे निर्माण पैरामीटर iniCapacity के बजाय Init विधि के तर्क के रूप में पारित किया जाता है ...

 private const int initCapacity = 32; 

समान नामों का उपयोग करते समय, आपको बहुत, बहुत सावधान रहने की आवश्यकता है। जहाँ कहीं भी समान नाम - C, C ++, C #, Java में प्रोजेक्ट के उपयोग से संबंधित गलतियाँ की गई थीं - वहाँ हर जगह इस तरह के टाइपो थे ...

वैसे, टाइपो के बारे में।

आइए निम्नलिखित सरल उदाहरण बनाएं और देखें कि यह कैसे काम करता है:

 static void Main(string[] args) { TextObject textObj = new TextObject(); textObj.ParagraphFormat = null; Console.WriteLine("Ok"); } 

जैसा कि आप अनुमान लगा सकते हैं, उत्पादन स्ट्रिंग "ओके" से अलग होगा :)

कौन सा? उदाहरण के लिए:

समस्या ParagraphFormat गुण और समान नामों का उपयोग करने में निहित है:

 public ParagraphFormat ParagraphFormat { get { return paragraphFormat; } set { ParagraphFormat = value; } } 

PVS-Studio चेतावनी : V3110 'ParagraphFormat' संपत्ति के भीतर संभावित पुनरावर्तन। TextObject.cs 281

ParagraphFormat गुण पैराग्राफफॉर्म फ़ील्ड पर एक आवरण है। इसके अलावा, इसके प्रापर्टी एक्सेसर को सही तरीके से लिखा जाता है, लेकिन सेट प्रॉपर्टी एक्सेसर में एक कष्टप्रद टाइपो होता है: फ़ील्ड के बजाय, रिकॉर्ड उसी प्रॉपर्टी में होता है, जो पुनरावृत्ति का कारण बनता है। फिर, समान नामों से संबंधित एक त्रुटि।

निम्नलिखित कोड स्निपेट पर विचार करें।

 public override Run Split(float availableWidth, out Run secondPart) { .... if (r.Width > availableWidth) { List<CharWithIndex> list = new List<CharWithIndex>(); for (int i = point; i < size; i++) list.Add(chars[i]); secondPart = new RunText(renderer, word, style, list, left + r.Width, charIndex); list.Clear(); for (int i = 0; i < point; i++) list.Add(chars[i]); r = new RunText(renderer, word, style, list, left, charIndex); return r; } else { List<CharWithIndex> list = new List<CharWithIndex>(); for (int i = point; i < size; i++) list.Add(chars[i]); secondPart = new RunText(renderer, word, style, list, left + r.Width, charIndex); list.Clear(); for (int i = 0; i < point; i++) list.Add(chars[i]); r = new RunText(renderer, word, style, list, left, charIndex); return r; } .... } 

PVS-Studio चेतावनी : V3004 'तत्कालीन' कथन 'और' कथन के बराबर है। HtmlTextRenderer.cs 2092

थोड़ा कॉपी-पेस्ट, और अब, अभिव्यक्ति के मूल्य की परवाह किए बिना r.Width> availableWidth , एक ही कार्रवाई की जाएगी। या तो स्टेटमेंट को हटा दें या किसी एक ब्रांच में लॉजिक बदल दें।

 public static string GetExpression(FindTextArgs args, bool skipStrings) { while (args.StartIndex < args.Text.Length) { if (!FindMatchingBrackets(args, skipStrings)) break; return args.FoundText; } return ""; } 

विश्लेषक चेतावनी : V3020 एक लूप के भीतर बिना शर्त 'वापसी'। कोड यूटिल्स .cs 262

बिना शर्त रिटर्न स्टेटमेंट के कारण , उपरोक्त लूप के लिए एक से अधिक पुनरावृत्ति निष्पादित नहीं की जाएगी। शायद यह कोड रीफैक्टरिंग के बाद निकला है, या शायद यह एक असामान्य तरीका है जो बिना लूप के किया जा सकता है।

 private int FindBarItem(string c) { for (int i = 0; i < tabelle_cb.Length; i++) { if (c == tabelle_cb[i].c) return i; } return -1; } internal override string GetPattern() { string result = tabelle_cb[FindBarItem("A")].data + "0"; foreach (char c in text) { int idx = FindBarItem(c.ToString()); result += tabelle_cb[idx].data + "0"; } result += tabelle_cb[FindBarItem("B")].data; return result; } 

चेतावनी PVS-Studio : V3106 संभावित नकारात्मक सूचकांक मूल्य। 'Idx' इंडेक्स का मान -1 तक पहुंच सकता है। BarcodeCodabar.cs 70

संभावित रूप से खतरनाक कोड। FindBarItem पद्धति -1 वापस आ सकती है यदि यह तत्व को पैरामीटर के रूप में पारित नहीं करता है। कॉलिंग कोड ( GetPattern method) में, यह मान idx वेरिएबल को लिखा जाता है और प्रारंभिक सत्यापन के बिना tabelle_cb एरे के सूचकांक के रूप में उपयोग किया जाता है। इंडेक्स -1 को एक्सेस करते समय , IndexOutOfRangeException प्रकार का एक अपवाद फेंक दिया जाएगा।

चलिए जारी रखते हैं।

 protected override void Finish() { .... if (saveStreams) { FinishSaveStreams(); } else { if (singlePage) { if (saveStreams) { int fileIndex = GeneratedFiles.IndexOf(singlePageFileName); DoPageEnd(generatedStreams[fileIndex]); } else { .... } .... } .... } .... } 

PVS-Studio चेतावनी : V3022 अभिव्यक्ति 'saveStreams' हमेशा झूठी है। HTMLExport.cs 849

FileIndex मान प्राप्त करने और DoPageEnd पद्धति को कॉल करने के साथ उपरोक्त कोड को कभी भी निष्पादित नहीं किया जाएगा, क्योंकि कोड में दूसरा saveStream अभिव्यक्ति का परिणाम हमेशा गलत होगा ।

सबसे दिलचस्प में से, यह शायद सब है (आप मोनो विश्लेषण की भावना में एक लेख की उम्मीद नहीं करते थे?)। अन्य विश्लेषक चेतावनी भी थे, लेकिन उन्हें मुझे लेख में शामिल करने के लिए पर्याप्त दिलचस्प नहीं लगा (भाग हमेशा पर्दे के पीछे रहता है)।

डिजाइन को जानना उनके विश्लेषण के लिए उपयोगी होगा, इसलिए आदर्श रूप से, लेखकों को इन चेतावनियों को स्वयं देखना चाहिए। ये V3083 (ईवेंट हैंडलर के लिए संभावित खतरनाक कॉल), V3022 (स्थिति हमेशा सही / गलत है (इस मामले में, अक्सर एक ही मान वापस करने के तरीके के कारण)), V3072 , V3073 ( आईडीसिसोप्लीकेशन पर काम कर रहे) और अन्य जैसे चेतावनी दे रहे हैं।

यदि इनमें से कोई भी अप्रासंगिक है, तो आप निम्न कर सकते हैं:

• डायग्नोस्टिक्स को बंद कर दें यदि इसके द्वारा पता लगाया गया पैटर्न प्रोजेक्ट पर ठीक है;
• यदि उनमें से बहुत सारे नहीं हैं तो झूठी चेतावनी के रूप में चेतावनी दें;
• यदि आप कोड में टिप्पणी नहीं करना चाहते हैं तो दमन डेटाबेस में चेतावनी जोड़ें । लेकिन फिर विश्लेषक के साथ काम करने वाले सभी के लिए दमन आधार तक पहुंच की आवश्यकता होगी।

निष्कर्ष

इस तथ्य के बावजूद कि लेख छोटा था, मुझे अपने हाथों से विश्लेषक की चेतावनियों को 'स्पर्श' करने की कृपा हुई - यह देखने के लिए कि व्यवहार में विश्लेषक की शपथ क्या है।

मैं परियोजना की सफलता, खोजे गए समस्याओं के सुधार के लेखकों की कामना करता हूं, और मैं एक कदम खुले स्रोत समुदाय की प्रशंसा करना चाहता हूं!

मैं बाकी को सलाह देता हूं कि अपने कोड पर विश्लेषक की कोशिश करें और देखें कि आपको क्या दिलचस्प मिल सकता है।

ऑल द बेस्ट!

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