एक 2 सप्ताह के स्प्रिंट के अंत में और एक कार्य की एक कोड समीक्षा होती है, समीक्षा में हम एक फ़ंक्शन की खोज करते हैं जो काम करता है, पठनीय है, लेकिन यह काफी लंबा है और इसमें कुछ कोड बदबू आ रही है। आसान रिफैक्टर का काम।
अन्यथा कार्य किए गए की परिभाषा फिट बैठता है।
हमारे पास दो विकल्प हैं।
- कोड समीक्षा में विफल, ताकि टिकट इस स्प्रिंट में बंद न हो, और हम मनोबल पर थोड़ा प्रहार करें, क्योंकि हम टिकट को पास नहीं कर सकते।
- रिफ्लेक्टर काम का एक छोटा सा टुकड़ा है, और अगले स्प्रिंट में (या इसके शुरू होने से पहले भी) एक छोटे, आधे बिंदु वाली कहानी के रूप में किया जाएगा।
मेरा सवाल यह है: क्या किसी भी तरह की अंतर्निहित समस्याएं या विचार हैं, जो असफल होने के बजाय समीक्षा की पीठ से टिकट काटते हैं?
जिन संसाधनों को मैं पा सकता हूं और आमतौर पर 100% या कुछ नहीं के रूप में डिटेल कोड समीक्षाएँ पढ़ता हूं, लेकिन मुझे लगता है कि आमतौर पर यथार्थवादी नहीं है।
23
इसलिए, यदि आप उसके लिए कोड समीक्षा को विफल नहीं कर सकते हैं, तो समीक्षा का उद्देश्य क्या है? अभी यह लगता है कि आपको यह देखना है कि क्या कुछ काम करता है , हालांकि, निश्चित रूप से यह एक परीक्षण या परीक्षक का काम है, न कि कोड की समीक्षा।
—
व्लाज
मुझे लगता है कि अधिकांश उत्तर आपके प्रश्न में एक महत्वपूर्ण बिंदु को याद करते हैं: अन्यथा कार्य किए गए की परिभाषा फिट बैठता है। क्या आप उन मुद्दों का उल्लेख करते हैं जो आपकी टीम "नहीं किया" कार्य मानती है? या इन मुद्दों पर विचार नहीं किया जाता है कि "किया" कार्य क्या होना चाहिए? यदि आपकी "की गई" परिभाषा में "कोई कोड गंध नहीं" शामिल है, तो कार्य बस नहीं किया जाता है।
—
जोश पार्ट
@ErdrikIronrose इसलिए ऐसा लगता है कि परिवर्तन मानक तक नहीं था और संभवतः (आसानी से) अनुरक्षित नहीं था। यद्यपि आपकी अन्य टिप्पणी यह संकेत देती है कि परिवर्तन अनुरोध का हिस्सा नहीं था, इस स्थिति में यह कोड समीक्षा का हिस्सा नहीं होना चाहिए। यदि कोई मौजूदा बदसूरत हैक के बगल में एक सही और अप-टू-स्टैंडर्ड कोड लिखता है, तो, बेझिझक, बदसूरत हैक को ठीक करने के लिए टिकट बढ़ाएं और वर्तमान कोड समीक्षा पास करें। यदि कोई ऐसा कोड लिखता है जो सही है लेकिन मानक तक नहीं है (जैसा कि आपका प्रश्न इंगित करता है), तो कोड समीक्षा को तब तक पूरा न करें जब तक कि यह सही तरीके से पूरा न हो जाए।
—
वीएलजी
@ErdrikIronrose: आह, इसलिए समीक्षा के तहत कहानी पर काम करते समय कोड गंध तरीका नहीं बनाया गया है , लेकिन पहले से ही अस्तित्व में है? यह एक महत्वपूर्ण अंतर है - प्रश्न को संपादित करने पर विचार करें।
—
स्लेजके
@vlaz आपको अपनी टिप्पणी से एक उत्तर देना चाहिए
—
Ister