一、实验目的
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代码中的错误，查找不符合代码风格标准和有潜在问题的代码。目前在eclipse的pydev插件中也集成了pylint，VSCcode可安装pylint插件
当发现了项目存在的问题后，可通过bug跟踪系统向项目维护者反馈问题（issue），管理issue的系统称为BTS。当今具有代表性的BTS有Redmine、Trac、BugZila等。GitHub自身也加入了BTS的功能。在GitHub上，可以将它作为软件开发者之间的交流工具。通过github的issues功能开发者可以便捷的发现软件的BUG并报告，想向项目所有人询问或用来追踪各种想法探讨准备实施的任务。
三、实验过程
代码评审项目地址 https://github.com/xiaoshi176/study
克隆到本地仓库
运行项目中的文件
运行正常，符合要求
经分析需要对项目中index.js文件进行代码评审，查阅资料了解到需要用到Nodejs软件。
1、配置代码审查工具。要求采用屏幕截图的方式配置的过程
Nodejs安装
官方地址 http://nodejs.org/
根据提示
安装完成
进入命令行，检查是否安装成功并配置环境变量，输入 npm install -g jshint 安装 Jshint 代码检查插件
2、使用工具对原始代码进行评审和分析，记录结果，期间不要有任何修改
命令行进入项目所在目录，输入 jshint xxxx.js(需审查的 js 文件)，进行代码审查，结果如下：
3、对工具执行结果进行人工分析，结合检查清单和人工走查的出代码修改建议
对结果进行分析发现主要存在的问题如下：
(1) Missing semicolon 缺少分号
(2) EOL转义错误，如果需要，请使用multistr
(3) 引用外部作用域变量的循环内声明的函数可能会导致语义混淆
4、通过github issues向项目维护者提交问题，注意一个issue只报告一个问题，多个问题需放在多个issue中，以便跟踪
通过github issues向项目维护者提交问题
5、记录总结实验过程中遇到的问题和解决过程
实验中，在Nodejs的安装与使用时我遇到了一些问题，主要在于Nodejs的配置和jshint插件的安装，通过查阅资料将问题逐个解决。
四、实验小结
通过本次实验，我认识到了代码评审的重要性，学习使用了代码审查工具。实验过程中，出现了不少错误，不过通过不断地调试软件总算基本解决。在审查代码的同时也发现了自己存在同样的问题，还需进一步学习，注意代码规范。