什么是好的代码审查流程？ 商业利益解释
已发表: 2023-03-14什么是代码审查？ 让我们从一点点历史开始。 第一个制定和发布的代码审查归功于 IBM 研究员 Michael Fagan，他于 1974 年引入了该流程。Fagan 的成就迅速成为最关键的软件开发流程之一，并已在全球范围内使用近五年。 在全球 DevSecOps 调查中，近 76% 的开发者承认，这个软件开发阶段是其最有价值的部分。
当有人审核他们的工作时，为什么技术员工会很高兴？ 答案很简单。 因为双方（Submitter 和 Reviewer）都有明显的好处。 雄心勃勃的开发人员应该寻求与最优秀的专家合作，并定期促进他的专业发展。 没有比编写大量代码更好的方法来提高编程技能了——更重要的是，阅读优秀的代码。 因此，任何软件公司、初创公司或个人开发人员都需要定期和精确的代码审查。
关注代码审查的技术原因很清楚。 但是从业务角度来看代码审查有什么好处呢？ 什么是好的代码审查流程？ 它对应用程序现代化有什么影响？ 让我们深入探讨所有这些主题。
什么是代码审查
让我们从一个简短的定义开始。 什么是代码审查？ 代码审查是软件开发服务中的一个过程，开发人员提交他们的代码以供其他开发人员或团队成员审查。 审阅者检查代码是否存在错误、逻辑错误、编码标准和最佳实践。 代码审查的主要目标是提高软件质量，在开发团队内提供良好的知识流动，并在公司内培养知识共享文化。
什么是好的代码审查过程
简而言之，一个好的代码审查应该是：
- 有建设性的
- 有限的
- 有启发性的。
但是，这在实践中意味着什么？
最重要的是，经验丰富的开发人员应该执行良好的代码审查过程。 它不仅可以指导经验不足的初级或普通开发人员并简化组织内的知识流动，而且还可以保证对错误和不良软件开发实践具有更高的敏感性。 换句话说，良好的代码审查是一种双赢的局面——对开发人员及其团队以及客户都是如此。
关于良好的代码审查流程，您还应该了解哪些内容？
建设性的代码审查需要时间
建设性的事情需要很长时间，这是显而易见的。 在代码审查的情况下也没有什么不同。 作为一个拥有既定代码审查文化的软件开发公司的后端团队负责人，每当我听到类似“我从上到下查看代码，看起来还不错”之类的话时，我就知道这种代码审查不是建设性的– 因此，应该重复。 自由职业者和外包开发人员往往会忘记在他们的估算中计算代码审查所需的时间。 因此，确保在项目开始之前就在软件开发过程中计算代码审查是一个很好的做法（尤其是在选择时间和材料定价模型时）。
代码审查增加了功能交付时间
有时，给定的功能会像乒乓球一样不断地被记者和审阅者抛出。 那不好吗？ 它不是！ 至少在一定程度上。 在处理建设性反馈时，双方会在某些主题上存在分歧。 但是，如果代码审查转变为对解决方案的持续争论，影响代码交付时间和 sprint 收尾——是时候进行干预了。 如果出现利益冲突，您应该要求您的软件开发合作伙伴进行结对编程。
软件开发团队的良好代码审查实践
想象一下有两个影评人：一个专门研究喜剧，另一个专门研究犯罪电影。 他们执行类似的活动：分析电影的情节、人物和视觉元素。 同时，他们使用不同的方法——因此关注不同的细节。 这同样适用于程序员。 两个不同的开发人员可能并不总是能够以相同的形式、质量甚至审查水平来进行代码审查。 然而，作为在同一技术行业工作的人，他们有义务遵循类似的良好代码审查实践。 在进行代码审查时，您应该注意几件事，以确保代码的质量。
那么，在贵公司建立代码审查文化时需要重点关注哪些关键领域？
设置代码审查的时间限制
很难在这里定义一个具体的数字。 执行代码审查所需的时间可能因多种因素而异，例如代码的数量和复杂性、审查人员的经验水平和代码的质量。 显然，花必要的时间执行彻底的代码审查很重要。 然而，看到程序员花几个小时分析别人的解决方案会让我有点怀疑。
聘请经验不足的开发人员
代码审查过程是双赢的。 它帮助作者，他的技能得到审查，代码审查者通过阅读和分析代码学习如何提供反馈并发展他的编程技能。 因此，让初级程序员参与这个有价值的过程至关重要。 通常，新鲜血液会带来新鲜的想法和令人兴奋的软件开发解决方案。
利用同行评审
同行评审应该是您在代码评审过程中的首选。 同行评审员是经验丰富的开发人员，具有给定语言和框架的专业知识； 了解项目目标和要求。 假设您别无选择，必须聘请项目外的人员进行代码审查。 在这种情况下，您应该意识到此人（无论他们的技术知识和经验如何）对您的业务或产品了解不多。 这可能会导致代码审查无法在整个项目的上下文中对给定功能进行体面的功能分析。 虽然外部代码审阅者可能会分析整体代码质量，但可能会忽略与整个网站或应用程序正常运行相关的元素。
代码审查的商业利益
从业务角度来看，代码审查的主要目标之一是向最终用户提供完美的服务或功能。 但这就是代码审查所能提供的全部吗？ 良好代码审查的其他结果是什么？
边缘案例识别
代码审查帮助团队识别在开发过程中可能遗漏的边缘案例。 这有助于减少客户投诉的数量并提高客户满意度。 此外，尽早识别边缘情况可以降低修复代码中潜在错误的成本。
逻辑问题识别
代码审查允许开发人员识别即使在交付软件后也可能遗漏的逻辑问题。 在开发过程中识别此类问题可以降低修复它们的成本并防止客户投诉。 此外，它还可以帮助团队生成更高效和可维护的代码。
提高安全性并检测代码中的潜在威胁
代码审查对于确保软件安全并防止客户数据受到损害至关重要。 它有助于识别代码中的任何潜在安全威胁，这有助于降低与修复错误和漏洞相关的成本，并最大限度地降低客户数据被盗或泄露的风险。
此外，它有助于确保代码遵守编码标准和最佳实践，并确定改进机会。 审查过程有助于确保代码具有最高质量，并且安全可靠。 此外，它还为开发人员提供有关其代码的宝贵反馈，并有助于确定需要进一步关注的任何领域。
就实施征求第二意见
获得关于实施的第二意见可以帮助识别任何潜在的问题或改进的领域，并帮助开发人员更好地理解代码。 此外，它还有助于确保代码具有最高质量并遵守编码标准和最佳实践。 此外，它还为开发人员提供有关其代码的宝贵反馈，并有助于确定需要进一步关注的任何领域。
团队之间积极的知识共享
代码审查带来双重好处：订阅者从审查中获得有价值的知识，审查者也可以从订阅者那里学到新东西。 这就是为什么流程的这一部分通常对双方都是双赢的。 此外，有关业务功能的知识会在订阅者和评论者之间自动传播。 因此，可以肯定地说，代码审查确保了开发团队对业务的出色理解。
改善团队成员之间的合作
让多人参与代码审查过程：代码的作者、同行审查者、技术主管以及初级和高级开发人员可以确保代码的高质量，同时也提供了知识共享和协作的机会。 基于团队内部的相互信任和尊重，执行良好的代码审查可以培养持续改进和团队合作的文化。
为什么代码审查是软件开发的重要组成部分
代码审查在节省时间和成本以及提高客户满意度方面为企业带来许多优势。 它有助于识别边缘案例和逻辑问题，提高代码中的安全性和潜在威胁，获得关于实施的第二意见，并改善团队合作。 此外，它还有助于确保代码具有最高质量并遵守最佳编码标准和实践。 通过将代码审查作为开发过程的一部分，您可以确保您的项目步入正轨并产生高质量的产品。
代码审查可以改善团队内部的沟通，简化错误识别，简化开发，并帮助公司在预算范围内按时交付项目。 如果您期待开发软件产品，请在您的开发过程中包括代码审查，并遵循上述清单以使该过程简单有效。