作为初级开发人员如何进行代码审查？

“代码必须经过高级开发人员的审查。” “后辈的评论很好，但他们的认可毫无价值。” 如果您从未听过这些，那么您很幸运。当然，他们完全错了。

作为初级开发人员，参与代码审查提供了宝贵的学习机会以及为团队的成功做出贡献的机会。

在这篇文章中，我将探讨如何作为初级开发人员提供有效的代码审查反馈。

作为初级开发人员，您的旅程中需要指导吗？我可以帮助您学习技能，变得更加自信，并实现您梦想的工作！

为什么要进行代码审查？

代码审查有多种目的，对个人开发人员和整个团队都有好处。以下是代码审查至关重要的几个关键原因：

从代码中学习

代码审查让您接触到不同的编码风格、技术和方法。通过查看其他开发人员（无论是否有经验）编写的代码，您可以获得宝贵的见解并提高自己的技能。

确保良好实践

代码审查有助于维护项目内的编码标准和最佳实践。通过识别与既定准则的偏差，您可以为整体代码质量和可维护性做出贡献。

例如，如果您的团队决定文件应按业务模块组织，而开发人员在其拉取请求中没有遵循此约定，那么代码审查是告诉他们的正确方法。

检查错误或边缘情况

通过仔细检查，代码审查有助于发现开发过程中可能被忽视的潜在错误、逻辑错误或漏洞。尽早识别并解决这些问题可以节省时间并最大限度地减少未来的复杂性。

请注意，根据您的团队同意的内容，审阅者可能有责任也可能没有责任手动测试更改。

挑战实施

代码审查鼓励批判性思维，并提供挑战设计决策或建议替代方法的机会。通过分享您的观点，即使作为初级开发人员，您也可以为团队的集体解决问题的能力做出贡献。

代码审查不应该用于哪些用途

虽然代码审查是一个有价值的过程，但了解其范围和局限性也很重要。作为初级开发人员，了解代码审查不应该主要用于哪些用途会很有帮助：

代码格式化和 Linting

代码审查不是对次要格式或样式首选项挑剔的地方。自动化工具和 linting 流程可以处理这些问题，使代码审查能够专注于更实质性的问题：ESLint、Prettier等。

大架构讨论

代码审查通常不是大规模架构讨论的理想场所。虽然提供有关架构决策的意见可能很有价值，但最好在单独的讨论或会议中解决这些问题。

对初级开发人员的期望是什么

作为参与代码审查的初级开发人员，您可以发挥至关重要的作用。以下是对您的期望：

仔细阅读代码并从中学习

花时间彻底理解正在审查的代码。吸收更有经验的开发人员使用的逻辑、模式和技术，因为这将有助于提高您自己的技能。

您还将了解您正在从事的具体项目、应用程序的设计方式、其技术债务等。

如果有不清楚的地方请说出来

如果您遇到不清楚或令人困惑的代码部分，请毫不犹豫地提出问题或寻求澄清。有效的沟通可确保每个人都达成共识，并有助于防止误解。

每个开发人员，包括初级开发人员，都应该能够理解项目中的每一段代码。如果您无法理解某些代码，可能是因为它没有以最清晰的方式编写。

请注意，有时，代码由于某些原因而难以理解，例如处理性能问题。在这种情况下，可以使用代码中的注释来简化其他开发人员的工作。

分享另类观点

作为初级开发人员，您可以为代码审查过程带来全新的视角。如果您相信有一种不同的、更有效的或更优雅的方式来实现某件事，请尊重地分享您的想法并鼓励讨论。

不要因为您刚刚学习了技术而认为您没有足够的经验来挑战高级开发人员的实施。有时，询问开发人员为什么不遵循您刚刚学到的实践会引发富有洞察力的讨论！

在评论中使用正确的语气

以积极和建设性的方式提供反馈对于维持健康和协作的团队环境至关重要。以下技巧适用于各种经验水平的开发人员，但对于挑战高级开发人员代码的初级开发人员尤其相关。

使用“为什么不”而不是“你应该”

将您的建议或替代方法作为问题而不是命令。这种方法鼓励讨论并允许开发人员解释他们的思维过程。

示例：您注意到开发人员做了框架文档不赞成的事情。不要告诉“你应该这样做而不是那样做”，而是考虑“我注意到文档建议不要那样做。为什么不这样做呢？

保持开放和积极的态度

在评论中采用支持性的语气，强调代码的积极方面并认可开发人员的努力。通过营造积极的氛围，您可以鼓励持续改进并激励同事。

示例：“我不知道我们可以通过这种方式实现该功能，感谢您帮助我学习！只是想知道它是否不会导致性能问题，因为……”

如果您认为有问题，请提出替代方案

当您添加评论以建议更改时，请始终尝试提出进行更改的具体方法。例如，如果您认为变量或函数名称是错误的（命名事物很困难），不要只是告诉“请为变量/函数指定一个更好的名称”。相反，你可以说“我认为这个名字或那个名字更有意义，你觉得怎么样？”

作为初级开发人员，参与代码审查是提高技能、提高代码质量以及与团队有效协作的宝贵机会。

通过怀着学习的愿望进行代码审查、分享建设性反馈并营造积极的环境，您可以对开发过程产生有意义的影响。

将代码审查视为个人和团队成长和改进的机会。