Kako ste izlazili iz situacija kada se niste slagali sa ocenom kod revizije?

Када дође до неслагања у процени ревизије кода, примењујем следећи приступ:

1. Пажљиво проучавам коментар рецензента: Трудићу се да разумем главни разлог његове забелешке и предлог за побољшање. Могуће је да сам нешто пропустио или њихова тачка гледишта се заснива на другом искуству.

2. Анализирам своју позицију: Прегледам своју почетну одлуку и аргументујем зашто сам изабрао такав приступ. Важно је јасно формулисати своје аргументе.

3. Директно дискутујем о забелешци: Уместо пасивног неслагања, иницирам дискусију са рецензентом. Ово може бити синхрона комуникација преко месинџера, састанак или коментар у систему прегледа кода. Моја цел је не да "победим" у спору, већ да дођем до оптималног решења.

4. Предлажем алтернативна решења: Ако моје решење није задовољавајуће, спреман сам да размотрим друге опције које предлаже рецензент или да заједнички развијемо нови приступ који узима у обзир обе тачке гледишта.

5. Ако је потребно, укључујем трећу особу: Ако након дискусије не буде постигнут консензус, могу се обратити другом члану тима или вођи тима за независно мишљење.

Пример дискусије у систему прегледа кода:

// Мој коментар на забелешку
// Здраво! Хвала на ревизији. Разумем твоју тачку гледишта у вези са коришћењем this.findViewById.
// Одабрао сам овај приступ из разлога перформанси, јер је хијерархија изгледа овде мала.
// Можеш ли објаснити зашто у овом конкретном случају предлажеш коришћење ViewBinding?

Пример дискусије ако рецензент предлаже оптимизацију која изгледа прекомерна:

Донео бих податке у табеларном облику, илуструјући утицај предложене оптимизације на перформансе у овом конкретном случају у поређењу са временом које је потребно за њену имплементацију.

Оптимизација	Потенцијални утицај на перформансе	Време имплементације
Почетно решење	Базно	0
Предложена оптимизација	Минимална (око 1-2 мс)	1 сат

На крају, мој главни циљ је побољшање квалитета кода и његово унапређење, а не само одбрана мог становишта. Отворен сам за конструктивну критику и спреман сам да променим своју одлуку ако се представе убедљиви аргументи.