Herkesin kod incelemelerinde rutin olarak 100'den fazla yorum alması yaygın mıdır? Ben demem. Kod kalitesi "arzulanan bir çok şey bırakan" insanlar için çok fazla yorum almak kesinlikle yaygın mıdır.
Ancak, kod inceleme sürecinin "kurallarına" da bağlıdır. HERKES, bir şeyin nasıl yapılması gerektiği konusunda kendi fikirlerine sahiptir. Kod inceleme işleminiz yorumların "Bu şekilde yapmalısınız" biçiminde olmasına izin veriyorsa, yeterli kod için bile çok sayıda yorum alırsınız. İşleminiz "kusur" bulmayı amaçlıyorsa, yorumların sayısı çok daha az olmalıdır.
Deneyimlerime göre, alternatif yöntemler için "öneri" sağlayan incelemeler zaman kaybıdır. Bu "öneriler" inceleme sürecinin dışında birer birer ele alınmalıdır. Hata incelemeleri, insanların "neden yaptığım gibi yapmadınız?" Yerine hatalara odaklanmasını sağladığı için daha kullanışlıdır. Aynı zamanda daha kullanışlıdır, çünkü birisi bulursa bir hata inkar edilmez. Böylece, acı hissi yok ama bunun yerine büyük bir şükran var.
GÜNCELLEME: Bütün bunlar söylendiğinde, bazı kodlar hatasız olsa bile sadece kötüdür. Bu durumda, inceleme yorumu, benzer bir şey söyleyen tek bir yorum olmalıdır. "Bu kodun temizlenmesi gerekiyor. Lütfen kod [adınız burada] ile tartışılana kadar incelemeyi erteleyin." Bu durumda, yorum düzeltilene kadar kodun daha fazla gözden geçirilmesi durmalıdır.
GÜNCELLEME2: @Kullanıcı: Kodunuzu / tasarımınızı geliştirirken bunlardan biriyle tartışıyor musunuz? Önerilerine dayanarak nasıl kod geliştirdiğinize veya yolunuzun iyi olduğunu düşünmeye devam ettiğiniz konusunda bir şey değiştiriyor musunuz? Yorumlarından bir şey öğreniyor musunuz?
Bir projede lider olduğumda, TÜM iş ürünlerinden sorumlu olmak benim işim. Bir iş ürününü onaylarsam, ürünün kabul edilebilir olduğunu iddia ediyorum. Kaliteli ürünler üretmek konusunda bir üne sahip olmak istiyorum. Böylece beklentilerim var ve tatmin edici olmaktan daha azını kabul etmeyeceğim. Aynı zamanda tercihlerimin nedenlerini öğretmeye ve açıklamaya çalışıyorum. Bu tercihler her zaman ideal olmayabilir (özellikle başkalarının gözünde), ancak bu tercihlerin çoğu deneyimlerden gelmiştir. Genellikle kötü olanları tekrar etmekten kaçınmak için bir reaksiyon. Bu nedenle, geri çevirmeden bağımsız olarak, onayımı almak için gerekli olan kişisel "sopalarım" dan birkaçı var.
Diğer yandan, iş ürünlerinizi onaylatmak için gerekli beklentileri öğrenmeniz gerekir. Katılmıyorsunuz, ancak aşırı yönetme yetkisine sahip olmadığınız görülüyor, o zaman ne beklendiğini öğrenin. Takımın sizi başarısızlığa düşürmeye çalıştığından şüpheliyim. Bu da onları kötü gösteriyor. Bu bağlamda, öğrenmeye hevesli olduğunuzu gösterin (söylemeseniz bile), söylediklerini alın ve tercihlerine uyum sağlamak için elinizden gelenin en iyisini yapın ve muhtemelen onları biraz geri görürsünüz. Belki en azından tahammül edebileceğinizi bulun ve size yollarını öğretmek için biraz el tutma yapıp yapamayacaklarını görün. Kim bilir, bu süreçte becerilerinizi bir sonraki seviyeye taşıyabilecek bir şey öğrenebilirsiniz.