Ekibe kötü kod yazanlar nasıl “etkisiz hale getirilir”?


9

Ben her zaman JoelOnSoftware "Sadece bir homurdanma işlerin yapılması" adlı bu makaleyi sevdim . Özellikle acemi olduğumda ilişki kurabilirim (ve yine de HER ZAMAN biri olacağımı hissediyorum).

Yaklaşık # 4, bozoları nötralize ediyor. Bunu gerçek işyerinde uygulamak için ne gibi tavsiyeleriniz var? Birisinin kötü koduna karşı bir hata kaydetmek kadar kolay değil (en azından ekibimizde). Orada kalan herkes için ne işe yarar?


1
Silahlar. Çoğu.
CodesInChaos

Yanıtlar:


9

Kalıcı değerlendirme.

Her günün sonunda, yazdıklarını gözden geçirmek için 30 dakika harcayın. Eğer yanlış bir şey yapmışlarsa, yeniden yazmalarını sağlayın.

Bunu yapmadığınız sürece, bir gün başvurunuzun bir kısmının, işi yapabilecek gibi görünse de, tamamen sürdürülemez, yanlış tasarlanmış ve gelecekte - hatta yarın çok fazla soruna neden olacağını fark edeceksiniz.

Daha az üretken hale getirecek olsalar da, iki kat daha büyük ama sürdürülemez böcek şişkinliğine karşı iyi bir kod üretmeleri yine de çok daha iyi olacaktır.


2
Mükemmel cevap. Eğer buna ekleyebilirsem, eğer bu kişi bir akransa, o zaman en iyi takım lideri tarafından yapılır. Bu şekilde, dükkanın hiyerarşisi tarafından komut verildiği takdirde, onun iyi-olmayan-tekrar-yanıtı çok daha etkili olacaktır.

1
@Surfer, tam tersi. Sen hale ekibi ne sevecen tarafından, daha iyi çözümler önererek, bu gibi şeyler yaparak bir takım lideri. Tam tersi değil. (Ancak, elbette, daha yüksek hiyerarşi düzeylerinden yardım almak yardımcı olur).
P

1
Öyleyse soru, kimin bunları yeniden yazmasını sağlama yetkisi var? Sanırım cevap, eğer problemler tüm takıma yayınlanırsa, tüm ekibin ahlaki otoritesidir.
C Johnson

Her ekip üyesi için sağlam bir kod inceleme mekanizması olmadığında bu uygundur. Bozo'nun, yeniden işlemesi söylenmeden önce gerçekten derin bir delik (sabitleme maliyetine değmez) kazmamasını sağlamak da uygundur.
mattnz

5

Kişi daha iyisini bilmiyor, ancak öğrenmek istiyorsa, bazı rehberlik ve kod incelemesi sağlayın. İyi koda maruz kaldıklarından emin olun.

Gerçekten fakir geliştiriciler, kötü yollarla kurulmuş ve yeni bir şey öğrenmekle savaşanlardır. Tek umudunuz, kendi karmaşasını veya daha basit şeylerini koruyarak acı çekmelerine izin vermektir. İdeal olarak, otorite sahibi biri ayağa kalkar ve uygunluk veya ayrılma der.


1
Tutum gerçekten önemlidir. Genellikle acemi daha mütevazı ve kod incelemelerine ve eleştirilerine açık buluyorum. Bu insanlarla konuşmak kolaydır. Ve onlarla zayıflıkları hakkında kolayca konuşabilirsiniz. Çalışmaları üzerine eleştirildiklerinde bir milyon parça cam gibi parçalanacak küstah gaziler.
C Johnson
Sitemizi kullandığınızda şunları okuyup anladığınızı kabul etmiş olursunuz: Çerez Politikası ve Gizlilik Politikası.
Licensed under cc by-sa 3.0 with attribution required.