Damir Širola has told us why, when and how to use code review to increase your codebase quality and drastically reduce number of software defects. This is presentation in Croatian from our weekly educational session.

1. A brief introduction to
Code Review
Damir Širola

2. Code review
 Prolazak kroz programski kod u svrhu otkrivanja defekata u softveru
 Formalan
 Detaljno isplaniran postupak obavljanja code review-a
 Obično se obavlja na sastancima na kojima sudjeluje nekoliko developera
 Obavlja se jako detaljno
 Lightweight
 Zahtjeva manje vremena od formalnog
 Koristi se kao dio procesa razvoja softvera

3. Code review - lightweight
 Preko ramena
 Putem emaila
 Programiranje u paru
 Potpomognuto alatom

4. Code review - prednosti
 Automatizirano testiranje otkriva u prosjeku
 Unit testovi – 25% defekata
 Funkcionalno testiranje – 35% defekata
 Integracijsko testiranje – 45% defekata
 Code review prosječno otkriva 60% defekata
 Dijeljenje znanja i napredak developera
 Olakšava mentoriranje novih developera
 Potiče raspravu o kodu
 Defekti se otkrivaju u ranoj fazi razvoja softvera
 Developeri pišu bolji kod kada znaju da će ga netko drugi pregledati

5. Code review - nedostaci
 Uzima dosta vremena
 Može prouzrokovati konflikte unutar tima

6. Best practices
 Broj linija koda trebao bi biti manji od 200 po code review-u i nikako ne
prelaziti 400
 Autor koda trebao bi napraviti code review nad svojim kodom prije nego
ga pošalje drugima na code review
 Trajanje bi trebalo biti manje od 60 minuta

7. Alati za code review
 Ugrađeno u VS2012 i noviji (samo za TFS)
 Github (Git)
 Gerrit (Git)
 Review Board (Git, Mercurial, Subversion, Bazaar, CVS)
 Upsource (Git)