实验四 代码评审
一、实验目的
1) 了解代码审查的含义;
2) 掌握相关编程规范检查工具的安装与使用;
二、实验内容及要求
Code Review中文应该译作“代码审查”或是“代码评审”或“代码复查”,是指通过阅读代码来检查源代码与编码标准的符合性以及代码质量的活动。Code Review主要用来在软件工程过程中改进代码质量,通过代码评审可以达到如下目的:
●在项目早期就能够发现代码中的BUG
●帮助初级开发人员学习高级开发人员的经验,达到知识共享
●避免开发人员犯一些很常见,很普通的错误
●保证项目组人员的良好沟通
●项目或产品的代码更容易维护
代码评审主要内容是编程规范,重构方法,架构设计,性能安全,日志,可读性,扩展性等问题。通过代码评审可查找和修复引入到开发阶段的应用程序的错误,提高软件的整体素质和开发者的技能。代码评审的作用和意义已在很多技术团队内达成共识,可是很多时候并未被有效执行,甚至被认为是一项费时费力的工作。借助一些工具可以更容易,更有效率地来进行Code Review。
1、以小组形式,针对前面“实验一”中所完成的代码,进行代码评审(走查),重点检查以下情况。你也可有查询相关材料,建立更细化的检查清单(check list)
- 程序是否能正常工作,代码是否实现预期的功能,逻辑是否正确。
- 代码是否遵循的编程规范
- 代码是否尽可能的模块化
- 所有的数据输入是否都进行了检查
- 是否有注释,并且描述了代码的意图
- 代码的可理解性和可测试性
2、按“实验二”的分组方式,两人一组,随机分配另一组的代码作为本组评审和分析的对象
一些编码规范的检查工具如下,也可自行查找工具使用。
java语言
- 采用使用eclipse Checkstyle插件
- CheckStyle是SourceForge下的一个项目,提供了一个帮助JAVA开发人员遵守某些编码规范的工具。它能够自动化代码规范检查过程,从而使得开发人员从这项重要但枯燥的任务中解脱出来。它可以根据设置好的编码规则来检查代码。比如符合规范的变量命名,方法体的最大行数,重复代码检查等等。
C++语言
- 可使用Google代码规范工具Cpplint。
- Cpplint是一个python脚本,Google使用它作为自己的C++代码规范检查工具,VSCcode可配置Cpplint对C++代码进行规范检查。
python
- 可采用pylint
- Pylint 是一个 Python 代码分析工具,它分析 Python 代码中的错误,查找不符合代码风格标准(Pylint 默认使用的代码风格是 PEP 8,具体信息,请参阅参考资料)和有潜在问题的代码。目前在 eclipse 的 pydev 插件中也集成了 Pylint,VSCcode可安装pylint插件
当发现了项目存在的问题后,可通过Bug跟踪系统向项目维护者反馈问题(issue),管理Issue的系统称为BTS(Bug Tracking System,Bug跟踪系统)。当今具有代表性的BTS有Redmine、Trac、BugZilla等。GitHub自身也加入了BTS的功能。在GitHub上,可以将它作为软件开发者之间的交流工具。通过github的issues功能开发者可以便捷的发现软件的BUG并报告,想向项目所有人询问或用来追踪各种想法探讨准备实施的任务。
三、实验过程
1、检查小组项目
(1)、检查项目
被评审项目的 Github 地址:https://github.com/zhoudashuaibi/my-travel-plans
下面是仓库截图
2、配置代码审查工具
由于该项目是由C++完成的,可以使用Cpplint进行代码审查。
下载cpplint.py文件:https://github.com/google/styleguide并将其存放到D:\cpplint中;
在微信公众号(软件安装管家)下载python2.7.6版本
并将D:\电脑软件\Python2.7.6 添加到系统环境变量Path中。
3.使用工具对原始代码进行评审和分析(记录结果,期间不要有任何修改)
审查结果如下:
由上述所示共有216个错误,13种错误类型
序号 | 报错类型 | 具体错误 | 数量 | 置信度 |
1 | whitespace/braces | 方法吗好括号之间、“{“前缺少空格 | 12/8 | 4/5 |
2 | whitespace/tab | 没有使用四个空格代替缩进 | 62 | 1 |
3 | whitespace/semicolon | 分号后面缺少空格 | 15 | 3 |
4 | readability/utf8 | 使用了中文注释 | 12 | 5 |
5 | whitespace/blank_line | 代码块最后的空行应该被删除 | 5 | 3 |
6 | whitespace/end_of_line | 代码行最后存在空格 | 5 | 4 |
7 | whitespace/comma | ”,“缺少空格 | 9 | 3 |
8 | whitespace/operators | 运算符前后缺少空格 | 9/12 | 3/4 |
9 | whitespace/ending_newline | 在文件末尾找不到换行符 | 1 | 5 |
10 | whitespace/comments | 代码和注释之间至少两个空格 | 16 | 2 |
11 | whitespace/line_length | 存在大于80字符的行 | 1 | 2 |
12 | legal/copyright | 未找到版权信息 | 1 | 5 |
13 | whitespace/parent | 属性定义名为空 | 3 | 5 |
该程序只完成了部分功能,通过GitHub向第59组同学的项目提交issue。
四、实验小结
经过此次代码审查,我认识到了代码规范的重要性,一定要注意编辑规范的代码格式,要有良好的缩进和注释,不然给自己以后的维护和改良增加了很多困难,一个团队如果没有良好的代码规范,将会出现及其严重的后果,各组员之间没有了更好的衔接,将会出现一人犯错,期待人将无法修改的境地。其次,在代码审查时,我发现被审查的小组,提交在仓库里的代码是用文件形式,并没有用.c或者其他形式,在审查时,被审查的文件中也不能含有中文,否则会出错,最终将他们的程序名改成了四则运算.cpp才审查成功。以后再写代码时要注意代码规范。
第一次用的是python3.6.7版本,cpplint.py这个文件运行不出来,怎么都显示不了错误,如下图。最后又重装了一个python2.7.6版本的才能运行成功