RSS Feed LinkedIn Google Plus

Kód felülvizsgálat, ismételten

2012. február 17. péntek

A Budapest.js meetup után beszélgettem az egyik kollégával, mondta, hogy náluk is egy csomó emberen kell, hogy keresztülmenjen az elkészített kód. A programozó megírja, átnézi, utána egy másik fejlesztő jóváhagyja, majd jön a team leader, QA… Szeretik is a GitHubot (privát projektre) használni, hiszen a soronkénti véleményezés lehetősége jelentősen megkönnyíti a kommunikációt. És olyan hozzászólások szoktak lenni, hogy lemaradt a dokumentáció, egy optimálisabb algoritmust kellett volna választani, miért pont így csináltad, stb.

Mint én is írtam korábban róla, a code review a napi fejlesztési folyamat része kell, hogy legyen. A felhasználói tesztek az alkalmazásunknak csupán a külső felületét tudják elérni, de a belső logikát nem látják. Ha ezt a lépést kihagyjuk, az olyan, mintha laikusként mennénk el autót vásárolni: szépen le van fényezve, jó a hangja, mozog a kormány: mehet a release. Hogy meg kellett volna nézetni egy másik fejlesztővel (szakszerviz)? Bocs, erre most nincs erőforrás… Lehet, hogy bejön. Lehet, hogy az első bugreportnál az engine-t refaktorálni kell, vagy a szétrozsdált framework miatt full rewrite-ra lesz szükség. Bevállalod?


Nincs hozzászólás

Vélemény?

Bocsánat, de a bejegyzéshez egyelőre nem engedélyezett a hozzászólás.