Aggódnom kéne, vagy ez természetes? (munkahely)

Ez már csak ilyen. Nem létezik olyan programozó, aki elsőre tökéletes és bugmentes kódot ír. A projekt szerves része a tesztelés, ha az nincs meg, akkor az nagy hiba. De ha a hibáid is kijavítod, akkor nem lesz olyan, hogy emiatt elküldenek. :) Majd belejössz és kevesebb hibát hagysz a kódban, vagy jobban megérted a követelményeket, amikor már ismered a kollégákat, mindenki így kezdte ne aggódj. :)

A review célja a fejlődés segítése, nem a seggnyalás. Abból semmit nem tanulnál.

Szerintem ez a code review egyszerre hasznos és fontos. A volt munkahelyemen nem volt ilyen. Voltak jó programozóink és volt rengeteg kezdő, illetve medior. Senki kódjának a minősége nem lett átnézve, senkire nem lett rászólva, ha hanyag kódot rakott fel a repóba. A kezdők ugyan látszólag jól megírtak egy-egy modult, az adott problémát megoldotta a kódjuk, viszont egyikük sem gondolt, kezdők lévén a jövőtállóságra és újrafelhasználhatóságra. Így aztán rendszeresen abba a problémába futottunk, hogy ahogy nőtt a projekt és ahogy egyre komplexebb funkciókat kellett hozzáadni, a kezdők által írt modulok rendre elvéreztek, mivel nem bizonyultak eléggé flexibilisnek. Ezeket általában át kellett írni. Ha egy modul sok más helyen volt használva, akkor nehezebb volt átírni, mert általában a hívás is változott, mivel sokszor arra sem gondoltak, hogy hány különböző helyen lesznek ezek használva, és hányféle interfészt kell nekik biztosítani, valamint egy adott interfészt hogy lehet legjobban definiálni. Paraméterhegyek voltak mindig :D Ezért fontos a code reivew. Én nagyon örültem volna, ha anno az én kezdő kódomat valaki átrágja, és elmondja, mit csináltam helytelenül, mert én is belekeveredtem az előzőleg leírt hibába. :D Volt egy tök egyszerű projektem, ami valami borzalmasra duzzadt az ismétlődések miatt. Kellett egy egész év, mire megtanultam tiszta, átgondolt és logikus programot írni, ami mások számára is érthető, bővíthető és jövőtálló. Szerencsémre épp át kellett mennem Java-ra, ahol ezek a felsoroltak elengedhetetlenek startból, szóval nekem ez tökéletesen jött ki. Azóta az ott kondenzált tudást akámilyen más nyelvre tudom alkalmazni.