代码审查的有效实践

代码审查做得好是团队技术质量最重要的防线

剑飞
1/14代码审查的有效实践

却也最容易被误解的实践之一

大多数团队都知道要做代码审查但真正把代码审查做有效的团队并不多

命题先说清本页判断
解释补足为什么
行动留下下一步
2/14代码审查的有效实践

而是面向已经有一定工程经验

这篇文章不是代码审查的入门介绍而是面向已经有一定工程经验但对代码审查的实际效果不满意的团队
3/14代码审查的有效实践

审查行为就会演变成一种仪式

01命题

先说清本页判断

02解释

补足为什么

03行动

留下下一步

把“审查行为就会演变成”落到一个具体项目里看结果
4/14代码审查的有效实践

要理解代码审查的目标

命题先说清本页判断
解释补足为什么
行动留下下一步
把“要理解代码审查的目标”落到一个具体项目里看结果
5/14代码审查的有效实践

具体来说

具体来说代码审查在解决以下几个层面的问题

把“具体来说”落到一个具体项目里看结果
6/14代码审查的有效实践

不同团队有不同的答案

特别是"知识传播"这个目标经常被低估——当团队扩张时这个问题会突然爆发 老代码没人能看懂
7/14代码审查的有效实践

再来看常见的误区

理解了代码审查的目标再来看常见的误区就容易理解为什么这些做法偏离了本质
8/14代码审查的有效实践

也是危害最大的误区

持这种观点的团队代码审查的默认模式是 作者提交 PR审查者找出问题 作者逐个修复 直到审

命题先说清本页判断
解释补足为什么
行动留下下一步
9/14代码审查的有效实践

这个模式的问题在于

命题先说清本页判断
解释补足为什么
行动留下下一步
把“这个模式的问题在于”落到一个具体项目里看结果
10/14代码审查的有效实践

作者在提交审查之前

这个自我审查不是指跑完 CI 检查(CI检查的是最低标准 不是质量标准)而是作者站在审查者的角度 提前过一遍
11/14代码审查的有效实践

作者不是把审查当考试

这个模式的关键是作者不是把审查当考试把审查者当阅卷老师

命题先说清本页判断
解释补足为什么
行动留下下一步
12/14代码审查的有效实践

带走四步

找项目

从真实任务开始

出材料

把想法变成可处理内容

做交付

用结果判断能力

可复用

把完成沉淀为流程

13/14代码审查的有效实践

让能力长出来

代码审查做得好是团队技术质量最重要的防线

返回原文
上一篇没有更多文章下一篇没有更多文章