码迷,mamicode.com
首页 > 其他好文 > 详细

实验四 代码评审

时间:2020-05-13 11:40:37      阅读:61      评论:0      收藏:0      [点我收藏+]

标签:iss   iba   ima   放大   soft   规则   相关   对象   check   

一、实验目

1) 了解代码审查的含义;

2) 掌握相关编程规范检查工具的安装与使用;

二、实验内容及要求

Code Review中文应该译作“代码审查”或是“代码评审”或“代码复查”,是指通过阅读代码来检查源代码与编码标准的符合性以及代码质量的活动。Code Review主要用来在软件工程过程中改进代码质量,通过代码评审可以达到如下目的:

●在项目早期就能够发现代码中的BUG

●帮助初级开发人员学习高级开发人员的经验,达到知识共享

●避免开发人员犯一些很常见,很普通的错误

●保证项目组人员的良好沟通

●项目或产品的代码更容易维护

     代码评审主要内容是编程规范,重构方法,架构设计,性能安全,日志,可读性,扩展性等问题。通过代码评审可查找和修复引入到开发阶段的应用程序的错误,提高软件的整体素质和开发者的技能。代码评审的作用和意义已在很多技术团队内达成共识,可是很多时候并未被有效执行,甚至被认为是一项费时费力的工作。借助一些工具可以更容易,更有效率地来进行Code Review。

1、以小组形式,针对前面“实验一”中所完成的代码,进行代码评审(走查),重点检查以下情况。你也可有查询相关材料,建立更细化的检查清单(check list)

- 程序是否能正常工作,代码是否实现预期的功能,逻辑是否正确。

- 代码是否遵循的编程规范

- 代码是否尽可能的模块化

- 所有的数据输入是否都进行了检查

- 是否有注释,并且描述了代码的意图

- 代码的可理解性和可测试性

2、按“实验二”的分组方式,两人一组,随机分配另一组的代码作为本组评审和分析的对象

一些编码规范的检查工具如下,也可自行查找工具使用。

java语言      

- 采用使用eclipse Checkstyle插件 

- CheckStyle是SourceForge下的一个项目,提供了一个帮助JAVA开发人员遵守某些编码规范的工具。它能够自动化代码规范检查过程,从而使得开发人员从这项重要但枯燥的任务中解脱出来。它可以根据设置好的编码规则来检查代码。比如符合规范的变量命名,方法体的最大行数,重复代码检查等等。

如果你使用idea ,可以使用Alibaba Java Code Guidelines插件,参考链接如下:https://github.com/alibaba/p3c/wiki/IDEA%E6%8F%92%E4%BB%B6%E4%BD%BF%E7%94%A8%E6%96%87%E6%A1%A3

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并报告,想向项目所有人询问或用来追踪各种想法探讨准备实施的任务。

三、实验过程

审查为Java程序,故安装Checkstyle

(1)配置代码审查工具

首先进入官网下载相应的审查工具插件,官网为:https://sourceforge.net/projects/eclipse-cs/files/Eclipse%20Checkstyle%20Plug-in/

技术图片

 

 

 下载后进行配置,打开eclipse,点击帮助点击安装新软件进入如下界面

技术图片

 

 

 点击添加,输入名称,位置选择刚才下载插件的安装包的位置

技术图片

 

 

 不停的点击下一步直到安装完成,安装好后需要重启软件,屏幕中出现放大镜为安装成功

技术图片

(2)使用工具对原始代码进行评审和分析,记录结果

大概统计了以下类型的一些错误

技术图片技术图片技术图片

代码中出现了很多不规范的情况,因为没有找到checkstyle中是如何对错误进行统计的,只能人工进行统计

技术图片

 

 

 (3)对工具执行结果进行人工分析,结合检查清单和人工走查的出代码修改建议

分析:

1、代码中的错误很多,尤其是制表符和缩进符使用不当  

2、代码的错误虽然多但错误的种类少,主要还是体现在日常写代码时不注意规范导致的错误

修改建议:

1、写代码时先讨论是使用空格还是制表符来规范代码

2、在自定义的变量中,要使用Javadoc进行注释

3、不要在一行写太长的代码,若代码过长时要注意换行

4、使用if和for时即使是一行代码也需要加{}

(4)通过github issues向项目维护者提交问题(issue)

技术图片

 

 

 (5)记录总结实验过程中遇到的问题和解决过程

1、安装Checkstyle时,不知道该怎样安装插件,解决过程:在网上搜了一个比较简单的方法,在eclipse中直接可以将插件安装到其中,只需要找到下载的插件所在的文件夹,在安装到eclipse中就可以运行。

2、在运行代码时,出现了中文乱码,解决过程: 

1) eclipse->window->preferences->General->Content Types

2) 找到要修改的文件的类型(JAVA),在下面有个Default encoding,在输入框中输入GBK

3) 点击Update

四、实验小结

1、本次实验学习了怎么在eclipse中使用插件Checkstyle来审查代码

2、在审查别人的代码时,发现自己的写的代码十分的不正规,以后在写代码时要注意代码的规范情况

3、使用代码审查可以避免一些十分普遍的小错误,改善代码的质量

实验四 代码评审

标签:iss   iba   ima   放大   soft   规则   相关   对象   check   

原文地址:https://www.cnblogs.com/19990302rx/p/12881140.html

(0)
(0)
   
举报
评论 一句话评论(0
登录后才能评论!
© 2014 mamicode.com 版权所有  联系我们:gaon5@hotmail.com
迷上了代码!