代码复审1234跟我一起来

代码复审

标签： 软件工程 代码复审 结对作业

---

代码复审是软件开发过程中必不可少的环节，目的在于发现代码中潜在的错误并在成员之间传授代码经验，同时也能增进团队成员之间的相互了解。同伴复审是其中相对简便易行的方式。这次，我的工作是复审我的队友刘宇帆的词频统计代码；

复审前的准备：

重新明确此次代码的需求，并和开发人员讨论代码的基本实现流程，运行代码用基本数据测试通过；

复审流程：

1. 阅读代码，查看代码的实现是否和讨论的算法一致；
2. 验证代码是否能符合作业需求
3. 从代码的可读性上进行评价；
4. 代码是否有冗余的部分；
5. 测试代码性能，提出改进意见；

复审结果：

1. 从算法实现来看，基本能够实现当时讨论时开发人员的算法思路；
2. 代码能支持简单的数据，但没有支持大小写的判断；
3. 代码可读性差，没有必要的注释，在没有开发人员解释的情况下阅读代码很花时间；
4. 代码中的冗余部分较多，没有用到继承，有大量类似的代码；
5. 代码性能瓶颈为正则表达式的大量判断，当时认为正则的使用会话费大量时间，建议重写匹配的代码，后来发现自己写的匹配代码不但实现复杂而且性能很低，只好改进其他方面；

总结：

缺点：

• 在可读性上还有待提高；
• 没有全方面支持需求和规格；
• 由于冗余度较高，在维护上有很大难度；

优点：

• 函数式编程，在整体代码结构布局的方面值得学习；
• 灵活运用了STL，代码更加简洁；