2008-09-22 18 views
13

मैंने हाल ही में 30 या उससे अधिक डेवलपर्स वाली एक कंपनी से स्थानांतरित किया है, जो एक ही साइट पर आधारित है, एक छोटी कंपनी के लिए जो पूरे देश में काम कर रहे कुल 6 डेवलपर्स के साथ है।कोड समीक्षा कैसे करते हैं?

पहले जब मैं कोड समीक्षा आयोजित कर रहा था तो मेरे पास मेरे बगल में बैठे व्यक्ति होंगे, या मैं उनके पीसी पर जाऊंगा और उनके साथ कोड पर जाउंगा।

मेरी नई कंपनी में यह संभव नहीं है क्योंकि लोग पूरी जगह पर हैं। मुझे समीक्षा के लिए कोड की कुछ हज़ार लाइनें सौंपी गई हैं और मैं सिर्फ किसी को अपने विचार नहीं बता सकता।

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

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

क्या कोई ऐसा उपकरण है जो ऐसा करता है? यदि नहीं, तो वितरित कोड समीक्षा करने के बारे में लोग कैसे जाते हैं?

+0

मैं शीर्षक को बदलने के सुझाव देता हूं कि "आप वितरित कोड समीक्षा कैसे करते हैं?" –

+0

कोड-समीक्षा उचित टैग है: "एकाधिक शब्दों को एकल शब्दों में संयोजित करें", और अधिक उपयोग किया गया है। –

उत्तर

8

आप स्मार्ट भालू किताब पढ़ने के लिए पसंद कर सकते हैं: Best Kept Secrets of Peer Code Review

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

0

JIRA में आपके काम को कोड-समीक्षा करने के लिए किसी को असाइन करने का विकल्प है, फिर वह जेरा के भीतर टिप्पणियां दे सकता है।

0

हम पूरी तरह से उसी देश में थे - पूरे देश में डेवलपर्स, दिन के दौरान पर्याप्त ओवरलैप समय नहीं ... हर कोई अतुल्यकालिक रूप से काम कर रहा था ... इसलिए हमें ऑफ़लाइन कोड-समीक्षा टूल के साथ आना पड़ा।

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

5

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

TeamReview screenshot http://www.codeplex.com/Project/Download/FileDownload.aspx?ProjectName=TeamReview&DownloadId=32263

1

http://www.review-board.org/

मैं इस परियोजना नहीं भी बहुत पहले देखा था, शायद यह मतदान कर सकते हैं, तो लोग इसका उपयोग ।

1

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

इसके अलावा, निर्माण से स्वचालित रूप से जेनरेट की गई कोड कवरेज रिपोर्ट देखें। यह परीक्षण पथ पर एक अच्छा मीट्रिक हो सकता है।

यही वह जगह है जहां मैं शुरू करूंगा।

2

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

0

हमें वितरित कोड समीक्षा करने के लिए टीएफएस शेल्फसेट सुविधा का उपयोग करके कुछ सफलता मिली है। अलमारियों का उपयोग करके एक डेवलपर शाखा में आने के बिना एक परिवर्तन में 'चेक इन' कर सकता है। एक और डेवलपर तब परिवर्तन को अनदेखा कर सकता है और प्रतिबद्ध शाखा के खिलाफ अंतर की समीक्षा कर सकता है। एक बार जब संबंधित सभी लोग परिवर्तन से खुश होते हैं तो वे शाखा में जाते हैं। http://www.smartbear.com/codecollab.php

2

एक उपकरण नहीं है। अनुमोदित, यह चेक-इन नीति के रूप में औपचारिक नहीं है, लेकिन नीति में चेक में स्मारक करने से पहले मानकों को विकसित करने के लिए शुरू करने के लिए एक अच्छी जगह हो सकती है।

0

एक बहुत ही हल्के प्रक्रिया है कि हम प्रयोग कर रहे हैं माइक्रोसॉफ्ट SharedView और स्काइप दोनों का उपयोग करने जोड़ी प्रोग्रामिंग सत्र के लिए और कोड की समीक्षा करने के लिए है:

2

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

हमारे डेवलपर्स समीक्षा करने में सहायता के लिए, हमने एक कोड समीक्षा चेकलिस्ट (http://www.macadamian.com/index.php?option=com_content&task=view&id=27&Itemid=31 पर उपलब्ध) को एक साथ रखा है।

कोड समीक्षा के पीछे हमारा रहस्य: समीक्षा की समीक्षा करें, अक्सर समीक्षा करें।

0

ठीक है, मैं आपको बताऊंगा कि मेरी कंपनी क्या करती है क्योंकि यह हमारे लिए अच्छी तरह से काम करती है (लगभग 10 डेवलपर्स प्लस पीएम पीपीएल)। YMMV।

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

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