代码审查

如果要挑出一项极其重要，却又很容易被忽视的工作，在我看来代码审查几乎是无可争议的第一位。

代码审查的内容

• 帮助理解代码的描述
• 实际的代码变更主体
• 测试和结果

双方思考、争辩、以及妥协，目的都是寻求一个切合实际且可以改进代码质量的平衡。

代码审查的好处

• 代码审查是个人和团队提升的最佳途径之一
• 代码审查是团队关系建设和扩大双方影响力的有效方式
• 识别出设计的缺陷，找到安全、性能、依赖和兼容性等测试不易发现的问题。
• 设计团队质量标杆的最佳实践方式。

代码审查的小技巧

• 每次变更所包含的代码量一定要小
• 让团队中的牛人在代码审查中发挥作用
• 变更代码的质量要超过当前代码库的平均水平
• 新员工代码、股价代码的代码审查要更为严格
• 及时表达肯定，委婉表达意见
• 审查时，代码要过两遍，第一遍朱主要问题，第二遍看次要 entity。

思考

对于 code review 的用处挺有感触的。待过不少团队，有的团队明面上也会要求 code review，不过基本都是应付了事。唯独在龙哥的团队里面，TL 带头进行非常严格的 code review，一开始我还不太适应。但慢慢就发现自己写代码不敢乱来了，对自己的代码要求页提高了，渐渐地代码习惯也好了很多。
感觉 code review 这东西很多人反感，因为要费更多时间。但实际用下来是的的确确有所收获的。之后我也要试着去推 Code Review 的事宜，并严于律己。