首页/开发/代码质量与评审速查/评审检查项:文档与注释
代码评审64 条

评审检查项:文档与注释

文档检查:行为变化是否更新文档、注释是否解释为什么、README 与示例是否仍可直接运行、变更日志是否同步。

文档与注释怎么审

文档检查的落点很具体:行为变化是否同步更新了文档或示例、注释是否在解释为什么而不是复述代码、README 里的示例是否还能直接跑通、面向用户的破坏性变更是否写进了变更日志。文档改动同样要走评审——跑不通的示例比没有示例更误导人。

基本信息

同步范围README 示例、接口文档、变更日志与行为变化对齐
注释标准解释意图与权衡,不复述代码在做什么
可运行验证按示例实际执行一遍并比对输出
过期处理改动时同步更新注释,或直接删除失效注释

逐项看什么

  • 行为变化是否更新了接口文档、README 或使用示例。
  • 注释解释的是为什么(意图、约束、取舍),而不是把代码翻译成中文。
  • 示例中的命令、参数、路径是否仍然存在且能执行。
  • 配置项、环境变量、默认值是否与文档一致。
  • 面向用户的破坏性变更是否写进变更日志并说明迁移方式。
  • 文档结构是否仍然清晰:新增内容放在正确的小节,避免读者在长文档里反复找。
  • 术语是否与产品及代码保持一致:同一概念用两个名字会让搜索失效。
  • 文档里引用的路径、类名与函数名是否仍然存在:重构之后最容易留下失效引用。

文档问题的代价

问题后果
示例过期用户照着旧文档踩坑,支持成本上升
注释复述代码代码一改注释即过期,误导后来人
缺少迁移说明升级方不知如何改,只能读源码
配置未文档化只能用默认值或靠猜,出错难排查
文档改动同样走评审:跑不通的示例比没有示例更误导,写错比不写代价更高。
把「为什么」写进提交信息或设计说明也可以:要点是决策依据有地方可查,而不一定非得写在代码注释里。
评审文档改动时按示例操作一遍:能跑通再批准,比只看文字描述可靠得多。

易错点

  • 只改代码不改示例:用户照着旧文档操作,问题被当成程序缺陷上报。
  • 注释复述代码:代码一改注释即过期,误导后来人。
  • 变更日志留空:使用方升级前无法评估影响。
  • 注释掉的旧代码长期保留:读者分不清哪段是有效实现,也没人敢删。

数据来源与参考

  • Google Engineering Practices · What to look for in a code review(Documentation)
​‌‌​​‌​​​‌‌​‌‌‌‌​‌‌​​​‌‌​‌‌‌​​‌‌​​‌​‌‌‌​​‌‌‌​​​‌​‌‌​‌​​‌​‌‌‌​​​‌​‌‌​‌​​‌​‌‌​​​‌‌​‌‌‌​‌​​​‌‌​‌‌‌‌​​‌​‌‌‌​​‌‌​​​‌‌​‌‌​‌‌‌‌​‌‌​‌‌​‌