Code Review都有哪些坑


Code Review都有哪些坑,很多新手对此不是很清楚,为了帮助大家解决这个难题,下面小编将为大家详细讲解,有这方面需求的人可以来学习下,希望你能有所收获。代码审查是许多高效团队的工程实践。即使你的软件已经有很多优点了,但团队在做代码审查时仍然会遇到一些陷阱。这篇文章我讲向你介绍一些需要特别注意的陷阱,以防代码审查工作拖累你的团队。知道可能遇到的问题或陷阱,将会帮助你进行更加高效、有效的的代码审查工作。这是我们调查了900名微软员工后得到的结论。一个典型的使用工具进行的代码审查过程大致是这样的:开发者完成一段代码,她提交代码准备开始让别人review。然后她选择需要审查她的代码的人。审查代码的人开始查看代码并给出一些评论。作者按照这些评论完善代码。当所有人都觉得没问题了以后,代码就可以合并进代码库了。
代码审查并不总是一个平稳的过程这些步骤听起来像是 一个很平稳的过程,其实并不是,像其他所有事情一样,实际过程往往不如预期。在代码审查过程中,经常会遇到一些陷阱,这会降低整个审查代码的积极性。如果不能正确处理,代码审查会对整个团队的工作效率产生影响。所以让我们来看一下代码审查过程中究竟存在哪些坑。关键问题主要有两种:代码审查花费的时间和代码审查所能提供的价值。作者需要面对的一个最主要的陷阱就是及时收到回复。等待评论的过程中不能在代码中做其他工作是一个巨大的问题。即使开发者可以完成其他任务,如果代码审查工作耗时过长,也会对开发者对工作效率和满意度造成不好的影响。代码审查并不是代码审查人员唯一的任务。相反,它只是开发者日常工作之一(即使需要每天花费大量时间来做)。所以代码审查人员很可能在做其他工作,并且必须在开始代码审查之前停下或完成这些工作。如果时间不理想,或者代码审查人员在之前没有预料到这种变化,那么她可能在代码审查之前需要一点时间。远程团队还需要注意时差,这意味着代码审查可能花费更多的时间。对于作者和代码审查人员来说,时间都是有限的,如果团队想要开发者做代码审查工作,但又不把它计入工作时间,这是有很大问题的。

tweet
大量的代码更改非常难以审核,另外,如果审核人员对这部分代码并不熟悉,那这将是一场噩梦。大多数审核人员会面临无法理解代码修改的动机的困境。如果没有任何对修改的描述和解释,代码审查工作将会非常难以进行。研究表明如果审核人员不理解代码修改的目的,或者由于量大而不堪重负,那么她很难给出良好的建议。It’s just this big incomprehensible mess… then you can’t add any value because they are just going to explain it to you and you’re going to parrot back what they say.毫无疑问,花费时间去等待代码审查,却没有得到有用的反馈是一个常见的问题。虽然团队可能从知识分享中受益,但这样仍会降低开发者的动力和收益。审核人员不能给出有效建议的原因通常有这样几种审核人员不具备这方面专业知识审核人员没有时间去完整的审查代码审核人员没有理解修改的目的另一个在代码审查过程中很重要的问题叫做bikeshedding。它的意思是开发者专注于小问题的讨论(例如代码格式)而忽略了更严重的问题。导致这种行为的原因有很多,通常是由于审核人员并不理解改动的目的或者没有时间做代码审查。有时bikeshedding可能代表团队的动力出了问题。有时候会发生代码作者和审核人员或者是审核人员之间很难达成共识的情况。由于团队动力和这件事紧密 香港云主机相关,因此必须小心处理这种情况通过工具和书面形式只会加剧这种问题。如果要讨论有争议的问题,那么面对面或者视频也许是更好的方案。我希望这些陷阱不会改变你对代码审核的想法,因为如果你了解了这些陷阱并妥善处理它们,代码审核工作对于整个技术团队是非常有好处的,而且会有更有效的方法来做代码审核。看完上述内容是否对您有帮助呢?如果还想对相关知识有进一步的了解或阅读更多相关文章,请关注开发云行业资讯频道,感谢您对开发云的支持。

相关推荐: Redis分布式缓存的作用是什么

这篇文章主要介绍“Redis分布式缓存的作用是什么”,在日常操作中,相信很多人在Redis分布式缓存的作用是什么问题上存在疑惑,小编查阅了各式资料,整理出简单好用的操作方法,希望对大家解答”Redis分布式缓存的作用是什么”的疑惑有所帮助!接下来,请跟着小编一…

免责声明:本站发布的图片视频文字,以转载和分享为主,文章观点不代表本站立场,本站不承担相关法律责任;如果涉及侵权请联系邮箱:360163164@qq.com举报,并提供相关证据,经查实将立刻删除涉嫌侵权内容。

Like (0)
Donate 微信扫一扫 微信扫一扫
Previous 08/16 17:08
Next 08/16 17:08

相关推荐