http://spaces.msn.com/members/spiritauding/Blog/cns!1psm74keJLzaQ6CnZ_EB1mAw!127.entry
C++ Coding Standards Item 4: Invest in code reviews
Summary
Re-view code: More eyes will help make more quality. Show your code, and read others'. You'll all learn and benefit.
审查代码:更多眼睛能帮助提高质量。展示你的代码,并且去读别人的代码,你将从中学到很多知识,受益匪浅。
这条看起来有点像CMMI中的同级互查,:-),在北京的时候也曾经推动过这项活动,不过由于项目结束日的遥遥无期,功能修改的频繁,最终的效果不理想;现在加入了日本公司,这项工作被作的十分严格!在日本那边,我们有一个牛人,可以用他的眼睛抓住我们代码中的Bug,所以我们内部所作的代码互查也格外认真。说实话,如果认真、努力的去读代码的话,还是会发现很多问题的,注意,这可是在交给测试人员折腾之前。
Discussion
A good code review process benefits your team in many ways. It can:
好的代码互查可以让你的项目小组从多个方面受益。如:
* Increase code quality through beneficial peer pressure.
* 可以在同等的压力下提升代码的质量。
* Find bugs, non-portable code (if applicable), and potential scaling problems.
* 找到bugs,剔除不简便的代码,和潜在的问题。
* Foster better design and implementation through cross-breeding of ideas.
* 可以更好的按照设想进行设计和实现。
* Bring newer teammates and beginners up to speed.
* 可以让新的小组成员和新手更快的进步。
* Develop common values and a sense of community inside the team.
* 可以增进小组内部的平等的价值观和交流的感觉。
* Increase meritocracy, confidence, motivation, and professional pride.
* 促进知识的提炼,增加自信,明确动机和个人成就感。
呵呵,大师给出的理由是那么的诱人,不过每一条倒都是实实在在的,自己做的东西确实是很希望别人来评价的,批评也好,表扬也好,好歹也都是个人价值的体现。
Copy Left (C) Scorpio Auding 2005