为什么初级开发人员应该审查高级开发人员的提交

代码审查是发现错误、潜在缺陷以及其他代码作者可能忽略的问题的绝佳方法。审查不仅能提高整体代码质量，还能有效分享知识和信息。初级开发人员的提交当然应该由资深开发人员进行审查，但初级开发人员也应该成为审查者。原因如下。

每个项目都有一些事情需要熟悉，当一名开发人员（无论是否是初级开发人员）加入团队时，这些事情可能包括代码和项目组织、测试、构建、部署以及项目中的常规工作方式。尤其是在大型项目中，理解现有的架构和设计决策以及这些决策背后的原理，对于新开发人员来说可能是一个重大挑战。

技术细节和工作方式通常都可以从项目文档中找到。然而，在不熟悉的代码库中实现新功能或修复错误仍然具有挑战性。还有什么比在同一个项目中效仿更有经验的开发人员更好的学习方法呢？

除了在同一个项目中效仿更有经验的开发人员的例子之外，还有什么更好的学习方法呢？

让年轻的开发者通过评审来查看经验丰富的开发者的提交，对评审者来说是一个很好的学习经历。这不仅为设计和架构决策以及编码和代码风格实践提供了很好的参考，还能展示如何在开发者正在参与的实际项目中运用这些经验。对于被评审者来说，即使评审者经验不足，多一双眼睛来审视代码也无妨。事实上，对项目不太熟悉的人更有可能从全新的视角来看待它。

这种评审的益处很多，主要体现在初级开发人员身上。然而，它对项目本身也大有裨益。显然，开发人员可以更快地熟悉项目，但评审也提供了比手头任务本身更广阔的视角。新开发人员未来很可能会负责维护项目，因此从一开始就拥有更广阔的视角至关重要。

到目前为止，讨论的都是技术层面的问题。然而，审查实践也能产生社会影响。当新开发人员被要求审查现有成员的提交时，这表明他们被接纳为团队的一员，他们的意见和建议也受到重视。一如既往，成功的审查实践需要健康的工作环境。每个人都应该能够给予和接受反馈，而不会将评论当成针对个人的。为了强化这种氛围，最好让从初级到高级的每个人都既是审查者，也是被审查者。

这些相同的做法也可以应用于不同团队和不同能力的人员。为什么不让嵌入式开发人员审查一些 GUI 代码，或者让后端开发人员审查一些前端更改呢？即使某些内容超出了某人的主要能力范围，并不意味着他们无法发现错误或提供反馈（当然，一些与特定库或语言相关的更细微的错误仍然需要在该领域具有强大能力的审查人员）。

因此，让我们充分利用代码审查，让从初级到高级的每个人都参与其中。