第152题 如何做code-review
code review（简称CR）即代码走查。领导对下属的代码进行审查，或者同事之间相互审查。CR已经是现代软件研发流程中非常重要的一步，持续规范的执行CR可以保证代码质量，避免破窗效应。
CR 检查什么
- 代码规范（
eslint能检查一部分，但不是全部，如：变量命名） - 重复逻辑抽离、复用
- 单个函数过长，需要拆分
- 算法是否可优化?
- 是否有安全漏洞?
- 扩展性如何？
- 是否和现有的功能重复了？
- 是否有完善的单元测试
- 组件设计是否合理
何时 CR
- 提交
PR（或者MR）时，看代码diff。给出评审意见，或者评审通过。可让领导评审，也可以同事之间相互评审。 - 评审人要负责，不可形式主义。万一这段代码出了问题，评审人也要承担责任。
- 例行，每周组织一次集体
CR，拿出几个PR或者几段代码，大家一起评审。 - 可以借机来统一评审规则，也可以像新人来演示如何评审。