Kendime bir süperstar dev demezdim, ama nispeten deneyimli biri. Kod kalitesini yüksek bir seviyede tutmaya çalışıyorum ve kodlama stilimde her zaman iyileştirmeler yapmak, kodu verimli, okunaklı ve tutarlı hale getirmek ve ekibi tutarlılık sağlamak için bir kalıp ve metodoloji izlemeye teşvik etmek için çalışıyorum. Hem kalite hem de hız arasındaki dengenin gerekliliğini de anlıyorum.
Bunu başarmak için ekibime akran değerlendirmesi kavramını getirdim. İki başparmak yukarıya github çekme-birleştirme isteği. Harika - ama bence hiccoughs olmadan.
Sık sık benzer meslektaşlarımdan gelen meslektaş yorumları yorumlarını görüyorum -
- Sonra bir boşluk eklemek iyi olurdu
<INSERT SOMETHING HERE>
- Yöntemler arasında istenmeyen fazladan çizgi
- Tam durak, dokümanlardaki yorumların sonunda kullanılmalıdır.
Şimdi benim açımdan - gözden geçiren yüzeysel olarak kod estetiğine bakıyor - ve gerçekten bir kod incelemesi yapmıyor. Kozmetik kod incelemesi, kibirli / seçkin zihniyet olarak karşımıza çıkıyor. Maddeden yoksun, ancak gözden geçiren teknik olarak doğru olduğu için onunla fazla tartışamazsınız . Yukarıdaki türden incelemelerin daha az olduğunu ve aşağıdaki gibi daha fazla değerlendirme görmeyi tercih ederim:
- Döngüsel karmaşıklığı azaltabilirsiniz ...
- Erken çıkın ve / başkasından kaçının
- DB sorgunuzu bir depoya özetleyin
- Bu mantık gerçekten buraya ait değil
- Kendini tekrar etme - özetle ve tekrar kullan
X
Yönteme argüman olarak geçildiyse ne olurY
?- Bunun için birim testi nerede?
Ben her zaman kozmetik türden yorum yapanlarla aynı türde olduğunu düşünüyorum ve bence "Kalite ve Mantık Temelli" meslektaş değerlendirmelerini veren insanlarla aynı.
Akran incelemesine doğru yaklaşım (varsa). Ve aynı kişilerle temelde heceleme hataları ve estetik hatalar aramak yerine kodun üzerinden dolanıp aynı insanlarla sinirli olma konusunda haklı mıyım?
Eğer haklıysam - meslektaşları, kozmetik rötuşları önererek dengede bulunan hataları aramaya teşvik etmeye ne dersim?
Eğer hatalıysam - lütfen beni aydınlat. İyi bir kod incelemesini oluşturan şey için kurallar var mı? Kod incelemelerinin ne olduğu noktasını kaçırdım mı?
Benim bakış açıma göre - kod incelemesi kodun ortak sorumluluğudur. Mantık, okunabilirlik ve işlevselliği ele almadan / kontrol etmeden başparmak kodunu vermekte rahat hissetmem. Birinin bir doktora bloğunda tam olarak durduğunu fark edersem, katı bir kod parçası için bir birleştirme engellemekten de rahatsız olmazdım.
Kodları incelerken, her 500 Loc için 15-45 dakika harcıyorum. Yaptıkları inceleme derinliği buysa, bu sığ incelemelerin 10 dakikadan daha uzun sürdüğünü hayal edemiyorum. Dahası, sığ gözden geçiriciden elde edilen başparmak değeri ne kadardır? Elbette bu, tüm başparmakların eşit ağırlıkta olmadığı ve belki de 2 geçişli bir inceleme süreci olması gerektiği anlamına gelir. Derin incelemeler için bir başparmak ve "parlatma" için 2. bir başparmak?