Comments on: Není revize kódu jako revize kódu http://blog.krecan.net/2008/08/18/neni-revize-kodu-jako-revize-kodu/ Short remarks from Java world Wed, 07 Jan 2009 02:00:00 +0000 http://wordpress.org/?v=2.7 hourly 1 By: Jan Kotek http://blog.krecan.net/2008/08/18/neni-revize-kodu-jako-revize-kodu/comment-page-1/#comment-549 Jan Kotek Fri, 29 Aug 2008 01:16:49 +0000 http://blog.krecan.net/?p=89#comment-549 Neni to hrozne formalni? Podle me staci projit nove commity a pripadne soukromne mail s cislou radku. Funguje to skvele :-) Neni to hrozne formalni? Podle me staci projit nove commity a pripadne soukromne mail s cislou radku. Funguje to skvele :-)

]]> By: Luboš Račanský http://blog.krecan.net/2008/08/18/neni-revize-kodu-jako-revize-kodu/comment-page-1/#comment-548 Luboš Račanský Mon, 25 Aug 2008 09:00:01 +0000 http://blog.krecan.net/?p=89#comment-548 Stejně jako platí rčení: "Nejsme tak bohatí, abychom si mohli kupovat levné věci." by se říct: "Nemáme tolik času, abychom mohli ignorovat unit testy a code review." Protože ono se nám to vymstí a onen zdánlivě ušetřený čas se nakonec několikanásobně prodraží. Stejně jako platí rčení: “Nejsme tak bohatí, abychom si mohli kupovat levné věci.” by se říct: “Nemáme tolik času, abychom mohli ignorovat unit testy a code review.” Protože ono se nám to vymstí a onen zdánlivě ušetřený čas se nakonec několikanásobně prodraží.

]]> By: Lukáš Křečan http://blog.krecan.net/2008/08/18/neni-revize-kodu-jako-revize-kodu/comment-page-1/#comment-546 Lukáš Křečan Tue, 19 Aug 2008 09:08:46 +0000 http://blog.krecan.net/?p=89#comment-546 Proto jsem psal, že by ten kód měl být rozumně velký. Doba sezení by se měla pohybovat mezi jednou až dvěma hodinami. A znova opakuji, podle mého názoru je hledání defektů jenom taková záminka jak si popovídat o kódu a něco se naučit. Proto jsem psal, že by ten kód měl být rozumně velký. Doba sezení by se měla pohybovat mezi jednou až dvěma hodinami.
A znova opakuji, podle mého názoru je hledání defektů jenom taková záminka jak si popovídat o kódu a něco se naučit.

]]> By: tommy http://blog.krecan.net/2008/08/18/neni-revize-kodu-jako-revize-kodu/comment-page-1/#comment-545 tommy Tue, 19 Aug 2008 08:36:15 +0000 http://blog.krecan.net/?p=89#comment-545 Dle mych zkusenosti code review zabere cca 50% casu (a vice) co trvalo kod napsat (v pripade ze kod napsal jeden, review provadi 3). Je to vec prospesna a uzitecna, ale casove narocna a unavujici. Jen tezko udrzite po celou dobu (napr. 4 hodiny) plnou pozornost vsech zucastnenych. Caste prestavky take nejsou prilis efektivni. Kolac je prima, ale dovedu si predtavit jak po 2 hodinach zucasteni zacnou premyslet zda se soustredit na kolac ci na reviewovany kod :-) Dle mych zkusenosti code review zabere cca 50% casu (a vice) co trvalo kod napsat (v pripade ze kod napsal jeden, review provadi 3). Je to vec prospesna a uzitecna, ale casove narocna a unavujici. Jen tezko udrzite po celou dobu (napr. 4 hodiny) plnou pozornost vsech zucastnenych. Caste prestavky take nejsou prilis efektivni. Kolac je prima, ale dovedu si predtavit jak po 2 hodinach zucasteni zacnou premyslet zda se soustredit na kolac ci na reviewovany kod :-)

]]> By: Lukas http://blog.krecan.net/2008/08/18/neni-revize-kodu-jako-revize-kodu/comment-page-1/#comment-544 Lukas Tue, 19 Aug 2008 08:18:17 +0000 http://blog.krecan.net/?p=89#comment-544 Code Review zrout casu nejsou, naopak je to velmi efektivni zpusob jak najit defekty v kodu. Code Review zrout casu nejsou, naopak je to velmi efektivni zpusob jak najit defekty v kodu.

]]> By: Dagi http://blog.krecan.net/2008/08/18/neni-revize-kodu-jako-revize-kodu/comment-page-1/#comment-543 Dagi Tue, 19 Aug 2008 07:27:07 +0000 http://blog.krecan.net/?p=89#comment-543 Lukasi diky za tip, ten kolac zkusim nejak zaridit ;-). Lukasi diky za tip, ten kolac zkusim nejak zaridit ;-).

]]> By: Honza Novotný http://blog.krecan.net/2008/08/18/neni-revize-kodu-jako-revize-kodu/comment-page-1/#comment-542 Honza Novotný Tue, 19 Aug 2008 06:05:06 +0000 http://blog.krecan.net/?p=89#comment-542 Moc hezky shrnuto. Bohužel v současné době např. u nás děláme review pouze nad veřejným API a hrubou architekturou knihoven. Na revize vnitřního kódu prostě zatím není ve většině případů čas. Obvykle se spoléháme na to, že ze zkušenosti víme, že kolegové prostě píší dobrý kód. Jiná situace je s nováčky v týmu. Tam jsou review v podstatě nevyhnutelné a potvrzuji, že je to neuvěřitelný žrout času. Moc hezky shrnuto.

Bohužel v současné době např. u nás děláme review pouze nad veřejným API a hrubou architekturou knihoven. Na revize vnitřního kódu prostě zatím není ve většině případů čas. Obvykle se spoléháme na to, že ze zkušenosti víme, že kolegové prostě píší dobrý kód.

Jiná situace je s nováčky v týmu. Tam jsou review v podstatě nevyhnutelné a potvrzuji, že je to neuvěřitelný žrout času.

]]>