我们时常听到团队成员对代码审查的抱怨和困惑：

“为何我们要浪费时间进行代码审查？”、“我没时间做代码审查。”、“发布会延迟，都是因为那未完成的代码审查。”等等。那么，我们为什么要进行代码审查呢？

作为专业的软件开发人员，我们的核心目标之一是持续提升个人及团队的工作质量。而代码审查是实现这一目标的关键途径之一。通过代码审查，我们可以从另一个角度发现潜在的缺陷和更优的解决方案，确保至少还有一人熟悉并理解我们的代码。翻阅资深开发人员的代码，有助于培训新员工，促进团队间的知识共享。更重要的是，它能激励开发人员写出更高质量的代码，避免在审查时暴露出问题。

但值得注意的是，代码审查需要彻底且投入足够的时间和精力才能真正实现其目标。在我看来，大约25%的原始开发时间应该用于代码审查。例如，如果一个开发人员需要两天完成一个程序，那么他应该用大约四小时进行审查。

审查代码并不仅仅是检查语法，更重要的是理解代码是如何融入应用程序的，成为其中的一部分。如果我们对每一行代码的含义不够清晰，那么审查的效果将大打折扣。良好的代码审查需要时间来深入研究，如触发特定功能或确保第三方API的正确使用等。

在审查过程中，除了寻找代码缺陷，我们还应该确保测试得当、设计文档齐全。即使是最出色的开发人员，在修改代码时也可能忘记更新这些重要资料。代码审查时应该确保这些资料始终有用且准确。

我们也要避免过度的代码审查带来的问题。开发人员应努力清空积压的审查任务，合理安排审查时间，避免由于审查延迟导致发布推迟。应避免审查已经大幅修改或不再相关的代码，或由于时间紧迫而匆忙走过场。

为了实现更有效的代码审查，我们还需要编写可审查的代码。这意味着在项目初期就做好准备工作，如讨论棘手的架构决策、详细的项目架构设计、适当的单元测试等。如果项目包含第三方代码，应单独提交以便于审查。为降低审查难度，还可以在关键部分添加注释，帮助审查人员更好地理解你的代码。

《同行代码审查的实用经验》

在软件开发领域，重构代码是提高代码质量、优化性能的关键步骤。当增量重构不可行时，我们还有一个选择——结对编程。这种方式虽然可能需要更多的重构时间，但最终产出的代码质量更高，审查过程也更加轻松。

即便有了这些解决方案，团队中仍然可能出现意见分歧的情况。每个团队成员都是人才，都有自己独特的见解和方法。作为开发人员，我们需要保持开放的态度，虚心接受审查人员的反馈和建议。而审查人员在提出建议时，也需要以委婉的方式表达。

在提出改进意见之前，审查人员应该深入思考自己的建议是否真正有助于代码质量的提升，而不仅仅是因为个人喜好或风格差异。如果确实发现代码存在问题需要改进，说服过程就会变得简单许多。例如，使用委婉的语气和措辞，如“这里或许还有改进的空间……”，“有同行建议……”，而不是过于强硬或傲慢地表达自己的观点。

当双方僵持不下、互不让步时，可以邀请一位双方都尊重的开发人员介入，为团队提供中立的意见和建议。这样的做法有助于化解争端，促进团队的和谐与协作。通过有效的沟通、理解和尊重彼此的观点，我们可以共同推动项目的进展，提升代码质量。

同行代码审查是软件开发过程中不可或缺的一环。通过不断学习和实践上述实用经验，我们可以更好地进行代码审查，提高团队协作的效率和质量。

欲了解更多关于代码审查的实用技巧和建议，请访问译文链接：