成就高效DevOps团队的“降龙十八掌”第一式——codeReview？

什么不是codeReview

• Code reviews 不应该承担发现代码错误的职责。
  Code Review主要是审核代码的质量，如可读性，可维护性，以及程序的逻辑和对需求和设计的实现。代码中的bug和错误应该由单元测试，功能测试，性能测试，回归测试来保证的（其中主要是单元测试，因为那是最接近Bug，也是Bug没有扩散的地方）
• Code reviews 不应该成为保证代码风格和编码标准的手段。
  编码风格和代码规范都属于死的东西，每个程序员在把自己的代码提交团队Review的时候，代码就应该是符合规范的，这是默认值，属于每个人自己的事情，不应该交由团队来完成，否则只会浪费大家本来就不够的时间。个人认为“meeting”是奢侈的，因为那需要大家在同一时刻都挤出时间，所以应该用在最需要的地方。代码规范比起程序的逻辑和对需求设计的实现来说，太不值得让大家都来了。

为什么要codeReview

• Code reviews 中，可以通过大家的建议增进代码的质量。
• Code reviews 是一个传递知识的手段，可以让其它并不熟悉代码的人知道作者的意图和想法，从而可以在以后轻松维护代码。
• Code reviews 也鼓励程序员们相互学习对方的长处和优点。
• Code reviews 也可以被用来确认自己的设计和实现是一个清楚和简单的。