U svojoj petogodišnjoj karijeri u Amazonu, Curtis Einsmann, autor popularnog kursa 

, pregledao je preko 750 pull requestova. Kao tech lead bio je zadužen za pružanje kvalitetnog feedbacka i podizanje standarda kvaliteta koda. Evo njegovih savjeta.

Kao junior developer, bio sam baš loš u code review procesu. Nisam znao gdje da počnem, šta da tražim ili kako da komentarišem.  Pružao sam beskorisne komentare vezano za stil i formatiranje. Žurio sam kroz code review ostavljajući klasične “looks good” opaske nakon što bi propustio probleme velike kao kuća.  U nastavku slijede savjeti za bolji code review koji će vam pomoći da izbjegnete ove greške.

Kvalitet koda poboljšava njegovu čitljivost i razumijevanje sistema.To omogućava tvom timu da ubrza svoj rad na razvoju novih funkcionalnosti i donosi dugoročnu dobit.

Code review je takođe odlična prilika za uhodavanje drugih developera. Kvalitetan feedback ubrzava učenje, rast i razvoj svakog pojedinca.

Međutim, code review je i sjajna prilika za učenje za onog ko pregleda kod. Zašto? Zato što svaki developer rješava probleme na svoj način.  Pregledanjem njihovog koda, naučićeš nove strategije za rješavanje problema koje možeš iskoristiti kasnije.

Uzmi sebi vremena da naučiš principe programiranja

Poznavanje principa kao što su SOLID, DRY i KISS su temelj za dobar code review. Pročitaj knjigu Clean Code koja će ti dati ideju o tome kako dobar kod treba da izgleda. Odvoji vrijeme da razumiješ važnost odabira prave strukture podataka, ispravne apstrakcije, logičkog imenovanja i kontrole toka programa.

Bitno je upoznati se i sa kritikama na navedene principe i shvatiti da su to dobre smjernice a ne čvrsta pravila. Rade u određenim situacijama, ali ne baš u svim.

Najvažnija pitanja su:  da li ovaj kod radi? Da li je lak za održavanje? Hoćemo li ga svi razumjeti za dvije godine?

Dobar feedback traži vrijeme za izučavanje koda.  Code review zato treba raditi sporo i 100% razumjeti svaku liniju.

Pretvaraj se da postoji skrivena mana u kodu a da si ti detektiv.

Prelistaj pull request da pronađeš glavne dijelove implementacije.  Ovo bi mogao biti neki API endpoint ili refaktorisani data access layer.

Počni da čitaš.  Zapisuj komentare u hodu, planirajući da ih doradiš kasnije.

Nastavi da čitaš nasumično dok ti kod “ne klikne”

Prati funkcije i pozive. Skači kroz kod da vidiš kako se izmjena u PRu uklapa u širu sliku.  Pročitaj izvorni task da dobiješ kompletan konktest problema.

Kod treba pročitati tri,četiri, a nekada i pet puta da se formira jasna slika.

Smisli što više scenarija da kod pukne i provjeri da li su pokriveni

Ovo uradi tek kad postigneš dubinsko razumijevanje koda.  Popiši sve moguće edge caseve koji ti padnu na pamet i onda prođi kroz kod da vidiš da li su pokriveni.

Komentar treba da se odnosi na kod, nikada na osobu.  Izbjegavaj riječi poput “ti” ili “tvoje”. Ako je nešto sitnica, započni komentar sa “Sitnica”.  Formuliši većinu komentara kao pitanja ili prijedloge.

Ostavi bar jedan pozitivan komentar.  Ovo nikada ne treba zaboravljati.

Ohrabri kolegu da te kontaktira ako ima pitanja.

Komentare popisuj tokom iščitavanja koda, ali ih sredi tek nakon što potpuno razumiješ i više puta pročitaš kod. Nikada nećeš imati dobre komentare nakon prvog čitanja.

Izbjegavaj komentare bez obrazloženja.  Uvijek navedi:

Što se tiče formatiranja koda, prepusti to automatskim alatima.

Odobravaj PR kada je dovoljno dobar, a ne kada je savršen

Razumij da perfekcionizam nije vrlina, već mana. Odobravaj kod ako radi i ako je čitljiv.

Međutim, ovo ne znači da trebaš imati niske standarde.  Zatraži korekcije čak i ako je nešto sitnica koja može uticati kvalitet rada sistema ili buduće razumijevanje koda i očekuj od kolega da se prema tvojim PRovima odnose na isti način.

Teško je mjeriti kvalitet koda. Neke od metrika su broj problema u produkciji, brzina razvoja novuh funckionalnosti i brzina kojom novi članovi tima postaju produktivni.

Međutim, sve ih je teško mjeriti.  Takođe je teško mjeriti da li si odradio dobar code review.


Zato pitaj druge developere da li je tvoj code review koristan i od pomoći.  Iskoristi taj feedback da poboljšaš svoj budući rad i shvati da ćeš postati bolji što češće radiš code review.



i prevedena je i prilagođena uz odobrenje.

Pregledao sam preko 750 pull requestova u Amazonu.  Evo kako sam radio code review.

Rat u Ukrajini počeo je 24. februara 2022. godine i od tada su mediji i društvene mreže preplavljene ovom temom, uključujući IT zajednicu. Dok se svi nadamo što skorijem okončanju sukoba, nemoguće je ne zapitati se kako će se ovaj događaj bez presedana odraziti na stanje IT industrije u Istočnoj Evropi, a posebno u BiH.

Ukrajina je zemlja sa 44 miliona stanovnika i 

 u Evropi. Procjenjuje se da imaju oko 200.000 programera i 4.000 IT kompanija, a njih oko 40.000 radi u outsourcing sektoru. Ako vam kažemo da čak 100 od 

 kompanija u svijetu ima programere u Ukrajini, onda vam je jasno o kojem redu veličina govorimo.

U outsourcing segmentu, koji nas najviše i zanima, ističemo nekoliko Ukrajinskih firmi:

 je najveća tech kompanija u Ukrajini, koja zapošljava više od 10.000 ljudi od kojih su većina programeri. Imaju vrhunsku reputaciju i rade sa firmama kao što su Google, Expedia, Microsoft, Barclays, Oracle i UBS.

 preko 6.000 tech radnika, imaju i sjedište u Silikonskoj dolini i rade sa većim brojem  startupa iz tog područja.

: preko 3.500 radnika. Rade za Boeing, Dell, IBM i Deutsche Bank.

Šta se trenutno dešava sa ovim i drugim kompanijama koje outsourceaju radnike u Ukrajini? Prema 

 Gergely Orsoza, autora popularnog newslettera The Pragmatic Engineer,  kompanije su trenutno fokusirane na pružanje pomoći radnicima u vidu avansiranih plata i bonusa, uspostave sigurnih kuća ili evakuacije u neke od susjednih zemalja.

Primjera radi, Cimpress koji je izlistan na NASDAQu  ponudio je da evakuira svojih 500 radnika van zemlje ili u sigurniji zapadniji dio Ukrajine, izraelski Wix je ponudio da evakuira svojih 1.000 radnika u Tursku, a Fiverr se vodi istim razmišljanjem. Hopin je svim radnicima uplatio $3.000 kao prvu pomoć dok je Lemon.io avansirao plate za dva mjeseca. Trenutno je sav fokus na pomoć i evakuaciju, kako i treba biti.

Gledajući na trenutni razvoj situacije, možemo očekivati nekoliko paralelnih dešavanja:


. Bez obzira na to što su ratna zbivanja ograničena na Ukrajinu, Evropa će se sveukupno percipirati kao nestabilno područje i sve američke firme koje to budu mogle da priušte zapošljavaće primarno na svom terenu, što znači dalji rast plata u IT sektoru u SAD, Kanadi i Velikoj Britaniji.

Slabljenje evropskih i jačanje američkih IT firmi.  

Ukoliko sukob eskalira, Evropa će morati potražiti alternative za ruski gas. Skoro 40% gasa u EU se 

, a čak i jake zemlje poput Njemačke ne mogu lako podnijeti veću cijenu gasa. Isplativih opcija nema i procjenjuje se da bi građani mogli plaćati do tri puta skuplji gas, što će oslabiti evropsku ekonomiju i posljedično smanjiti moć 

Izbjegavanje outsourcinga u zemljama koje  graniče sa Ukrajinom. 

Za firme kojima outsourcing bude jedina isplativa opcija, Poljska i Rumunija, kao drugi i treći najveći outsourcing centar poslije Ukrajine, vjerovatno neće biti prvi izbor. Iako su u pitanju zemlje od 37 i 19 miliona stanovnika i članice su EU, obje se graniče sa Ukrajinom, a kapital se u pravilu fizički udaljava od bilo kakvog izvora nestabilnosti. Ovo će uključiti i manje zemlje, kao što su Slovačka i Moldavija, a naročito Bjelorusiju koja se percipira kao ruski partner.

Premještanje potražnje ka drugim zemljama u Evropi.

 Potražnja za IT kadrom će porasti u svim zemljama Evrope koje su udaljenije od Ukrajine, a najveći pritisak će biti upravo na zemljama čija je platežna moć slična Ukrajinskoj, u koje spada i BiH.

BiH, Srbija i Sjeverna Makedonija kao novi igrači

Ova posljednja tačka će lansirati BiH, Srbiju i Sjevernu Makedoniju kao nove ultra atraktivne outsourcing destinacije iz više razloga.

Naime, sve tri zemlje imaju prosječnu platu i isplativost dosta sličnu Ukrajinskoj. Iako je radnik iz Bosne 50% skuplji od onog iz Ukrajine, u Bosni i dalje možete zaposliti skoro dva radnika po cijeni jednog iz Hrvatske, što je dovoljan razlog da se obrati pažnju na nas. Iako statistike o prosječnoj plati nisu nužno reprezentativne za IT sektor, ukupno gledano i dalje je jeftinije zaposliti programera u Bosni, bez obzira na senioritet, a naročito ako se zapošljava kroz sistem SP/obrta.

Dalje, Bosna, Srbija i Sjeverna Makedonija imaju odličan geografski položaj i dobru uvezanost sa većim gradovima u Evropi u kojima međunarodne kompanije uobičajeno osnivaju hubove. Za razliku od Bugarske ili Turske, koje imaju 7 odnosno 8 sati razlike u odnosu na Njujork, Bosna ima razliku od samo 6 sati, što i dalje ostavlja dovoljno preklapanja u slučaju komunikacije sa SAD.

Problematika sa kojom će se kompanije susresti je skaliranje:  Bosna, Srbija i Sjeverna Makedonija skupa imaju tek 12 miliona stanovnika, što je nešto više od pola Rumunije. Ovo će neminovno dovesti do rasta cijene rada u IT sektoru i jagmom za programerima, što se osjetilo i prije Ukrajinske krize.

Vrlo je vjerovatno da u trenutku pisanja ovog teksta veće outsourcing kompanije u regiji, poput 

, već dobijaju mandate da zaposle dodatne programere i očuvaju postojeću radnu snagu, što će sigurno uraditi i svi drugi koji budu imali sredstva za takvo nešto.

Zbog velike potražnje, nije teško zamisliti ni scenario ukrupnjavanja u kojem će veće firme kupovati manje kao najefikasniji metod osiguravanja dominacije, naročito ako imamo u vidu da u Bosni i zemljama okruženja ima vrlo malo kompanija koje razvijaju svoje proizvode i većina IT sektora sastoji se od outsourcing firmi.

Za ljude zaposlene u ovom sektoru, ovo samo znači još veći broj ponuda i prilika za posao te izvjestan rast plata za sve pozicije, od QA do arhitekte.

Naravno, sve ovo su rane projekcije na bazi onoga što trenutno vidimo. Svijet nije i nikada više neće biti isti, a na nama je da pišemo kod - drugi neka pišu istoriju.

Blog

Strana 7

Pregledao sam preko 750 pull requestova u Amazonu. Evo kako sam radio code review.

Kako će kriza u Ukrajini uticati na IT industriju u BiH

Top IT poslovi u tvom inboxu

Top IT poslovi u tvom inboxu