Jak funguje code review v Seznam.cz a proč se to neučí na školách

Code review je jedna z nejdůležitějších praktik v moderním vývoji. A přesto se na žádné technické univerzitě, kterou jsem zažil, neučí.

Jak to funguje v Seznam.cz

V Seznam.cz máme nastavený proces, kdy každá změna kódu projde alespoň jedním review od kolegy. Nemusí to být senior, stačí kdokoliv z týmu. Cíl je jednoduchý - chytit chyby dřív, než se dostanou do produkce.

Co se hodnotí

Při review se dívám na čitelnost kódu, na logiku, na možné edge cases a na bezpečnost. Nehledám pravopisné chyby. Hledám problémy, které by mohly způsobit bug v produkci. Důležité je také to, jestli je kód srozumitelný pro ostatní členy týmu.

Proč je to důležité

Jeden programátor nikdy neudělá tak dobrý kód jako dva. Když někdo jiný čte váš kód, vidí věci, které vy přehlížíte. Je to stejné jako korektura textu - autor vždycky přehlédne své chyby.

Proč se to neučí na školách

Na škole píšete kód sám. Odevzdáte ho a dostanete známku. Nikdo vám neřekne, proč je váš kód špatně čitelný, proč máte špatně pojmenované proměnné nebo proč vaše řešení je příliš složité. A právě tohle vám code review dá - zpětnou vazbu.

Jak se naučit dělat review

Začněte tím, že si budete číst kód ostatních. Přispívejte do open source projektů. Když vidíte cizí kód, učíte se, jak různí lidé řeší stejné problémy. A pak začněte psát review ke kódu svých kolegů. Není to o tom být chytřejší. Je to o tom dívat se na kód zvenku.

Rada pro juniory

Nebojte se code review. Není to útok na váš kód. Je to příležitost se naučit. A čím dřív si na to zvyknete, tím rychleji porostete jako vývojáři.