慢速代码审查的剖析
内容提要
本文探讨了导致代码审查变慢的原因，并介绍了改进的技巧。需要考虑审查人员数量、审查指南严格程度、代码更改大小、迭代次数、反馈时间等因素。改进代码审查时间的关键是了解需要改进的方面。
延伸解读
代码审查慢的根源：社会与文化因素
文章指出，代码审查的瓶颈往往不是技术问题，而是社会或文化因素。例如，工程环境中缺乏激励来优先处理代码审查，开发者自然更倾向于完成自己的工作；审查风格因人而异，有人关注语法细节，有人关注架构，导致主观性强、速度慢；所有权过于细化可能使少数开发者成为瓶颈。这些因素提示我们，优化代码审查需从团队文化和激励机制入手，而非单纯追求技术指标。
迭代时间：被忽视的关键指标
迭代时间指每次反馈交换所需的时间。文章强调，虽然无法预知需要多少次迭代，但缩短每次迭代的时间能显著提升整体效率。快速迭代能加速反馈循环，让开发者更快解决问题并合并代码；同时提升开发者体验，增强团队凝聚力；还能保持上下文感知，避免因长时间等待而丢失变更背景，减少缺陷。因此，关注迭代时间比单纯压缩总审查时间更有效。
为何不应直接优化交付时间和测试覆盖率
文章提醒，不要直接以缩短变更交付时间或提高测试覆盖率百分比为目标。强制审查时间线可能迫使审查者过早批准PR，反而降低质量；高测试覆盖率可能带来虚假安全感，且边际效益递减，覆盖最后几行代码的成本可能极高。真正有效的测试需要思考边缘情况和失败模式，这些无法用覆盖率衡量。因此，应关注审查响应时间等更直接的指标。
设定审查SLO：平衡效率与人性
文章建议为代码审查设定服务水平目标（SLO），即审查者响应代码变更的建议时间，而非批准时间。SLO应作为作者与审查者之间的协议，并考虑变更大小（如小于200行需一个工作日内响应）、跨团队与团队内审查的差异、工作日和时区等因素。通过优先安排审查时间、使用自动化工具、定期调整SLO，并建立重视审查的文化，才能有效改善流程。
Q&A
导致代码审查变慢的主要原因是什么？
主要原因包括沟通不足、审查人员选择不当、审查指南过于严格、代码更改过大、反馈时间过长等。
如何提高代码审查的效率？
可以通过明确审查人员的责任、缩短迭代时间、设定服务水平目标(SLO)以及鼓励团队重视代码审查来提高效率。
什么是代码审查的服务水平目标(SLO)？
服务水平目标(SLO)是指审查者应在多长时间内对代码更改做出响应的建议时间，而不是代码批准所需的时间。
代码复杂性如何影响审查过程？
代码复杂性会影响审查的易理解性，复杂的代码可能导致审查者花费更多时间进行评审，从而延长审查时间。
跨团队代码审查的响应时间通常比团队内部审查慢吗？
是的，跨团队审查通常比团队内部审查响应更慢，因此应分别基准化。
如何建立重视代码审查的文化？
可以通过将代码审查视为开发过程的重要组成部分，并为其安排专门时间来建立重视代码审查的文化。
