अनपढ़ कोड समीक्षा वर्कफ़्लो को गंभीरता से धीमा कर देती है। जब कई दिनों (या सप्ताह!) के लिए बड़ी संख्या में परिवर्तन अटक जाते हैं, तो उत्पाद को बाजार से बाहर करना होगा। ऐसा होने के कुछ कारण यहां दिए गए हैं:

कोई मानक कोड डिज़ाइन नहीं है
कोई भी स्वचालित जांच नहीं की जाती है
प्रोग्रामर स्वतंत्र रूप से अपने कोड का विश्लेषण नहीं करते हैं।
विशाल पूल अनुरोध
अस्पष्ट पूल अनुरोध
कोड समीक्षा के लिए अनुपलब्ध समय सीमा

कोई मानक कोड डिज़ाइन नहीं है

प्रत्येक टीम को एक कोड प्रारूप मानक (प्रोग्रामिंग मानक, शैली गाइड) अपनाना चाहिए, जिसके साथ हर कोई सहमत हो। इसमें नामकरण सम्मेलनों, फ़ोल्डर संरचना, कोड स्वरूपण और सर्वोत्तम प्रथाओं की एक सूची शामिल है, जैसे कि इकाई परीक्षण, सत्यापन, आदि

। एक स्पष्ट मानक और सम्मेलनों की कमी हर डेवलपर को कोड लिखने के लिए मजबूर करती है जैसे वे चाहते हैं, जिससे कोड विकार हो जाते हैं। -review। यदि आप सम्मेलनों, नामकरण सम्मेलनों और अधिक के बारे में बहुत सारी टिप्पणियां देखते हैं, तो यह एक सामान्य मानक के बारे में बात करने का समय है।

आपकी टीम या तो सिफारिशों के अपने सेट को विकसित कर सकती है, या अन्य कंपनियों से प्रसिद्ध सिफारिशों के साथ शुरू कर सकती है। यहाँ Google से एक उदाहरण दिया गया है ।

कोई भी स्वचालित जांच नहीं की जाती है

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

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

प्रोग्रामर स्वतंत्र रूप से अपने कोड का विश्लेषण नहीं करते हैं।

एक कोड समीक्षा के लिए सहयोगियों से संपर्क करने से पहले, लेखक को अपने स्वयं के परिवर्तनों की समीक्षा करनी चाहिए। यह पते पर भेजने से पहले टाइपोस और त्रुटियों के लिए एक पत्र के पाठ की जाँच करना पसंद करता है।

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

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

विशाल पूल अनुरोध

पूल अनुरोध के लिए टिप्पणियों की संख्या इसमें निहित परिवर्तनों की संख्या के विपरीत आनुपातिक है। अर्थात्, एक बड़ा पीआर → कुछ टिप्पणियां, एक छोटा पीआर → कई टिप्पणियां।

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

पूल के अनुरोध को छोटे टुकड़ों में तोड़ दें जो एक दूसरे से अलग अर्थ रखते हैं। तो आप उन्हें ठीक से और जल्दी से विचार करने में मदद करेंगे।

अस्पष्ट पूल अनुरोध

अक्सर आपको एक अस्पष्ट विवरण के साथ या इसके बिना पूल अनुरोध की पेशकश की जाती है। समीक्षक को परिवर्तनों के अर्थ को समझना चाहिए, बैठक में या बग ट्रैकर में चर्चा की गई कार्य को याद करने की कोशिश करना। इसलिए, पीआर को सहायक जानकारी प्रदान करना बेहतर है:

इसमें क्या परिवर्तन होते हैं
क्या फ़ाइलों के साथ शुरू करने के लिए
बग ट्रैकर में कार्य से लिंक
स्क्रीनशॉट अगर यह किसी प्रकार का दृश्य परिवर्तन है

यह जानकारी समीक्षक के लिए एक बेहतर संदर्भ प्रदान करेगी और इस प्रकार कोड समीक्षा को गति प्रदान करेगी।

कोड समीक्षा के लिए अनुपलब्ध समय सीमा

कोड की समीक्षा को हमेशा के लिए फैलाने का एक तरीका यह है कि समीक्षक के लिए कोई समय सीमा नहीं रखी जाए। उदाहरण के लिए उचित समय सीमा निर्धारित करें:

पूल अनुरोध भेजने के क्षण से 48 घंटों के भीतर समीक्षा कोड पूरा किया जाना चाहिए।
हॉटफ़िक्स के लिए, अवधि 30 मिनट है।

पढ़ने के लिए धन्यवाद! :)

कोड समीक्षा को कैसे गति दें