Code review – Deo 1.

Par stvari koje ne želite da senior programer zatekne u vašem kodu:

• metoda koja vraća niz
• umetnuti (nested)  tipovi koji imaju public vidljivost
• metoda sa boolean parametrima (ako ih ima više korisiti enumeracije, a ako ima jdena napraviti dve metode)
• metoda sa više od 3 parametra
• metoda sa out parametrima
• metoda sa kodiranom povratnom vrednošću (npr. if ( Do() == 0) , već koristiti exceptions )
• iskomentarisan kod (npr. /* Console.Write(“Provera:” ,  bar()) */ )

I jedno zapažanje sa danasnjeg code review-a u firmi:

Kada sledeći put pritisnes Ctrl-C pa Ctrl-V budi siguran da praviš grešku.