A brief introduction to Code ReviewDamir Širola

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

Code review - lightweight

Preko ramena

Putem emaila

Programiranje u paru

Potpomognuto alatom

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

Code review - nedostaci

Uzima dosta vremena Može prouzrokovati konflikte unutar tima

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

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)