Siz ve cevap verenlerin çoğu, iki meslektaş arasında bu bir iletişim sorununa yaklaşıyor, ancak ben öyle olduğunu sanmıyorum. Tanımladığınız şey, her şeyden çok , korkunç derecede bozuk bir kod inceleme sürecine benziyor .
İlk önce, meslektaşınızın ikinci sırada olduğunu ve kodunuzu gözden geçirmesi beklenir. Bu açıkça yanlış. Tanım olarak, eş kod incelemeleri hiyerarşik değildir ve kesinlikle sadece kusurları bulmakla ilgili değildir. Ayrıca, öğrenme deneyimleri (katılan herkes için), sosyal etkileşim için bir fırsat sunabilir ve ortak kod sahipliği oluşturmak için değerli bir araç olduğunu kanıtlayabilirler. Ayrıca zaman zaman kodunu gözden geçirmeli, ondan bir şeyler öğrenmeli ve yanlış olduğu zaman onu düzeltmelisin ( her seferinde kimse onu doğru bulmaz ).
Ayrıca, meslektaşınızın derhal değişiklikler yaptığını söylersiniz. Bu da yanlış, ama elbette zaten biliyorsun; Gung ho yaklaşımının bir sorun olmadığını, bu soruyu sormazdın. Ancak yanlış yerde bir çözüm aradığınızı düşünüyorum. Dürüst olmak gerekirse, meslektaşınız bana biraz ... bana hatırlatıyor ve benzer durumlarda benim için çalışan iyi tanımlanmış ve sağlam bir inceleme süreci ve bir dizi harika araç oldu. İş arkadaşınızdan kodunuzu incelemesini durdurmak istemezsiniz ve her küçük değişiklik işe yaramayacaksa ondan önce sizinle konuşmasını isteyin. Bir süre olabilir, ancak yakında çok sinirleneceği bir noktaya ulaşacak ve başladığınız yere geri döneceksiniz ya da daha da kötüsü: sadece kodunuzu incelemeyi bırakacak.
Buradaki bir çözümün anahtarı, bir eş kod inceleme aracı olabilir. Genellikle ürün önerilerinden kaçınırım ancak kod incelemeleri için Atlassian's Cruciblegerçekten bir hayat kurtarıcı. Yaptıkları çok basit görünebilir ve öyle, ancak bu şaşırtıcı bir şekilde harika olmadığı anlamına gelmiyor. Deponuza bağlanır ve size tek tek değişiklik setlerini, dosyaları veya dosya gruplarını inceleme fırsatı sunar. Herhangi bir kodu değiştiremezsiniz, bunun yerine doğru hissetmeyen her şey hakkında yorum yaparsınız. Ve kesinlikle başka birinin kodunu değiştirmeniz gerekiyorsa, değişikliklerinizi açıklayan değişiklik setiyle ilgili bir yorum bırakabilirsiniz. Daha fazla ayrıntı istiyorsanız, Crucible'ın ürün sayfasındaki tanıtım videosu izlemeye değer. Potanın fiyatı herkes için değildir, ancak serbestçe temin edilebilecek çok sayıda akran değerlendirme aracı vardır. Çalıştığım ve keyif aldığım biri Review Board ve basit bir Google aramasıyla başkalarını bulacağınızdan eminim.
Hangi aracı seçerseniz seçin, sürecinizi tamamen değiştirecektir. Durmanıza, sandalyenizden çıkmanıza, diğer kişiyi rahatsız etmenize ve değişiklikleri tartışmanıza gerek yok; Yapmanız gereken tek şey, her hafta biraz zaman ayırıp yorumları incelemektir (haftada bir öneridir. Programınızı ve günlük rutini benden daha iyi bilirsiniz). Daha da önemlisi çekirdek incelemeler bir yerde bir veritabanında saklanır ve istediğiniz zaman geri alabilirsiniz. Su soğutucusu etrafında geçici tartışmalar yapmıyorlar. Eski incelemelerde en sevdiğim kullanım durumu, kod tabanımıza yeni bir ekip üyesi tanıtırken. Tam olarak nerede sıkışıp kaldığımızı, farklı fikirlerin olduğunu vb. Belirten kod tabanında yeni birisine yürüyebildiğim zaman her zaman güzeldir.
Devam edersek, bu meslektaşın kodunu her zaman okunabilir bulmadığınızı söylersiniz. Bu, ortak bir kodlama standardı dizinizin olmadığını ve bu kötü bir şey olduğunun haberini veriyor. Yine buna bir halk sorunu olarak yaklaşabilir ya da buna bir süreç sorunu olarak yaklaşabilirsiniz ve yine ikincisini şiddetle tavsiye ederim. Takımınızı bir araya getirin ve en kısa sürede ortak bir kodlama stili ve standartlar dizisi benimseyin. Gelişim ekosisteminizde ortak olan bir dizi standart seçmeniz ya da kendinizle gelmeniz önemli değil. Asıl mesele, standartlarınızın tutarlı olması ve onlara bağlı kalmanızdır. Çok sayıda araç var ve size yardımcı olabilir, ama bu tamamen farklı bir tartışma. Sadece başlaman için Yapılması gereken çok basit bir şey, kodunuzda bir çeşit stil formatlayıcıyı işleyen bir kancaya sahip olmaktır. İstediğiniz halde kodunuzu yazmaya devam edebilirsiniz ve aracın başkaları görmeden önce otomatik olarak "düzeltmesine" izin verin.
Son olarak , yorumda , yönetimin kişisel gelişim dallarının gerekli olduğuna inanmadığını belirtiyorsunuz. Onlara "dev dalları" dememizin bir nedeni var, "yönetim dalları" değil. Kafamda oluşan rantun dışarı çıkması için bir neden olmadığı için burada duracağım.
Bütün bunlar, meslektaşınızın burada hatalı olduğunu düşündüğümden emin değilim. Bu benim amacım değil, benim açımdan tüm gelişim sürecinizin de hatalı olduğu ve düzeltilmesi daha kolay olan bir şey olduğu. Kendinizi uygun araçlarla donatın, sayısız resmi ve gayrı resmi süreci keşfedin ve ekibinize uygun olanları seçin. Yakında “insan problemlerinin” artık varolmadığının farkına varacağınız bir noktaya varacaksınız. Ve lütfen "kendimiz dahil) kimseyi dinlemeyin" biz küçük bir ekibiz, tüm bunlara ihtiyacımız yok "mazereti. Yetkili geliştiricilerden oluşan bir ekip, gerekli araçları bir haftadan daha az bir sürede hazırlayabilir, otomatikleştirilebilecek her şeyi otomatikleştirebilir ve bir daha asla geriye dönüp bakmaz.
PS. "Kod sahipliği", sürekli tartışılan, terimsiz bir terimdir ve farklı insanlar için farklı anlamlara gelir. C2 hakkındaki farklı (ve bazen de antitetik) görüşlerin çoğunun mükemmel bir koleksiyonunu bulabilirsiniz .