Sobes.tech
Middle+

Ako ste riešili situácie, keď ste nesúhlasili s hodnotením kódovej revízie?

sobes.tech AI

Odpoveď od AI

Keď vzniknú nezhody pri hodnotení revízie kódu, uplatňujem nasledujúci prístup:

  1. Dôkladne preskúmam komentár recenzenta: Snažím sa pochopiť hlavný dôvod jeho poznámky a návrh na zlepšenie. Je možné, že som niečo prehliadol alebo jeho pohľad je založený na inom skúsenosti.

  2. Analyzujem svoju pozíciu: Prehodnotím svoje pôvodné rozhodnutie a argumentujem, prečo som zvolil práve tento prístup. Je dôležité jasne formulovať svoje argumenty.

  3. Priamo diskutujem poznámku: Namiesto pasívneho nesúhlasu začínam diskusiu s recenzentom. To môže byť synchronná komunikácia cez messenger, stretnutie alebo komentár v systéme revízie kódu. Môj cieľ nie je "vyhrať" spor, ale dosiahnuť optimálne riešenie.

  4. Navrhujem alternatívne riešenia: Ak moje riešenie nie je uspokojivé, som ochotný zvážiť iné možnosti navrhnuté recenzentom alebo spoločne vyvinúť nový prístup, ktorý zohľadní obe názory.

  5. V prípade potreby zapájam tretieho: Ak po diskusii nie je dosiahnutý konsenzus, môžem sa obrátiť na iného člena tímu alebo vedúceho tímu pre nezávislý názor.

Príklad diskusie v systéme revízie kódu:

// Môj komentár k poznámke
// Ahoj! Ďakujem za revíziu. Chápem tvoj pohľad na použitie this.findViewById.
// Tento prístup som zvolil z dôvodov výkonu, pretože hierarchia pohľadov je tu malá.
// Môžeš vysvetliť, prečo v tomto konkrétnom prípade navrhuješ použiť ViewBinding?

Príklad diskusie, ak recenzent navrhne optimalizáciu, ktorá sa zdá byť prehnaná:

Urobil by som prezentáciu údajov v tabuľkovej forme, ilustrujúc vplyv navrhovanej optimalizácie na výkon v tomto konkrétnom prípade v porovnaní s časom, ktorý je potrebný na jej implementáciu.

Optimalizácia Potenciálny nárast výkonu Čas implementácie
Pôvodné riešenie Základ 0
Navrhovaná optimalizácia Minimálna (približne 1-2 ms) 1 hodina

Nakoniec, mojím hlavným cieľom je zlepšiť kvalitu kódu a urobiť ho lepším, nie len brániť svoj názor. Som otvorený konštruktívnej kritike a pripravený zmeniť svoje rozhodnutie, ak budú predložené presvedčivé argumenty.