Kako provodite učinkovite i poštovane recenzije koda?

Question

Accepted Answer

Recenzija koda jedan je od aktivnosti s najvećom polugom za TL: štiti kvalitetu *i* poučava. Cilj je **bolji kod i jači tim**, ne čuvanje vrata. Recenzije trebaju biti brze, ljubazne i fokusirane na ono što je važno.

## Načela

```text
✓ Review the CODE, never the person ("this function" not "you")
✓ Distinguish must-fix from nice-to-have (label nits explicitly)
✓ Ask questions, don't issue commands ("what about X?" invites discussion)
✓ Praise good work, not just problems
✓ Be fast — a PR blocked for two days kills momentum
✓ Approve when it's good enough, not perfect
```

## Konkretan primjer

Umjesto "Ovo je pogrešno, koristi mapu," napišite: "nit: mapa bi ovdje izbjegle ugniježđenu petlju, vrijedi li? Nije blokirajuće." Ista točka, ali poučava, ostavlja mjesta za nesuglasicu i ne blokira spajanje.

## Pazite na dinamiku moći

Kao TL vaši komentari imaju dodatnu težinu. Usputna "Ja bih to uradio drugačije" može zvučati kao naredba. Budite eksplicitni o tome što je pravi problem nasuprot preferenciji, i dopustite ljudima da se suprotstave.

## Zašto je to važno

Recenzije postavljaju standard kvalitete tima i njegov emocionalni ton.

Ostre recenzije čine ljude obranom i spore u slanju.

Dobre recenzije hvataće greške, šire znanje i čine inženjere boljima, što se pojačava tijekom vremena.