一个软件项目的质量定义并不是代码review的职责，换句话说，良好的质量定义是代码review发挥效果的必要前提。
代码review到底要review哪些东西？
想要通过代码review来检测每一行代码，并确保检查出所有问题是不可能的，它更侧重于处理核心且明显的问题。
谁来review？
这个要看开发组采取的review形式，一般分为独立review和集中review，前者指定要review的人，后者开发组都在场集中review。
独立review适合项目进度没那么紧张，且人员配置充足的情况，可以对每一笔提交进行review。
集中review则相反，比较适合人员和时间有限的情况，一般每周一次，需要投屏。
如何review？
明确需求（case） -> 代码结构 -> 核心逻辑 -> 代码风格。
review过程产生的问题，要记录下来，一般review工具（比如gerrit）提供的有记录方式，review结束后要推动问题的修改落实。
原文：https://blog.51cto.com/9797337/2368801