code review

• (bkz: kod inceleme)
• kod inceleme.
  genelde master/production kodunun kostugu branch’e merge yapilmadan ortaligi yikip dokecek bi sey var mi diye kodunuz bi baskasi tarafindan incelenir.
  
  eger bi ruh hastasi kodunuzu inceliyosa isimlendirmeye kadar takilabilir. neden camel case yaptin da neden underscore kullanmadin tartismalarina girmek isterken kafasini duvarlarda parcalamak isterken kendinizi bulabilirsiniz.
  
  ben code review yaparken branch’i locale cekip test ediyorum, bi bakiyorum abukluk var mi, yoksa cok basit abi: lgtm yazip geciyorum.
  
  ama ruh hastalari icin bunu yapmak cok zor. ariza cikarmak icin yanip tutusan bunyelerde code review adeta bi govde gosterisi yeri. isin mi yok abi ya.. git hayati yasa mk, yemek yap/ye/seyahat planla ya da git sanat eserleriyle ilgili bi belgesel ac izle. benim tanimladigim degiskene bu kadar kafani takip, bu kadar koyu koyu savunacak haldeysen, kafana takmak icin yanlis seyi secmissin hayatta. uzuyosun beni :(
• görsel
  
  görsel
  
  hesabin statu kontrolu yapilmamis, return'den once bosluk gerek. ayrica alttaki iki method da util class'i yapilabilir. bir de 30. satirda null check gerekli.
• kendi kendine çok iyi kod yazanların bir işe girdiklerinde karşılaşacakları aktivite. ee serbest serbest takılıyordun, karışan yoktu şimdi yazdığın kodu okuyup eleştiren birileri var. code review harika bir şeydir, ortaya çıkan yazılım main brancha gitmeden bir standarta kavuşur, bir kişinin değil bir ekibin elinden çıkmış hale evrilir. önemlidir, tartışma çıkabilir, birbirinin koduna daha dikkat kesilip açık arayanlar aslında projeye faydalıdır :))
• pair programmingin kardeşi. pp kadar sizi geliştirmez ama eğer iyi bir yazılım ekibinde çalışıyorsanız diğerlerinin kodlarını incelemek size bakış açısı katar, öğreticidir.
• oturup aylarca bir kültür oluşturmaya çalışırsın, binlerce satırda daha iyi nasıl isler yaparızı konuşur, güncel pratikleri değerlendirirsin. güzel sonuçlar beklersin ama sonuç olarak elin çocuğu gelir hiç umurunda olmadan seni kod incelemeye dahil etmez bütün pratikleri hiçe sayıp kafasına göre yazar geçer.
• ekip arkadaşlarının birbirine mobbing yapması için yegane ortam. hele ki hayriye hanım gibi aman ağzımızın tadı bozulmasın zihniyetinde bir yöneticiniz varsa seyleyin cümbüşü. devrecilik, mobbing vb gibi şeyler diğer mesleklerde de var ama başka hiç bir meslekte bu kadar profesyonel mobbing yapamazsınız. adam kankasının pr'ını yirmi dakikada commentsiz approve edip de sizin yarattığınız her değişken adına yorum yapar. öyle garip commentler girer ki aman hadi sussun yapayım geçeyim dersiniz 850 tane testi falan fixlersiniz. sonra 2 gün geçer paşa bey yeni commentler filan girer. eğer siz de bu pasif agresiflerden biriyseniz bak evde hatırlayıp hatırlayıp beddua ediyorum. adam olun lan. yapmayın böyle şeyler.