中等團隊的代碼審查實踐

Question

我正在尋找組織代碼審查過程的方法。

我輸入：

1. 4-5成員在團隊;
2. SVN/IntelliJ IDEA社區版。

解決方案標準：

1. 應該支持人工審覈與評論/高亮顯示;
2. 應該很容易（安裝，快速進行審查）;
3. 應該是免費的（理想情況下）;
4. 應該根據代碼庫在SVN倉庫中進行審查;
5. 不應該在SVN倉庫中進行任何提交;
6. 不應該在第三方服務上進行任何發佈（理想情況下不應該與這些服務進行交互）。

我感興趣的任何建議：

1. 流程組織;
2. 工具;
3. 最佳做法。

感謝您的建議！

Answer 1

既然你似乎是使用Java工具下，考慮聲納 - http://www.sonarsource.org/

雖然你可以使用出的現成的配置文件之一，我會建議花時間驗證700+它的規則隨身攜帶，以確定哪些適合你，哪些不適合你。還有大量的插件可用於捕獲不同類型的指標，如技術債務，死/無用代碼，質量指標（基於違反規則的類型）等。

如果您使用Maven構建代碼，Sonar與它很好地結合在一起。只需運行mvn sonar:sonar。有關詳細說明，請參閱here。

此外，還有一個Ant task也可用，以允許您通過Ant運行Sonar。

您可能還想要設置連續集成。 Sonar與Jenkins很好地結合，如果您的項目是Mavenized的，並且使用SCM plugin跟蹤SVN歷史記錄。如果您正在使用Ant任務，設置CI的最簡單方法是以Jenkins作業運行Ant任務。那麼'集成'將僅限於觸發構建。您需要爲每個SVN分支提供一個Jenkins作業。對於使用Ant的項目，您需要使用SVN Ant任務來獲取SVN信息（最新版本，分支名稱等），以便在Sonar的報告中顯示。

您需要決定運行Sonar的頻率，以便您可以跟蹤進度並有時間採取措施。我會建議在一兩天內運行一次。

曾經有一個IntelliJ插件，但似乎不再支持。如果你可以在你的代碼中直接在IntelliJ中安裝用於運行Checkstyle，Pmd和Findbugs規則的插件，並且如果你在IDE中設置了相同的規則，那麼開發人員在提交代碼之前就可以更容易清理代碼，從而聲納將成爲一個監測/跟蹤工具。

編輯：

對於同行的評論，你可以登錄像的Bugzilla或JIRA一個bug /任務跟蹤工具的所有意見。將它與電子郵件整合在一起，每個「評論」都會通過電子郵件發送給分配給他的人。