हम ऑर्चर्ड सीएमएस कोड में त्रुटियों को खोजते हैं और उनका विश्लेषण करते हैं

यह लेख PVS-Studio स्थिर विश्लेषक का उपयोग करके ऑर्चर्ड परियोजना की फिर से जाँच करने का परिणाम है। ऑर्चर्ड एक ओपन सोर्स कंटेंट मैनेजमेंट सिस्टम है जो प्रोजेक्ट्स की एक गैर-लाभकारी ASP.NET गैलरी Outercurve Foundation का हिस्सा है। परीक्षण दिलचस्प है कि परियोजना और विश्लेषक पहले विश्लेषण के बाद काफी बढ़ गए हैं। नई सकारात्मक और दिलचस्प बग आपका इंतजार कर रही हैं।

ऑर्चर्ड सीएमएस के बारे में

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

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

ऑर्चर्ड प्रोजेक्ट कोड यहाँ से डाउनलोड किया जा सकता है , जैसा मैंने किया था। परियोजना का पूरा विवरण यहां पाया जा सकता है । यदि आपके पास अभी तक हमारे पीवीएस-स्टूडियो विश्लेषक नहीं हैं, तो आप यहां से एक परीक्षण संस्करण डाउनलोड कर सकते हैं । मैंने पीवीएस-स्टूडियो संस्करण 7.05 बीटा के संस्करण का उपयोग किया। मैं उसके काम के परिणामों को साझा करूंगा। मुझे आशा है कि आप पीवीएस-स्टूडियो का उपयोग करने की उपयोगिता से सहमत होंगे। मुख्य बात यह है कि नियमित रूप से विश्लेषक का उपयोग करना है।

मान्यता परिणाम

मैं आपको पिछले लेख के कुछ आंकड़े प्रदान करूंगा ताकि आपको तुलना करने के लिए स्विच न करना पड़े।

".Cs एक्सटेंशन के साथ सभी फाइलों (3739 टुकड़े) ने अंतिम जांच में भाग लिया। कुल में, उनके पास कोड की 214,564 लाइनें थीं। ऑडिट के परिणामों के आधार पर, 137 चेतावनी प्राप्त हुई थीं। पहले (उच्च) विश्वास स्तर पर, 39 चेतावनियाँ प्राप्त हुई थीं। दूसरे स्तर (औसत) पर, 60 चेतावनियाँ मिलीं। ”

फिलहाल, इस परियोजना में 2767 .cs फाइलें हैं, अर्थात यह परियोजना एक हजार फाइलों से कम हो गई है। फ़ाइलों की संख्या में कमी और रिपॉजिटरी के नाम में बदलाव से इसे देखते हुए परियोजना से एक कर्नेल आवंटित किया गया ( 966 के लिए )। कर्नेल में कोड की 108,287 लाइनें हैं और विश्लेषक ने 153 चेतावनी जारी की, 33 एक उच्च स्तर पर, 70 औसतन। हम आमतौर पर तीसरे स्तर की चेतावनी पर विचार नहीं करते हैं, मैंने परंपरा को नहीं तोड़ा है।

PVS-Studio चेतावनी : V3110 'TryValidateModel' पद्धति के भीतर संभावित पुनरावृत्ति। उपसर्गकृतमोडुलेपदत्र ed४ule ule

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

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

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

मुझे लगता है कि, अतिभारित विधि के साथ, डेवलपर _updateModel के माध्यम से तरीकों को कॉल करना चाहता था । कंपाइलर को कोई त्रुटि नहीं दिखाई दी, _updateModel IUpdateModel प्रकार का है और वर्तमान वर्ग भी इस इंटरफ़ेस को लागू करता है। चूंकि StackOverflowException के विरुद्ध सुरक्षा के लिए विधि में एक भी स्थिति नहीं है, इसलिए विधि को एक बार भी नहीं बुलाया जा सकता है, लेकिन मैं इस पर भरोसा नहीं करूंगा। यदि धारणा सही है, तो सही किया गया संस्करण इस तरह दिखाई देगा:

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

PVS-Studio चेतावनी: V3008 'सामग्री' चर को क्रमिक रूप से दो बार मान दिया गया है। शायद यह एक गलती है। चेक लाइन: 197, 190. डायनेमिकचैटेग हेल्पर। 197

 public override async Task ProcessAsync(....) { .... IHtmlContent content; .... try { content = await output.GetChildContentAsync(); } finally { _cacheScopeManager.ExitScope(); } content = await ProcessContentAsync(output, cacheContext); .... } 

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

पीवीएस-स्टूडियो चेतावनी : V3080 संभावित अशांति । 'आइटमटैग' का निरीक्षण करने पर विचार करें। CoreShapes.cs 92

 public async Task<IHtmlContent> List(....string ItemTag....) { .... string itemTagName = null; if (ItemTag != "-") { itemTagName = string.IsNullOrEmpty(ItemTag) ? "li" : ItemTag; } var index = 0; foreach (var item in items) { var itemTag = String.IsNullOrEmpty(itemTagName) ? null : ....; .... itemTag.InnerHtml.AppendHtml(itemContent); listTag.InnerHtml.AppendHtml(itemTag); ++index; } return listTag; } 

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

 public async Task<IHtmlContent> List(....string ItemTag....) { .... string itemTagName = null; if (ItemTag != "-") { itemTagName = string.IsNullOrEmpty(ItemTag) ? "li" : ItemTag; } var index = 0; foreach (var item in items) { var itemTag = ....; if(String.IsNullOrEmpty(itemTag)) throw .... .... itemTag.InnerHtml.AppendHtml(itemContent); listTag.InnerHtml.AppendHtml(itemTag); ++index; } return listTag; } 

PVS-Studio चेतावनी: V3095 नल के खिलाफ सत्यापित होने से पहले 'RemoteClient' ऑब्जेक्ट का उपयोग किया गया था। जाँच लाइनें: 49, 51. ImportRemoteInstanceController.cs 49

 public async Task<IActionResult> Import(ImportViewModel model) { .... var remoteClient = remoteClientList.RemoteClients.FirstOrDefault(....); var apiKey = Encoding.UTF8.GetString(....(remoteClient.ProtectedApiKey)); if (remoteClient == null || ....) { .... } .... } 

विश्लेषक ने डेरेफ्रेंसिंग को दूरस्थ रूप से देखा और नीचे शून्य के लिए जाँच की। यह वास्तव में एक संभावित NullReferenceException है , क्योंकि FirstOrDefault विधि डिफ़ॉल्ट मान (संदर्भ प्रकारों के लिए रिक्त ) लौटा सकती है। मुझे लगता है, कोड को ठीक करने के लिए, बस ऊपर दिए गए चेक को स्थानांतरित करें:

 public async Task<IActionResult> Import(ImportViewModel model) { .... var remoteClient = remoteClientList.RemoteClients.FirstOrDefault(....); if (remoteClient != null) var apiKey = UTF8.GetString(....remoteClient.ProtectedApiKey); else if (....) { .... } .... } 

हालाँकि, यह First के साथ FirstOrDefault को बदलने और चेक को पूरी तरह से हटाने के लायक हो सकता है।

पीवीएस-स्टूडियो 7.05 बीटा से:

फिलहाल, हमने LINQ से सभी orDefault methods को एनोटेट किया है। इस जानकारी का उपयोग नए डायग्नॉस्टिक्स द्वारा किया जाएगा, इन तरीकों को बिना जांच के कॉल करने के परिणाम की डीफ्रेंसिंग को ध्यान में रखते हुए। प्रत्येक orDefault विधि के लिए एक ऐसा एनालॉग है जो एक अपवाद को फेंकता है यदि उपयुक्त तत्व नहीं मिला। यह अपवाद अमूर्त NullReferenceException की तुलना में समस्या को समझने में अधिक मदद करेगा।

मैं इस परियोजना पर विकसित निदान का परिणाम साझा नहीं कर सकता। 27 संभावित खतरनाक स्थान। यहाँ उनमें से कुछ हैं:

ContentTypesAdminNodeNavigationBuilder.cs 71:

 var treeBuilder = treeNodeBuilders.Where(....).FirstOrDefault(); await treeBuilder.BuildNavigationAsync(childNode, builder, treeNodeBuilders); 

ListPartDisplayDriver.cs 217:

 var contentTypePartDefinition = ....Parts.FirstOrDefault(....); return contentTypePartDefinition.Settings....; 

ContentTypesAdminNodeNavigationBuilder.cs 113:

 var typeEntry = node.ContentTypes.Where(....).FirstOrDefault(); return AddPrefixToClasses(typeEntry.IconClass); 

PVS-Studio चेतावनी : V3080 विधि वापसी मान की संभावित अशांति । निरीक्षण करने पर विचार करें: CreateScope ()। SetupService.cs 136

 public async Task<string> SetupInternalAsync(SetupContext context) { .... using (var shellContext = await ....) { await shellContext.CreateScope().UsingAsync(....); } .... } 

इसलिए, विश्लेषक ने CreateScope विधि को कॉल करने के परिणाम की dereferencing पर ध्यान दिया। CreateScope विधि बहुत छोटी है, मैं इसे इसकी संपूर्णता में दूंगा:

 public ShellScope CreateScope() { if (_placeHolder) { return null; } var scope = new ShellScope(this); // A new scope can be only used on a non released shell. if (!released) { return scope; } scope.Dispose(); return null; } 

जैसा कि आप देख सकते हैं, यह दो मामलों में अशक्त लौट सकता है। विश्लेषक यह नहीं कह सकता है कि कार्यक्रम किस शाखा से गुजरेगा, इसलिए यह कोड को संदिग्ध मानता है। मेरे कोड में, मैं तुरंत एक अशक्त जांच जोड़ूंगा ।

शायद मैं पक्षपाती हूं, लेकिन सभी अतुल्यकालिक तरीकों का यथासंभव NullReferenceException के खिलाफ बीमा किया जाना चाहिए। इस तरह की चीजों को डिबेट करना बहुत ही खुशी का विषय है।

इस मामले में, CreateScope पद्धति में चार कॉल हैं, और दो में एक चेक है, लेकिन अन्य दो में यह गायब है। इसके अलावा, सत्यापन के बिना एक जोड़ी कॉपी-पेस्ट (एक वर्ग, एक विधि, यूजेरसिंस्क कॉल करने के लिए dereferenced) के समान है। मैंने पहले ही इस जोड़ी से पहली कॉल की है और निश्चित रूप से, दूसरी कॉल में भी एक समान विश्लेषक चेतावनी है:

V3080 विधि वापसी मूल्य की संभावित अशक्तता । निरीक्षण करने पर विचार करें: CreateScope ()। SetupService.cs 192

PVS-Studio चेतावनी: V3127 दो समान कोड अंश पाए गए। शायद, यह एक टाइपो है और 'ConsumerSecret' TwitterClientMessageHandler.cs 52 के बजाय 'AccessTokenSecret' चर का उपयोग किया जाना चाहिए

 public async Task ConfigureOAuthAsync(HttpRequestMessage request) { .... if (!string.IsNullOrWhiteSpace(settings.ConsumerSecret)) settings.ConsumerSecret = protrector.Unprotect(settings.ConsumerSecret); if (!string.IsNullOrWhiteSpace(settings.ConsumerSecret)) settings.AccessTokenSecret = protrector.Unprotect(settings.AccessTokenSecret); .... } 

क्लासिक कॉपी-पेस्ट। ConsumerSecret को दो बार चेक किया गया था , और AccessTokenSecret - एक बार नहीं। जाहिर है, आपको ठीक करने की आवश्यकता है:

 public async Task ConfigureOAuthAsync(HttpRequestMessage request) { .... if (!string.IsNullOrWhiteSpace(settings.ConsumerSecret)) settings.ConsumerSecret = protrector.Unprotect(settings.ConsumerSecret); if (!string.IsNullOrWhiteSpace(settings.AccessTokenSecret)) settings.AccessTokenSecret = protrector.Unprotect(settings.AccessTokenSecret); .... } 

PVS-Studio चेतावनी: V3139 दो या दो से अधिक केस-शाखाएँ एक ही क्रिया करते हैं। SerialDocumentExecuter.cs 23

एक और कॉपी-पेस्ट गलती। इसे स्पष्ट करने के लिए, मैं पूरी कक्षा दूंगा, क्योंकि यह छोटा है।

 public class SerialDocumentExecuter : DocumentExecuter { private static IExecutionStrategy ParallelExecutionStrategy = new ParallelExecutionStrategy(); private static IExecutionStrategy SerialExecutionStrategy = new SerialExecutionStrategy(); private static IExecutionStrategy SubscriptionExecutionStrategy = new SubscriptionExecutionStrategy(); protected override IExecutionStrategy SelectExecutionStrategy(....) { switch (context.Operation.OperationType) { case OperationType.Query: return SerialExecutionStrategy; case OperationType.Mutation: return SerialExecutionStrategy; case OperationType.Subscription: return SubscriptionExecutionStrategy; default: throw ....; } } } 

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

 switch (context.Operation.OperationType) { case OperationType.Query: case OperationType.Mutation: return SerialExecutionStrategy; case OperationType.Subscription: return SubscriptionExecutionStrategy; default: throw ....; } 

यदि यह एक कॉपी-पेस्ट त्रुटि है, तो आपको इस तरह से दिए गए फ़ील्ड को सही करने की आवश्यकता है:

 switch (context.Operation.OperationType) { case OperationType.Query: return ParallelExecutionStrategy; case OperationType.Mutation: return SerialExecutionStrategy; case OperationType.Subscription: return SubscriptionExecutionStrategy; default: throw ....; } 

या इसके विपरीत। मैं परियोजना से परिचित नहीं हूं और संचालन और रणनीतियों के प्रकारों के नामों को सहसंबंधित नहीं कर सकता।

 switch (context.Operation.OperationType) { case OperationType.Query: return SerialExecutionStrategy; case OperationType.Mutation: return ParallelExecutionStrategy; case OperationType.Subscription: return SubscriptionExecutionStrategy; default: throw ....; } 

पीवीएस-स्टूडियो चेतावनी : V3080 संभावित अशांति । 'अनुरोध' का निरीक्षण करने पर विचार करें। ग्राफक्लाइनमेडवेयर.केएस 148

 private async Task ExecuteAsync(HttpContext context....) { .... GraphQLRequest request = null; .... if (HttpMethods.IsPost(context.Request.Method)) { .... } else if (HttpMethods.IsGet(context.Request.Method)) { .... request = new GraphQLRequest(); .... } var queryToExecute = request.Query; .... } 

पहले अगर ब्लॉक में, अनुरोध चर को कई स्थानों पर शून्य के अलावा एक मूल्य मिलता है, लेकिन हर जगह पर नीडिंत स्थिति के साथ। मैंने इन सभी स्थितियों का हवाला नहीं दिया, क्योंकि उदाहरण बहुत बोझिल होगा। पहले प्रकार जो IsGet या IsPost विधि के http प्रकार की जाँच करते हैं वे पर्याप्त हैं । Microsoft.AspNetCore.Http.HttpMethods वर्ग में अनुरोध के प्रकार की जाँच के लिए नौ स्थिर विधियाँ हैं। इसका अर्थ है कि यदि, उदाहरण के लिए, हटाएं या सेट करें अनुरोध ExecuteAsync विधि में हो जाता है, तो NullReferenceException को फेंक दिया जाएगा। यहां तक ​​कि अगर इस समय ऐसे तरीकों का समर्थन नहीं किया जाता है - तो अपवाद के साथ जांच करना बेहतर है। आखिरकार, सिस्टम की आवश्यकताएं बदल सकती हैं। नीचे सत्यापन उदाहरण:

 private async Task ExecuteAsync(HttpContext context....) { .... if (request == null) throw ....; var queryToExecute = request.Query; .... } 

PVS-Studio चेतावनी : V3080 विधि वापसी मान की संभावित अशांति । निरीक्षण करने पर विचार करें: Get <ContentPart> (...)। ContentPartHandlerCoordinator.cs 190

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

 public override async Task LoadingAsync(LoadContentContext context) { .... context.ContentItem.Get<ContentPart>(typePartDefinition.Name) .Weld(fieldName, fieldActivator.CreateInstance()); .... } 

विश्लेषक इस लाइन पर शपथ लेता है। आइए देखें विधि :

 public static TElement Get<TElement>(this ContentElement contentElement....) where TElement : ContentElement { return (TElement)contentElement.Get(typeof(TElement), name); } 

यह उसके अधिभार का कारण बनता है। चलो उसे देखो:

 public static ContentElement Get(this ContentElement contentElement....) { .... var elementData = contentElement.Data[name] as JObject; if (elementData == null) { return null; } .... } 

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

विकल्प एक:

 public static ContentElement Get(this ContentElement contentElement....) { .... var elementData = contentElement.Data[name] as JObject; if (elementData == null) { throw.... } .... } 

विकल्प दो:

 public override async Task LoadingAsync(LoadContentContext context) { .... context.ContentItem.Get<ContentPart>(typePartDefinition.Name) ?.Weld(fieldName, fieldActivator.CreateInstance()); .... } 

पीवीएस-स्टूडियो चेतावनी : V3080 संभावित अशांति । 'परिणामों' का निरीक्षण करने पर विचार करें। ContentQueryOrchardRazorHelperExtensions.cs 19

 public static async Task<IEnumerable<ContentItem>> ContentQueryAsync(....) { var results = await orchardHelper.QueryAsync(queryName, parameters); .... foreach (var result in results) { .... } .... } 

पिछले एक के सापेक्ष काफी सरल उदाहरण। विश्लेषक मानता है कि QueryAsync को कॉल करने का परिणाम एक शून्य संदर्भ हो सकता है। यहाँ विधि है:

 public static async Task<IEnumerable> QueryAsync(....) { .... var query = await queryManager.GetQueryAsync(queryName); if (query == null) { return null; } .... } 

यहाँ GetQueryAsync एक इंटरफ़ेस विधि है, आप हर कार्यान्वयन के बारे में सुनिश्चित नहीं हो सकते। इसके अलावा, एक ही परियोजना में इस तरह के एक विकल्प है:

 public async Task<Query> GetQueryAsync(string name) { var document = await GetDocumentAsync(); if(document.Queries.TryGetValue(name, out var query)) { return query; } return null; } 

GetDocumentAsync विधि का विश्लेषण बाहरी फ़ंक्शन और कैश एक्सेस तक कॉल की प्रचुरता से जटिल है। आइए हम इस तथ्य पर ध्यान दें कि यह चेक जोड़ने के लायक है। इसके अलावा, विधि अतुल्यकालिक है।

 public static async Task<IEnumerable<ContentItem>> ContentQueryAsync(....) { var results = await orchardHelper.QueryAsync(queryName, parameters); if(results == null) throw ....; .... foreach (var result in results) { .... } .... } 

निष्कर्ष

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

हालाँकि एकमुश्त जाँच यथासंभव प्रभावी नहीं है, लेकिन मैं आपसे आग्रह करता हूं कि आप अपने प्रोजेक्ट पर पीवीएस-स्टूडियो डाउनलोड करें और कोशिश करें - क्या होगा अगर आप कुछ दिलचस्प पा सकते हैं?

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