GUIDE深度 · Deep dive
代码 review 怎么看 diff?
代码 review 的 diff 不是越长越好。先看改动的意图和边界,再看上下文;把格式化、重命名和逻辑改动分开,避免被噪音淹没。
三遍阅读
第一遍看改了哪些文件/函数,是否超出范围。
第二遍看逻辑和边界条件;第三遍看命名、注释、测试。
减少噪音
格式化改动单独提交,别和逻辑混在一起。
移动代码用工具识别,避免把 move 看成 delete+add。
常见问题
diff 太大怎么办?
拆小 PR;先看测试和关键路径。
要逐行看吗?
关键路径逐行,格式/生成文件可跳过。