首页/开发/代码质量与评审速查/评审作为知识传递
代码评审64 条

评审作为知识传递

评审是低成本团队学习机制:评论解释理由让新人学到规则,评审他人代码接触陌生模块;用好它可以降低单点依赖。

  • 评论带背景与参考:规则、文档链接、历史决策
  • 轮换评审人:避免只有少数资深成员掌握全貌
  • 新人先从评审测试与文档改动开始,逐步加深
  • 作者在描述里讲清设计:读者顺便获得领域知识

带背景的评审评论三段式

带背景的评审评论三段式
结论:这处建议从参数传入,而不是读全局变量。
依据:该服务并发处理请求,全局变量会互相覆盖(正确性问题)。
参考:团队规范「状态管理」一节,src/cache.py 有同类写法可沿用。
要点:评论给出规则出处与既有实现,让作者学到团队约定而非只听结论。

知识传递动作

场景动作
新人首次评审配对资深评审人共同完成一轮,事后对照关注点复盘
跨模块改动邀请目标模块维护者参与,顺带补齐模块认知
重复出现的评论写进模块文档或评审清单,减少口头重复与单点依赖
评审的隐藏产出是集体知识:每个 PR 都是团队共享上下文的载体。

易错点

  • 长期固定同一批评审人:形成瓶颈与隐性权威
  • 评审只挑错不解释:新人只学到别惹评审者

数据来源与参考

  • Google Engineering Practices(评审作为团队习惯)
​‌‌​​‌​​​‌‌​‌‌‌‌​‌‌​​​‌‌​‌‌‌​​‌‌​​‌​‌‌‌​​‌‌‌​​​‌​‌‌​‌​​‌​‌‌‌​​​‌​‌‌​‌​​‌​‌‌​​​‌‌​‌‌‌​‌​​​‌‌​‌‌‌‌​​‌​‌‌‌​​‌‌​​​‌‌​‌‌​‌‌‌‌​‌‌​‌‌​‌