2012-01-05 54 views
6

可能重複:
Web Based Code-Review tools for Team Foundation Server如何使用TFS進行代碼審查?

我們有大約4開發人員和我們使用TFS源控制。

我們想介紹一下Code Reviewing,以便籤入的代碼由同行成員審查,以獲得性能,設計和標準。

你會推薦我們用什麼工具來幫助? (開放源碼和免費希望)

我發現這是谷歌:http://teamreview.codeplex.com/,我試圖測試它是如何工作的。

感謝,

+0

http://stackoverflow.com/questions/1235410/web-based-code-review-tools-for-team-foundation-serverPossible duplicate? – tafa 2012-01-05 09:55:15

+0

TeamReview完全是我們所需要的 – pantelif 2012-01-05 10:15:28

回答

1

我不知道的任何工具,本身,而是在一個小團隊使用TFS的時候,我發現,只是備份更改了上shelveset是適當的。通過這種方式,其他開發人員可以查看其他任何擱置的代碼,並在他們自己的時間內查看代碼差異。如果他們希望檢查代碼的編譯和執行情況,他們也可以在自己的機器上取消更改。

你可以做的其他事情,以幫助確保代碼是正確的,編寫單元測試,並將它們置於代碼審查。這樣,測試系統可以幫助您識別代碼審查人員可能會漏掉的潛在行爲缺陷。

+0

謝謝,是的,這會有所幫助,但我們的團隊領導目前反對編寫單元測試出於某些原因!所以我們不能採取這種方法。代碼評審員是否應該只是一名技術專家或所有團隊成員? – 2012-01-05 10:03:34

+0

可能最好是所有團隊成員。也許並非所有成員都在一段代碼上(除非它非常重要)。但是確保所有成員都花時間在某個時候檢查每個開發人員的代碼。這將有助於保持與您的編碼標準的一致性,並且團隊將更好地瞭解更大部分的代碼庫。對於記錄 - 單元測試是一件非常奇怪的事情,雖然可能需要一段時間才能開始設置,但從長遠來看,可以節省數小時的調試時間和大量公司資金! – 2012-01-05 10:06:45

8

我不知道這將如何適用於你,但在這裏。我更改了我們的WIT流程模板(使用TFS power tools),以便代碼簽入後的狀態是「等待代碼審查」。然後,您需要將此WI分配給另一個用戶,如果代碼審查是正確的,則將其移動到「已關閉」狀態;如果失敗,則將其重新分配給另一個用戶。

因此,處理變換現在是這樣的:

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

這是遠遠不夠完善,這在很大程度上是基於我們與Subversion和一個單獨的bug跟蹤系統的過程,但它的作品。

在其他消息,VS 11看起來有這out of the box(也here - 取自埃瓦爾德的答案)。

+1

+1 - 這是一個不錯的方法。我喜歡VS11看起來已經整合的事實。 – 2012-01-05 10:13:14

+0

+1,看起來很有趣。但在我看來,它也有以下缺點:所有WI都由審閱者關閉。那麼,你怎麼知道每個開發者的工作量? – pantelif 2012-01-05 10:35:17

+0

@pantelif,我們還是TFS的新手,所以我們還沒有穿過那座橋。 – DaveShaw 2012-01-05 13:16:08

1

它現在不會幫助你,但Dev11(目前作爲開發者預覽版可用)包含代碼審查。你可以在session from Jamie Cool at //BUILD看到一個很好的概述。

因此,在您作出重大決定之前,您可能需要等待幾個月才能將其用於日常工作。

埃瓦爾德·霍夫曼 - 項目經理TFS

1

我可以推薦Review Assistant code review tool。它集成到Visual Studio 2010和2012中,並與TFS協同工作。 正如我從您的帖子中看到的,提交後代碼審閱過程對您更可取。這個精確的模型是在該工具中實現的。