代码复审1234跟我一起来
代码复审
标签： 软件工程 代码复审 结对作业
代码复审是软件开发过程中必不可少的环节，目的在于发现代码中潜在的错误并在成员之间传授代码经验，同时也能增进团队成员之间的相互了解。同伴复审是其中相对简便易行的方式。这次，我的工作是复审我的队友刘宇帆的词频统计代码；
复审前的准备：
重新明确此次代码的需求，并和开发人员讨论代码的基本实现流程，运行代码用基本数据测试通过；
复审流程：
- 阅读代码，查看代码的实现是否和讨论的算法一致；
- 验证代码是否能符合作业需求
- 从代码的可读性上进行评价；
- 代码是否有冗余的部分；
- 测试代码性能，提出改进意见；
复审结果：
- 从算法实现来看，基本能够实现当时讨论时开发人员的算法思路；
- 代码能支持简单的数据，但没有支持大小写的判断；
- 代码可读性差，没有必要的注释，在没有开发人员解释的情况下阅读代码很花时间；
- 代码中的冗余部分较多，没有用到继承，有大量类似的代码；
- 代码性能瓶颈为正则表达式的大量判断，当时认为正则的使用会话费大量时间，建议重写匹配的代码，后来发现自己写的匹配代码不但实现复杂而且性能很低，只好改进其他方面；
总结：
缺点：
- 在可读性上还有待提高；
- 没有全方面支持需求和规格；
- 由于冗余度较高，在维护上有很大难度；
优点：
- 函数式编程，在整体代码结构布局的方面值得学习；
- 灵活运用了STL，代码更加简洁；
