代码评审

代码评审

程序员必备的代码审查(Code Review)清单

软件工程学院的研究表明,程序员们会犯15-20种常见的错误。所以,通过把这些错误加入到检查清单当中,你可以确保不论什么时候,只要这些错误发生了,你就能发现它们,并且可以帮助你杜绝这些错误。

代码能够工作么?它有没有实现预期的功能,逻辑是否正确等。所有的代码是否简单易懂?是否存在多余的或是重复的代码?代码是否尽可能的模块化了?是否有可以被替换的全局变量?是否有被注释掉的代码?循环是否设置了长度和正确的终止条件?是否有可以被库函数替代的代码?是否有可以删除的日志或调试代码?

安全

所有的数据输入是否都进行了检查(检测正确的类型,长度,格式和范围)并且进行了编码?在哪里使用了第三方工具,返回的错误是否被捕获?输出的值是否进行了检查并且编码?

是否有注释,并且描述了代码的意图?所有的函数都有注释吗?对非常规行为和边界情况处理是否有描述?第三方库的使用和函数是否有文档?数据结构和计量单位是否进行了解释?是否有未完成的代码?如果是的话,是不是应该移除,或者用合适的标记进行标记比如‘TODO’?

测试

代码是否可以测试?比如,不要添加太多的或是隐藏的依赖关系,不能够初始化对象,测试框架可以使用方法等。是否存在测试,它们是否可以被理解?比如,至少达到你满意的代码覆盖(code coverage)。单元测试是否真正的测试了代码是否可以完成预期的功能?是否检查了数组的“越界“错误?

你同样需要把特定语言中有可能引起错误的问题添加到清单中。

优化你的清单把使用清单作为你的起点,针对特定的使用案例,你需要对其进行优化。一个比较棒的方式就是让你的团队记录下那些在代码审查过程中临时发现的问题,有了这些数据,你就能够确定你的团队常犯的错误,然后你就可以量身定制一个审查清单。确保你删除了那些没有出现过的错误。(你也可以保留那些出现概率很小,但是非常关键的项目,比如安全相关的问题)。

有了一个好的清单,可以提高你在代码审查过程中发现的缺陷个数。这可以帮助你提高代码标准,避免质量参差不齐的代码审查。

参考资料

程序员必备的代码审查(Code Review)清单

相关推荐

365彩票所有官方app下载平台 个人征信不好会有哪些影响?征信有问题会怎么样?
365beat中文版 如何正确临摹?动漫手绘入门临摹教程
365彩票所有官方app下载平台 机械先驱这个英雄怎么样?

机械先驱这个英雄怎么样?

📅 08-12 👁️ 313