शीर्ष डाउन कोड समीक्षा कितनी फायदेमंद है?

Question

ऐसा लगता है कि अधिकतर कोड समीक्षा नीचे तक हैं - यह है कि निचले इंजीनियरों को उनके कोड की समीक्षा उच्चतम और प्रबंधकों द्वारा उच्च अप्स द्वारा की जाती है या नहीं।

कुछ नए स्नातकों के साथ काम करते समय यह मेरे लिए हुआ कि शीर्ष डाउन कोड समीक्षा एक बड़ी बात हो सकती है। विचार यह है कि हम बड़े लोगों ने शायद कुछ बुरी आदतों को अपनाया है जिस तरह से अब ध्यान नहीं दिया जा रहा है - कुछ नए लोग सर्वोत्तम प्रथाओं पर ताजा होने पर उठा सकते हैं।

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

किसी के पास इसका अनुभव है?

Answer 1

मैं सहकर्मी-आधारित कोड समीक्षाओं के साथ जाता हूं जहां हर कोई सप्ताह में एक बार प्रोजेक्टर पर कोड को देख सकता है और टिप्पणी कर सकता है, प्रश्न पूछ सकता है, सुझाव दे सकता है।

हमें एहसास है कि हमें पूर्ण कवरेज नहीं मिल सकता है, लेकिन हम गर्म स्थानों पर हमला करते हैं और आम तौर पर उन अन्य क्षेत्रों में टीम द्वारा लाभ प्राप्त किए जाते हैं जिन पर हम काम कर रहे हैं।

हम नियमित रूप से एक दूसरे कोड स्निपेट को ईमेल करते हैं जिन पर हमारे पास प्रश्न हैं, या तो हम कर रहे हैं, या हम पिछले पुनरावृत्तियों से बाएं पाते हैं। कोई भी कमरे में दो लोगों को पकड़ सकता है और बैठकर कोड पर जा सकता है क्योंकि इसे दोबारा दोबारा लिखा/पुनः लिखा जा सकता है।

Answer 2

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

हालांकि, इसके अलावा, एक शीर्ष-डाउन कोड समीक्षा जैसा कि आप उल्लेख करते हैं, नए डेवलपर के लिए एक महान सीखने का अवसर है और उच्च स्तरीय डेवलपर के लिए एक अच्छा शिक्षण/सीखने का अवसर हो सकता है।

आखिरकार, मुझे तनाव होगा कि "बूढ़े लोगों" को सर्वोत्तम प्रथाओं पर बने रहने के लिए क्या करना चाहिए। 99.9% मामलों में क्षेत्र में नया लोग आपसे सीखने जा रहे हैं, और एक अच्छा उदाहरण स्थापित करना कुछ महत्वपूर्ण है। कम से कम खुद के लिए, और टीम के सदस्य जो मैं सर्वोत्तम प्रथाओं को जानने के साथ काम करता हूं, और कम से कम जितना संभव हो सके "सही तरीके" कर रहा हूं, हमारी दिन-प्रतिदिन की गतिविधियों का एक अभिन्न अंग है।

Answer 3

मेरे अनुभव में आप अपने कोड होने एक सहकर्मी, एक कम वरिष्ठ इंजीनियर या एक अधिक वरिष्ठ इंजीनियर द्वारा समीक्षा से अलग लाभ मिल ...

• एक सहकर्मी रचनात्मक आलोचना, प्रतिक्रिया प्रदान करता है और यह भी एक और हो जाता है यह समझने के लिए संसाधन कि आप क्या कर रहे हैं और भविष्य में इसे कैसे बनाए रखा जाए।

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

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

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