2009-05-25 27 views
7

हमारी टीम पर, हमें कोड समीक्षा करना पसंद है, लेकिन जब भी आप "ज़ोन" प्रोग्रामिंग में होते हैं और समीक्षा को पूर्ववत करने के लिए टैप करते हैं तो कोड समीक्षा किसी भी समय हो सकती है। आप समीक्षा करते हैं, जो कि बहुत अच्छा है, लेकिन फिर आपको "ज़ोन" में आने से पहले दस मिनट लगते हैं।अनुसूचित कोड समीक्षा?

मैं इस बारे में बहस कर रहा था कि कोड समीक्षा करने की ज़रूरतों को कैसे संभालना सर्वोत्तम है, लेकिन उत्पादन पर प्रभाव को कम करना। इसलिए मैं किसी प्रकार का शेड्यूल करने का विचार मनोरंजन कर रहा था: एक समय जहां प्रोग्रामर जोड़े जाते हैं और दिन के कोड के बारे में बात करते हैं, लेकिन इसका मतलब यह है कि आपको समीक्षा करने के लिए समीक्षा समय तक इंतजार करना होगा। तो मैं अभी भी इसके बारे में अनिश्चित हूँ।

तो आप बाधाओं को कम करते समय कोड समीक्षाओं का सर्वोत्तम प्रबंधन कैसे करते हैं?

धन्यवाद!

उत्तर

5

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

सर्वोत्तम कोड समीक्षा तब होती है जब कोड लिखने वाला व्यक्ति वास्तव में उन्हें खोजता है। ऐसा कहा जा रहा है कि, हमारे पास ऐसा नियम है कि इसकी समीक्षा होने तक कोई कोड चेक नहीं किया गया है। इस तरह यह एक निर्धारित समीक्षा का एक प्रकार है। हालांकि, यदि आप अंत तक प्रतीक्षा करते हैं, तो आपको डिज़ाइन को प्रभावित करने में सहायता करने वाली कोड समीक्षा का लाभ नहीं मिलता है।

आप स्वयं को परेशानी भी बचा सकते हैं और कुछ जोड़ी प्रोग्रामिंग कर सकते हैं। आप उत्पादकता मिलता है और एक ही समय में सभी की समीक्षा;)

+0

हम जो कहते हैं उसमें हम 100% मानते हैं। सवाल, मुझे लगता है कि, जब आपको लगता है कि कोड की समीक्षा करने में बड़ी मात्रा में समय बिताया जाता है तो क्या करना है। जब आपकी टीम चार या पांच लोगों की है, तो आप इसे देखते हैं। कुछ लोगों को कोड के कुछ हिस्सों की तुलना दूसरों के मुकाबले ज्यादा करने की ज़रूरत है। यहां तक ​​कि यदि आप भार को संतुलित करने और सदस्यों के बीच समीक्षा साझा करने का प्रयास करते हैं, तो यह वास्तविकता है कि हर कोई कोडबेस के हर हिस्से के बारे में नहीं जानता है। – djcouchycouch

0

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

1

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

या बेहतर अभी तक, जोड़ी प्रोग्रामिंग चीज (आपको व्यावहारिक रूप से निरंतर कोड समीक्षा दे रही है) करें, लेकिन अभी भी औपचारिक कोड समीक्षाएं हैं, बस कम बार।

0

G'day,

मैं एक impromtu समीक्षा की इस तरह लगता है जब आप मुसीबत कुछ कोड को समझने कर रहे हैं इस्तेमाल किया जाना चाहिए।

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

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

तो संक्षेप में, समीक्षाओं को पहले से शेड्यूल करें और सुनिश्चित करें कि समीक्षक के पास अपने समय में कोड पर जाने के लिए पर्याप्त समय है।

HTH

चियर्स,

0

व्यापार बंद लोगों की समीक्षा के लिए तैयार करने के लिए समय दे रही है और अवरोधों को कम करने के बीच एक कठिन एक है। दूसरी बार ट्रेड-ऑफ, समीक्षा अभी भी हो रही है, जबकि कोड अभी भी "ताजा" है, और अधिक जटिल है।

मैंने देखा है कि समूहों का एक समूह सड़क के नीचे सबसे अच्छा इरादा रखता है, केवल अभ्यास को सूखने और मरने के कारण, नेट पर, यह बहुत विघटनकारी है।

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

आप अभी भी महत्वपूर्ण कोड की समूह समीक्षा करना चुन सकते हैं, लेकिन कम समीक्षा के साथ, लोगों को एक विशेष व्यक्ति के लिए तैयार होने का मौका मिलता है।

यह एक ऐसा अभ्यास है जो हर किसी के लिए राक्षस रूप से नहीं है, लेकिन जिन टीमों ने मैंने काम किया है, उन्होंने टीडीडी और जोड़ी प्रोग्रामिंग को कम दोष वाले गिनती के साथ बहुत साफ कोड दिया है।

0

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

मैं मानता हूँ कि जोड़ी प्रोग्रामिंग एक मूल्यवान अभ्यास है, और यह कोड की समीक्षा करने के कुछ लाभ प्रदान करता है कि (और निश्चित रूप से अन्य कोड की समीक्षा के लिए असंबंधित लाभ!)

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

0

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

तत्काल प्रतिक्रिया है, गुणवत्ता में सुधार हुआ है क्योंकि कोड बदल गया है और यह डेवलपर के दिमाग में ताजा है इसलिए वह इससे सीखता है।

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

संबंधित मुद्दे