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

实验四 代码审查

时间:2020-05-17 01:12:31      阅读:83      评论:0      收藏:0      [点我收藏+]

标签:避免   red   系统   技术   bsp   redmin   lips   tps   跟踪系统   

一、实验目的

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

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

二、实验内容及要求

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

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

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

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

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

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

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

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

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

代码是否遵循的编程规范

代码是否尽可能的模块化

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

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

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

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

  当发现了项目存在的问题后,可通过Bug跟踪系统向项目维护者反馈问题(issue),管理Issue的系统称为BTS(Bug Tracking System,Bug跟踪系统)。当今具有代表性的BTS有Redmine、Trac、BugZilla等。GitHub自身也加入了BTS的功能。在GitHub上,可以将它作为软件开发者之间的交流工具。通过github的issues功能开发者可以便捷的发现软件的BUG并报告,想向项目所有人询问或用来追踪各种想法探讨准备实施的任务。

  一些编码规范的检查工具如下,也可自行查找工具使用。本次实验使用eclipse的checkStyle插件来完成代码审查

三、实验过程

1、配置代码审查工具。要求采用屏幕截图的方式配置的过程;

     1.1确定所需要的代码审工具:(审查项目链接:https://github.com/pxw3170701339/pp_project.git

        在GitHub中先clone我需要进行审查的项目。经clone该项目为python 语言,故使用pylint插件审查。

技术图片

 

 技术图片

 

技术图片

 

 

 1.2下载安装cpplint工具,查阅并使用。

2、代码审查

技术图片

 

 技术图片

 

 技术图片

 

 技术图片

 

 技术图片

 

 3、对主要问题进行分析

1)在in if (之前 缺少空格

2)在,之后缺少空格

3)有些地方存在异常缩进,应正确使用缩进

4)Found C system header after C++ system header. Should be:四则运算.h, C system, c++ syste

5)代码和注释之间的空格不对,应当使用正确的空格数。

4、向github提交issues

技术图片

 

5、下一组成员解决问题。

四、实验小结

   本次实验是对第二次实验结对编程代码规范的审查,本次实验中收获良多。现在,对于一个团队来说,代码的规范往往比写出一个代码更加重要。在本次实验对代码进行测试和检测对于今后的学习和工作有了很大帮助。

 

 

 

 

实验四 代码审查

标签:避免   red   系统   技术   bsp   redmin   lips   tps   跟踪系统   

原文地址:https://www.cnblogs.com/Jiangxing123/p/12903011.html

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