打印
[其它应用]

高效执行代码审查的3个技巧

[复制链接]
886|2
手机看帖
扫描二维码
随时随地手机跟帖
跳转到指定楼层
楼主
duo点|  楼主 | 2024-2-24 15:59 | 只看该作者 回帖奖励 |倒序浏览 |阅读模式
本文将探讨一些技巧,帮助大家实现代码审查现代化,从而获得开发人员的认可,并提高工作效率。
进行代码审查是发现bug、共享知识和创建高质量产品的有效机制。不幸的是,大多数开发人员宁愿等着报错也不愿参加代码审查。因为他们常常对代码审查感到痛苦并且感觉毫无成效。您是否曾试图培养团队的代码审查习惯,却发现几周后就不了了之?或者您对代码审查投入的时间并没有获得回报?
在今天的文章中,我们将探讨一些技巧,帮助您实现代码审查现代化,从而获得开发人员的认可并提高您的效率。
技巧1:“离线”执行代码审查
我一直都不太喜欢传统的代码审查。这些“在线”代码审查需要让一队开发人员在同一个房间里通读代码。应该有一个记录员、一个阅读者和各种其他角色。阅读者陈述代码应该做什么,然后开始逐行阅读,而十几个同事坐在旁边听,如果他们可以保持清醒,偶尔也会提供反馈意见。
在线代码审查存在很多问题。例如,当许多团队成员共处一室时,要想提高效率就需要付出努力。我们很可能会走神,不得不不断把话题拉回到审查上来。为了获得成功,我们应该限制花在审查代码上的时间,并且每次审查不超过一个小时。为了达到最佳效果,我们应该限制审查的代码行数不超过400行。
这些都是很好的指导原则,并且已被证明是有效的。但是,当12名团队成员每天生成100个LOC(可执行代码行数)时,您该怎么办?我们需要团队中的每个人每天进行三小时的代码审查!在某些时候,人力成本和投资回报是相互竞争的。
除在线代码审查外,另一种方法是离线审查。与其把所有人召集到一个房间,不如使用软件管理框架进行代码审查。我最喜欢的做事方法之一就是在合并请求之前不执行代码审查。您和您的团队应该经常进行提交和合并代码。合并请求不正是让团队成员审查代码、添加评论和反馈的最佳时机吗?
在合并请求时,我们可以允许团队成员在合并截止日期之前的闲暇时检查代码。开发人员就有机会在合并代码之前做出改进。希望他们能够验证行为并提前找出所有bug。
技巧2:为“离线”审查编写代码并进行注释
在进行代码审查时,通常会遵循一个标准程序。通常包括:

从代码模块的概述开始,了解其目的和功能;

逐行审查代码,检查语法错误、逻辑问题以及是否符合编码标准;

找出潜在的bug、安全漏洞或需要改进的地方。
如果您在编写和注释代码时考虑到代码审查,那么您的团队在合并请求时执行离线审查会容易得多。例如,您对模块及其用途的概述应该记录在您的模块中!该函数的目的是什么,以及您在编写该函数时做出的决定都应记录在案!

如果您编写的代码可以轻松地进行代码审查,您会发现您的团队无需聚集在同一个房间就可以审查代码。他们可以在您的管理工具中发表评论,您可以在提交最终合并请求之前收集这些评论并进行修改。
技巧3:分配小型审查团队
单个开发人员都希望审查代码库中的每一行代码。这有助于填补系统每一部分的细节。然而,对于当今的许多嵌入式产品来说,系统太大且太复杂,无法实现这一点。相反,对于团队来说,确保每一行代码都经过审查更有意义(但仅限某些人)。
将代码审查流程分解为若干个小型团队是确保产品中的每一行代码都得到审查的绝佳方法,但也有时每个人都要进行审查。开发人员A的代码可能会由开发人员B和C进行审核。同时,开发人员B会由开发人员C和D进行审核,依此类推。诚然,每个审查模块的开发人员都会提供独特的见解,但我们可以通过将每段代码的审查人数限制在合理的范围内,来更有效地执行代码审查。
平衡审查特定模块的团队规模可以帮助获得代码审查的好处,同时不会让开发团队陷入困境。有助于确保团队仍在生产代码的同时,确保所生产的代码具有合理的质量。
额外的代码审查技巧:
嵌入式软件中代码审查的需求至关重要,但在我看来,代码审查是团队最容易丢弃的第一个也是最直接的流程。前面的三个技巧让您深入了解了如何简化流程,同时从代码审查中获得见解和好处。
代码审查非常重要,因此我认为向您提供一些额外的提示会很有用:

营造建设性反馈的环境,反馈并不是对开发人员工作能力的评判;

为代码的审查、修正和确定设定时限;

利用人工智能工具提供最初的反馈列表(这可能是未来代码审查的方式);

除非代码通过了测试,否则不要审查代码;

除非代码已通过代码样式检查,否则不要审查代码;

让代码审查成为你日常习惯的一部分(只要有合并请求);

享受乐趣。代码审查是与队友积极互动和学习的好机会,不要以为自己无所不知!

结论:
代码审查可以提高代码质量、可读性、可维护性和团队协作。传统的代码审查方法往好了说是痛苦的,往坏了说是浪费时间。其实大可不必如此。
在这篇文章中,我们探讨了一些技巧,您可以利用这些技巧来帮助改进执行代码审查的方式。通过正确编写代码并利用现代协作工具,您应该能够实现代码审查带来的所有好处,同时使它们更容易接受。没有借口了!如果您还没有代码审查流程,请使用这些技巧开始吧。如果您有代码审查流程,我希望这些技巧能帮助您加强代码审查。
祝你好运,编程愉快!

使用特权

评论回复
沙发
小夏天的大西瓜| | 2024-2-26 18:16 | 只看该作者
进行代码审查是发现bug、共享知识和创建高质量产品的有效机制。

使用特权

评论回复
板凳
中国龙芯CDX| | 2024-2-26 21:33 | 只看该作者
逐行审查代码,检查语法错误、逻辑问题以及是否符合编码标准;

使用特权

评论回复
发新帖 我要提问
您需要登录后才可以回帖 登录 | 注册

本版积分规则

379

主题

1429

帖子

1

粉丝