一、结对,找到一个伙伴进行结对
张剑斌 :http://www.cnblogs.com/janbin/p/8594002.html
二、代码复审核查表
代码复审核查表 |
||
概要部分 |
代码符合需求和规格说明么? |
符合 |
代码设计是否考虑周全? |
否 |
|
代码可读性如何? |
一般 |
|
有冗余的或重复的代码吗? |
无 |
|
代码的每一行都执行并检查过了吗? |
是 |
|
设计规范部分 |
设计是否遵从已知的设计模式或项目中常用的模式? |
不遵循 |
有没有硬编码或字符串/数字等存在? |
有 |
|
代码有没有依赖于某一平台? |
没有 |
|
有没有无用的代码可以清除? |
有 |
|
代码规范部分 |
修改的部分符合代码标准和风格么? |
基本符合 |
具体代码部分 |
数据结构中有没有用不到的元素? |
无 |
对于调用的外部函数,是否检查了返回值 |
是 |
|
效能 |
代码的效能如何? |
一般 |
循环中是否有明显可优化的部分? |
是 |
|
可读性 |
代码可读性如何?有没有足够的注释? |
可读性强,但没有注释 |
可测试性 |
是否需要更新或创建新的单元测试 |
是 |
代码复审感想:编写规范并不是很好,一个注释都没有,不易阅读。当测试邮箱有多处错误时,只能报出来一个错误,也不易于用户的修改。通过这次复审,感觉自己平常编程的时候很多地方没有注意,小项目可能影响不大,但是大项目尤其重视代码规范,特别是多人开发的时候,你自己的随心所欲可能会让其他小伙伴难受,为了大家编程都舒服,在项目开始的时候就要定好一个代码规范。 |