Kaip išėjote iš situacijų, kai nesutinkate su kodo peržiūros įvertinimu?

Kai kyla nesutarimai dėl kodo peržiūros įvertinimo, aš taikau šį požiūrį:

1. Atidžiai nagrinėju peržiūros komentaru: Stengiuosi suprasti jų pastabos pagrindinę priežastį ir pasiūlymą tobulinti. Galbūt kažką praleidau arba jų požiūris grindžiamas kitu patirtimi.

2. Analizuoju savo poziciją: Peržiūriu savo pradinį sprendimą ir argumentuoju, kodėl pasirinkau būtent tokį požiūrį. Svarbu aiškiai suformuluoti savo argumentus.

3. Tiesiogiai diskutuojame: Vietoj pasyvios nesutikimo, inicijuoju diskusiją su peržiūros dalyviu. Tai gali būti sinchroninis bendravimas žinute, susitikimas ar komentaras pačioje kodo peržiūros sistemoje. Mano tikslas — ne „laimėti“ ginče, o pasiekti optimalų sprendimą.

4. Pasiūlyti alternatyvius sprendimus: Jei mano sprendimas netenkina, esu pasiruošęs apsvarstyti kitus variantus, pasiūlytus peržiūros dalyvio, arba kartu sukurti naują požiūrį, atsižvelgiant į abi nuomones.

5. Reikalingas įtraukiu trečią asmenį: Jei po diskusijos nesusitariama, galiu kreiptis į kitą komandos narį ar vadovą, kad gauti nepriklausomą nuomonę.

Pavyzdys apie diskusiją kodo peržiūros sistemoje:

// Mano komentaras prie pastabos
// Sveiki! Ačiū už peržiūrą. Suprantu jūsų požiūrį dėl this.findViewById naudojimo.
// Pasirinkau šį požiūrį dėl našumo, nes čia View gylis mažas.
// Ar galėtumėte paaiškinti, kodėl būtent šiuo atveju siūlote naudoti ViewBinding?

Jei peržiūros dalyvis siūlo optimizaciją, kuri atrodo perteklinė:

Aš pateikiu duomenis lentelės formatu, iliustruodamas siūlomos optimizacijos poveikį našumui šiame konkrečiame atveju, palyginti su jos įgyvendinimo laiku:

Optimizacija	Potencialus našumo padidėjimas	Įgyvendinimo laikas
Pradinė sprendimas	Bazinis	0
Siūloma optimizacija	Minimalus (apie 1-2 ms)	1 valanda

Pagrindinis mano tikslas — pagerinti kodo kokybę ir padaryti jį geresnį, o ne tik apginti savo požiūrį. Esu atviras konstruktyviai kritikai ir pasiruošęs keisti savo sprendimą, jei bus pateikti įtikinami argumentai.