2009-05-19 67 views
0

免責聲明我不做代碼審查,所以這個問題純粹是學術興趣。代碼審查之前應該在哪裏存儲代碼?

我看到很多帖子在stackoverflow,主張/要求代碼審查之前它被允許進入源代碼管理。

如果你這樣做,你在哪裏存儲代碼未經審查的代碼,以及你如何處理開發人員必須更新代碼來處理其他檢查產生的衝突的問題 - 你是否需要他來獲取他的代碼代碼再次審查?

謝謝。

回答

4

代碼應該檢查到你的倉庫(SVN,TFS等)。如果你想阻止它進入主幹,那麼你可以設置一個開發(甚至是每個開發者)分支。

3

未審覈的代碼仍在測試中。信任是發展中的重要事情。所以答案是代碼只屬於版本控制系統。從那裏有一些得到審查。如果有問題,可以回滾。其他的事情在大多數時間裏都不是什麼大事。請記住,存儲庫中的代碼不是「正在生產的代碼」

每個人都應在提交之前從存儲庫進行更新。如果您檢測到很多更新,再次運行測試真的可行。如果沒有問題比提交。

-2

我們將它存儲在開發人員的桌面上。它沒有簽入SVN,因爲它尚未完成。

如果代碼很多,那就是一個問題 - 您等待太久才能查看。

如果這是一個合理的代碼量,它可以通過電子郵件發送給審閱者。也許它需要一個ZIP文件來保持它的組織。

有時我們會將其發佈到SharePoint,但這很少見。電子郵件通常工作正常

+0

代碼具有值(例如,用於公司或開發團隊),即使它沒有完成。在版本控制之前,它沒有獲得版本控制提供的優勢。 – 2009-05-21 04:15:21

0

即使在向利益相關者交付結果之前,我目前的團隊也會進行代碼審查,所以我在倡導僅在代碼審查後提交源代碼控制權的陣營中。

也就是說,一種可能性是將補丁文件存儲在磁盤上的目錄中,而不是提交到源代碼管理。另一種選擇是使用一個單獨的分支,以便在合併到目標分支之前承諾進行更改,但是我擔心這種方法會帶來危險。

1

學術問題的學術答案,因爲我們不做代碼審查。

所有東西都被檢入源代碼管理。如果它沒有完全正常工作/測試/審查,它將進入該開發者的個人分支。

1

這種做法是使用像Git這樣的DVCS的一個很好的理由。開發人員可以在沒有提交的情況下工作更長的時間,從而允許進行代碼審查,但仍然可以使用我們已經瞭解的版本控制技術。如果你正在使用類似SVN的東西,你必須爲每個bug /特性/需要編寫的東西分支,並在代碼檢查之後重新整合它......這可能是痛苦的。

1

這實際上取決於團隊設置了哪些工具和程序。

對於非正式的代碼審查,你可以簡單地檢查到版本控制,並讓其他開發者審覈他們自己的時間表;然後單獨檢查審查發現的問題。 (這就是我們的團隊所做的)

對於更正式的工具輔助代碼審查工具,如Google的Rietveld和(我敢肯定)智能熊的Code Collaborator可以讓您上傳代碼進行審查並擁有微型版本控制功能,因爲它在審查過程中進行了更新,因此每次提交的樣式歷史記錄。 (如果您有興趣瞭解更多關於代碼審查,智能熊有關於這個問題的free book