代码质量与技术债系列分享之一 - 如何做好 Code Review

文章预览

名词解释 CR: Code Review CR：代码审查 CL: Stands for "changelist", which means one self-contained change that has been submitted to version control or which is undergoing code review. Other organizations often call this a "change", "patch", or "pull-request". CL：代表“变更列表”，表示已提交到版本控制或正在进行代码审查的独立更改。其他组织通常将其称为“更改”、“补丁”或“拉取请求”。 LGTM: Means "Looks Good to Me". It is what a code reviewer says when approving a CL. LGTM：意思是“对我来说看起来不错”。这是代码审查员在批准 CL 时所说的。 CR意义 灵魂拷问：为什么我们接手的每个代码库都如此难以维护？ 重要原因之一：Code Review 执行不彻底 万能借口：太忙！ 疲于应付眼前 不可见，意识不到 CR 非功能性开发 CR 不是当务之急，没有眼前收益 危害被低估，忽视“复利”的威力（非线性） 意义 现代代码评 ………………………………