关于check-in应该是在代码审查之前还是之后

 

关于这个问题，一直有争论。

在check-in之前进行review的好处是保证repository中代码的整洁，缺点是增加了代码分享的延迟，延长了集成测试的周期。

关于check-in之后进行review，参加此文：《Check In Early, Check In Often》

http://www.codinghorror.com/blog/2008/08/check-in-early-check-in-often.html

在stackoverflow对此也有讨论：

http://stackoverflow.com/questions/20327/code-review-vs-check-in-often

http://stackoverflow.com/questions/246319/peer-review-code-before-or-after-check-in

其中提出了使用Git作为版本控制工具的方案。

本人认为，选择之前还是之后要看开发组成员的整体开发水平，如果水平都比较高，则可采取先checkin再审查，以提高效率；

如果整体水平不高，还是先review再checkin比较可取，以保证代码质量。