关于 Code Review 的观点和思考记录

[hawtim]

“主要是让你的代码可以更好的组织起来，更易读，有更高的维护性，同时可以达到知识共享，找到bug只是其中的副产品”

[hawtim]

代码有这几种级别：1. 可编译 2. 可运行 3. 可测试. 4. 可读 5. 可维护 6. 可重用。 通过自动化测试的代码只能达到第 3 级，而通过Code Review的代码少会在第 4 级甚至更高。

[hawtim]

code review 在前端的实践

1. 确保代码风格的一致，符合 eslint ，统一的语法，经过讨论后更优的代码组织方式。
2. 严格关注 typos，避免因为研发人员的英语背景为项目带入 typos, 可以用 Webpage Spell-Check 的谷歌拓展工具，非常实用。
3. 重要业务先了解实现方案，即不写代码先想清楚，然后同步团队成员进行review
4. 大量的代码或者需要仔细评审的代码，如核心功能或者组件库等基础建设，使用 WIP 提前提交 MR，避免非常大心智开销的集中 review
5. 值班人员和项目负责人承担主体的 code review 工作