Kod incelemesi fikrinden hoşlanmayan biriyle nasıl baş edilir?

Açıkçası, eğer yönetim kod incelemeleriyle zaman harcarsa, o zaman herkesin yapması gerekir.

Ancak her zaman varlıklarının her onsuyla direnen bu adamlar (veya kızlar) vardır.

Bu senaryo ile baş etmeyi meslektaş yorumcusu olarak ele alırken nasıl etkili bir şekilde yönetirsiniz?

Korku yüzünden direniyor . Bu şartlanma , çocukken, okulda, işte veya hatta şu anki ekibinizde gözden geçirme konusundaki kötü deneyimlerin bir sonucu olabilir. Modern toplumlarımızda, birisinin çalışma çıktısını bir insan olarak değeri ile karıştırmamız çok yaygın. İşte bu yüzden iş yerindeki eleştiriler iyi algılanmıyor. Bu yüzden, halk arasında en yaygın fobilerden birinde (yargılama korkusu) konuşmak da bu yüzdendir.

Bu tür davranışlardan kaçınmak için biraz psikolojiye ihtiyacınız olacak. Kertenkelesinin beynine , incelemeleri kodlaması için onu duyarsızlaştırarak (o yargılanmayacak, aşağılanmayacak, öldürülmeyecek, hiçbir şey olmayacak) ispat etmelisin .

Birinin engellemesini kaldırdığım en etkili yöntemlerden biri, kodunu gözden geçirmeden önce kodunuzu incelemesini istemek.

Bir süre sonra teklif ondan öğrenmek için onun kodunu okumasını ve neden önermek iyileştirmeler. Değiştirecek bir şey bulduğunuzda yazdıklarınıza dikkat edin. Korkacak bir şey olmadığını anlayacak ve yalnızca inceleme sürecinin olumlu bir kısmını üstlenecek: bilgisini öğrenmek ve artırmak .

Çiftler halinde çalışmayı denerdim - fikirden nefret eden birini, hoşuna giden birisiyle ekip edin ve birkaç hafta boyunca birbirlerinin kodlarını gözden geçirmelerini isteyin. Açıkçası, bu yardım edebilir veya etmeyebilir, ancak incelemenin her iki ucunda olmak, en azından sürecin daha ayrıntılı bir görüntüsünü verecektir. Bir çift çalışmanın birlikte olması, birbirlerinin tarzına ve genel hatalarına aşina olmalarını sağlayacak ve lastik damga yerine, birbirlerinin daha iyi olmalarına yardımcı olmaları için zaman tanıyacaktır. Bu aynı zamanda çalışma ortamınızda çift programlamayı teşvik etmenize yardımcı olabilir, çünkü sadece gözden geçirmek için değil, sıfırdan planlamak ve hatta planlamak ve kodlamak için artan bir eğilim görebileceğinizi düşünüyorum.

İlgisiz partiler denemek istedikleri sürece, bu yardımcı olabilir. Eğer düşünmeyi reddederlerse, takımda oldukları sürece bu konuda yapabileceğiniz pek bir şey yoktur.

@ Pierre'in cevabı, kod incelemesinden korkan biri için doğru yolda. Başka bir durum hayal edebiliyorum. Kod incelemesi yapan bir yıldız programcısı zaman kaybıdır, çünkü kod kabul edilebilir bir kalite ve doğruluk standardına ulaşır. Bu durumda bir kod incelemesinin zaman kaybı ve cadı avı olduğunu düşünebilirler. (Bu, hiçbiri olmadığında bir problem araştırmasıdır.)

Bu durumda, gözden geçirmenin amacını yeniden yönlendiririm. Kod incelemesinde kodda "problemler" bulma ile ilgili olmak yerine, hedefleri veya olası gelecekteki geliştirmeleri veya ek tasarım özelliklerini yeniden değerlendirmek için bir arama olarak düşünün. Bu şekilde, hem kodlayıcı hem de hakem bu sürece katılır ve umarım bu kodlayıcı, zamanın iyi bir şekilde kullanılmaya başlandığı gibi hissedecektir.

Açıkçası, iyi yönetilen bir mağazanız varsa, bu sorunun bir anlamı yoktur:

1) Eğer işin bir parçasıysa, yapmalısınız, ya da asılsızsınızdır. Yapmaları gereken işin bir bölümünü yapmayı kesinlikle reddeden biri konserve almalıdır. Programlama bir zanaat ve bir meslektir - yorumcular ve yöneticiler şımarık çocuklarla uğraşmamak için (her yaşta) işin yapılmasına yardımcı olmak için oradadır.

2) Eğer iyi yönetilen bir kaynak kontrol sisteminiz varsa (ki bu herhangi bir profesyonel yazılım mağazasında olmazsa olmaz), kodları beğenip beğenmeyecekleri incelenebilir. Bu yüzden onların kodunu gözden geçirin:

• Eğer iyiyse, onları bilgilendirin ve onlara arkalarına birer pat verin - bu da katılımı teşvik edecektir.

• Eğer iyi değilse, bilmelerini sağla. Bunun, kendilerini savunmak için onları katılmaya motive etme etkisi olmalı. Olmazsa, cezai önlemleri kullanabilirsiniz: Mali cezalar, statüdeki düşkünlükler, vb. Çalışmanıza rağmen, bu çalışan gelemezse, IMO kötü bir çalışanınız olur ve kapı gösterilmeli.

Kod incelemelerinin uygun şekilde yapılmadığı yerlerde bazı olumsuz deneyimler var mı? Yasal kaygıları olabilir.

Tatbikat için kesinlikle bir hak görmüyorlarsa, sabırlı olmalarını isteyin ve sonuç olarak kodlarına ve özellikle diğerlerinin (eğer mükemmel olduğunu düşünüyorlarsa) ne olacağını görün.

Kod Gözden Geçirme 'gelişmeyi iyileştirmeli', ancak gerçekte çalışan bir sisteminiz olana kadar neden birileri bunu yapmak istesin ki?

Şahsen, nüfusun% 100'üyle kazanamayacağımız bazı kavgalar olduğunu düşünüyorum.

Çift programlamanın neden birileri bunu yapmaya zorlandığında işe yaramayacağına dair yeterli sebep görebiliyorum.

Ancak kod incelemeleri farklıdır - çalışma alışkanlıklarınızı değil, verimliliğinizi değiştirir.

Yönetim, üretkenlik nedeniyle direnci azaltmak için birkaç şey yapabilir: 1) Tüm geliştiriciler için hızdaki düşüşü kabul edin. 2) Gözden geçirme döngüleri nedeniyle (örneğin, geliştiricilerin yerel bir depoya sahip olmalarını sağlamak için) birden fazla sürümün yönetimi ve birleştirilmesi ile ilgili uygun araçları sağlayın. yorum.

Bunu yaparlarsa, herkesin katılmasını istemek meşru, IMHO. Şimdi çalıştığım şirket bunu global olarak zorlar - bir sahibinin onayı olmadan teslim edemezsiniz. Bu da işleri yavaşlatırken, birçok kazayı da engelliyor.

Kod incelemesini zorunlu hale getirmek için teknik önlemler kullandık.

Kod incelememizi tanıttığımız yol, kaynak kontrolümüzde, bir başkası tarafından, daha sonra onu basan tarafından imzalanmayan kodu birleştirmek imkansız olmasıdır.

Onları kov

Bu kadar basit - yalnız bir proje alırlar ya da gitmek zorunda kalırlar. Onları ekibinizden uzaklaştırın. Sadece rollerini yapmakla kalmıyorlar, takım moralini ve uygulamalarını da yıpratıyorlar.

Şimdi, ekibinizin% 50'sini kovmanız gerekiyorsa, o zaman ...

Anlama

Neden bazıları reddediyor? Zamanları yok mu? Yanmışlar mı? Deneyimleri hiç yaşamadıkları bir şey hakkında mı? Bunun bir zaman kaybı olduğunu düşünüyorlar mı, öyleyse neden?

Çevik metodoloji bu konuda yardımcı olacaktır - silolara karşı sürekli çalıştığınızı (yani otobüs faktörünü azaltmak için) ve ekibinizdeki kişilerin başkalarının yaptıklarına karıştığını farz ediyorum.

Bireysel birleştirme isteklerinin oldukça küçük olmasını sağlamak için çalışın. 1'den fazla değişiklik ekranıysa, ne yapıldığını açıklamak için bir stand-up veya şimşek konuşması gerekir. 10 sayfa ise, slaytlar ve mimari şemaları ile bir sunum yapması gerekir.

Söz konusu herkes aynı proje üzerinde çalışıyor mu?

Proje zaten teknik borç dağının altına gömüldü mü?

Projeye ve sürekli gelişime inanıyorlar mı?