C++ Coding Standards Item 4: Invest in code reviews

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