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

• समीक्षा प्रक्रिया
• गति (और यह क्या प्रभावित करता है)
• टिप्पणी कैसे लिखें,
• समीक्षा के दौरान चर्चा।

1. समीक्षा में परिवर्तन की सूची के माध्यम से नेविगेशन

टीएल, डॉ

अब जब आप समझते हैं कि आपको किस पर ध्यान देने की आवश्यकता है, तो आपको यह तय करने की आवश्यकता है कि समीक्षा का क्रम क्या है। 3 मुख्य चरण हैं:

1. यह समझें कि क्या सिद्धांत में प्रस्तुत कार्यक्षमता की आवश्यकता है और क्या इसका अच्छा वर्णन है ;
2. संशोधित कोड में सबसे महत्वपूर्ण बात पर ध्यान देना और समाधान को पूरी तरह से कितनी अच्छी तरह तैयार किया गया है;
3. आपके द्वारा देखे गए क्रम में शेष परिवर्तन देखें।

1.1। अवलोकन बदलें

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

: " , , , FooWidget, . . , BarWidget . ?"

, : CL . : , .

, , , — , . "" , , .

1.2.

CL — , . — . CL , , CL .

CL, , CL . , , — .

:

• CL . , .
• . , , , .

1.3.

CL , . . , . — .

2.

2.1. - ?

Google , , , . , .

, - :

• . , , , . , , , , .
• . , . . , , , CL , . , .
• . , . , CL.

2.2. ?

- , , .

— ( , , ).

, CL ( ) .

2.3. vs

, : - , , , . , , . , , , .

: , , , .

2.4.

-, , , , , CL . , .

, , .

, , , CL , . . , — .

, , . , , .

2.5.

, , , , . , .

2.6. LGTM (looks good to me = )

, , CL . :

• , .
• , .

LGTM , "LGTM, Approval".

2.6. CL

CL , CL CL, . , .

CL , , CL, , . , , .

2.7. -

, , . , CL . . , , — , .

2.8.

, CL . . , , " ".

3.

TL;DR

• .
• .
• , , .
• , .

3.1.

, , , , . — , . , , , -, . :

: " , ?"

: " , , . , , ."

3.2.

"" , . , — , .

3.3.

CL, . .

. , , . , , , , , , , , .

, . , CL , — , .

3.4.

, , , . , , , , .

, , . : , , , , , .

4.

, .

4.1. ?

, — , . , . ? ? , .

, . , . , - .

, , , . .

, . , . , .

4.2. ?

, , . , , , , , . , , , . , , .

4.3. " "

( ) , . , - CL, . CL . , , CL, - . , . , — , , , "". .

CL , CL , . CL , , , , TODO- .

4.4.

, , . .

, , , .

यहां तक ​​कि सख्त समीक्षाओं के सबसे प्रबल विरोधी आपके सबसे बड़े सहयोगी बन सकते हैं जब व्यवहार में वे एक ऐसी स्थिति का सामना करते हैं जो यह साबित करता है कि समीक्षा में आपकी कठिन स्थिति सही थी।

4.5। संघर्ष समाधान

यदि आप उपरोक्त सभी नियमों का पालन करते हैं, लेकिन डेवलपर्स के साथ अभी भी संघर्ष हैं और आप उन्हें हल नहीं कर सकते हैं, तो प्रथाओं और मानकों को समझने के लिए कोड समीक्षा मानकों का संदर्भ लें जो संघर्ष को सुलझाने में आपकी सहायता कर सकते हैं।