进行代码复审训练
一、结对,找到一个伙伴进行结对;伙伴的博客链接:http://www.cnblogs.com/Chenjx-666/p/8558209.html
二、各自对自己的伙伴上周进行的“单元测试”练习所完成的代码进行复审,形成“代码复审检查表”。
概要部分 |
代码符合需求和规格说明么? |
大致符合 |
代码设计是否考虑周全? |
不周全 |
|
代码可读性如何? |
简单 |
|
有冗余的或重复的代码吗? |
无 |
|
代码的每一行都执行并检查过了吗? |
是 |
|
设计规范部分 |
设计是否遵从已知的设计模式或项目中常用的模式? |
否 |
有没有硬编码或字符串/数字等存在? |
有 |
|
代码有没有依赖于某一平台? |
有 |
|
有没有无用的代码可以清除? |
无 |
|
代码规范部分
|
修改的部分符合代码标准么? |
符合 |
修改的部分的设计是否规范? |
大致符合规范 |
|
具体代码部分 |
数据结构中有没有用不到的元素? |
无 |
对于调用的外部函数,是否检查了返回值? |
是 |
|
效能 |
代码的效能(Performance)如何? |
良好 |
代码中,特别是循环中是否有明显可优化的部分 |
无 |
|
可读性
|
有没有足够的注释? |
无注释 |
逻辑是否容易理解? |
是 |
|
段落间和符号旁有没有空白? |
无 |
|
可测试性 |
是否需要更新或创建新的单元测试? |
否 |
代码复审感想:
通过分析该代码相对简单易懂,代码相对较少,出现问题更易查找。尽管部分特殊情况没有考虑周全,相对而言在某一方面上来说代码审查本身提高开发者的能力,让其从自身犯过的错误中学习,从他人的思路中学习。在代码审查中也意识到自己存在的不足,例如没有做好注释,没有为读代码的人员考虑。代码复审的目的我认为除了检查代码存在的问题,也是一种提升自己在代码规范上的机会。尤其在软件开发中bug在所难免,过度苛求本身有悖常理。发现问题要立刻指出问题,不然代码审查就没有任何的价值和意义。