2012-01-05 23 views
6

Olası Çoğalt:
Web Based Code-Review tools for Team Foundation ServerTFS ile Kod İnceleme nasıl yapılır?

Biz yaklaşık 4 geliştiriciler var ve biz kaynak kontrolü için TFS kullanıyoruz.

Kodu gözden geçirmeyi, kontrol edilen kodların performans, tasarım ve standartlar için akran üyeleri tarafından gözden geçirilmesini sağlamak istiyoruz.

Buna yardımcı olmak için hangi aracı kullanmanızı öneririz? (açık kaynak ve ücretsiz umarım)

Bunu Google'dan öğrendim: http://teamreview.codeplex.com/ ve nasıl çalıştığını test etmeye çalışıyorum. Küçük bir takımda TFS kullanırken herhangi araçların bilmiyorum

sayesinde

+0

http://stackoverflow.com/questions/1235410/web-based-code-review-tools-for-team-foundation-serverYanıt yinelenen? – tafa

+0

TeamReview biz evet bu yardımcı olacağını, – pantelif

cevap

1

, haddi zatında, ama, ben sadece bir shelveset değişiklikleri yedeklemek yeterli olduğunu ortaya koymuştur. Bu şekilde, diğer geliştiriciler başkalarının raflarını görebilir ve kendi zamanlarındaki kod farklılıklarından geçebilirler. Kodun derlenmesini ve uygulanmasını kontrol etmek istiyorlarsa, kendi makinelerindeki değişiklikleri de çözemezler.

Kodun doğru olduğundan emin olmak için yapabileceğiniz başka bir şey, birim sınamaları yazıp bunları kod incelemesi altında tutmaktır. Bu şekilde, test sistemi, bir kod gözden geçiricisinin gözden kaçırdığı olası davranış bozukluklarını belirlemenize yardımcı olabilir.

+0

sayesinde, gereken ancak ekibimiz kurşun bazı nedenlerle yazma birimi testlerinde karşı şu anda ne çok oldu! bu yüzden bu yaklaşımı alamayız. Davranış Kuralları yalnızca bir teknoloji uzmanı mı yoksa tüm ekip üyeleri mi olmalı? –

+0

Muhtemelen tüm takım üyeleri olmak en iyisidir. Belki de tek bir parça koddaki tüm üyeler (çok önemli olmadığı sürece). Ancak, tüm üyelerin her geliştiricinin kodunu bir noktada gözden geçirmeleri için zaman ayırdığından emin olun. Bu, kodlama standartlarınızla tutarlılığı korumaya yardımcı olacak ve ekip, kod tabanının daha büyük bir bölümünü daha iyi anlayacaktır. Ve kayıt için - birim test wonderous şeydir ve başlangıçta kurmak için zaman alabilir, hata ayıklama saatleri ve uzun vadede şirket çok para kaydedebilirsiniz! –

8

Bunun sizin için nasıl uygulanacağından emin değilim, ama işte geliyor. WIT işlem şablonumuzu (TFS power tools kullanarak) değiştirdim, böylece kod kontrolünden sonraki durum "Kod İncelemeyi Bekliyor". Daha sonra bu WI'yı, kod gözden geçirme tamamlanmışsa veya başarısız olduğunda atanana geri döndüğünde "Kapalı" durumuna geçiren başka bir kullanıcıya atamanız gerekir.

Yani süreç geçişleri artık böyle bir şey vardır:

 
Assigned --Check-In-->Code Review --Success-->Closed 
    ^-------Failed---------| 

Bu mükemmel olmaktan uzak olduğunu ve büyük ölçüde biz Subversion ve ayrı bir hata izleme sistemi ile vardı süreçlere dayanıyor, ama işe yarıyor.

Diğer bir haberde, VS 11 bu out of the box'a (ayrıca here - Ewald'ın cevabından alınmıştır) sahip görünüyor.

+1

+1 - Bu güzel bir yaklaşım. Ve VS11'in zaten entegre edilmiş olduğu gerçeğini beğeniyorum. –

+0

+1, ilginç görünüyor. Yine de şu dezavantajlarım var gibi görünüyor: Tüm WI'larınız yorumcular tarafından kapatılıyor. Peki, her geliştiricinin iş yükünü nasıl bilebilirsin? – pantelif

+0

@pantelif, Hala TFS'ye yeni geldik, bu yüzden bu köprüyü henüz geçmedik. – DaveShaw

1

Şu anda size yardımcı olmayacaktır, ancak Dev11 (şu anda Geliştirici Önizleme olarak kullanılabilir) kod incelemesini içeriyor. session from Jamie Cool at //BUILD'da güzel bir genel bakış görebilirsiniz.

Büyük kararlar vermeden önce, bunu günlük işinizde kullanmadan önce birkaç ay beklemek isteyebilirsiniz.

Ewald Hofman - Program Yöneticisi TFS

1

Ben Review Assistant code review tool tavsiye edebilir. Visual Studio 2010 ve 2012'ye entegre olur ve TFS ile çalışır. Gönderinizde görebileceğim gibi, post-commit kod inceleme süreci sizin için tercih edilir. Bu hassas model araçta uygulanmaktadır.