正视代码Review,养成良好的编程习惯

2024-06-16 13:08

本文主要是介绍正视代码Review,养成良好的编程习惯,希望对大家解决编程问题提供一定的参考价值,需要的开发者们随着小编来一起学习吧!

什么是代码Review?

代码review是指在软件开发过程中,通过对源代码进行系统性检查来确认代码实现的质量保证机制

为什么不做代码Review?

  • ​业务需求大,工作时间紧张
  • 项目小,协作的人少,没必要

为什么要做代码Review?

  • 提高代码质量,提升自身水平
  • 及早发现潜在缺陷与BUG,降低事故成本
  • 促进团队内部知识共享,提高团队整体水平
  • 保证项目组人员的良好沟通
  • 避免开发人员犯一些很常见,很普通的错误

总而言之目的是查找系统缺陷,保证软件总体质量和提高开发者自身水平,使项目代码更加容易维护。

代码Review的好处

  1. 在代码提交之前如果有很多双眼睛盯着看可以发现bug,这是代码审查最广为人知的好处。(人们的确可以在代码审查中发现bug,但是这些bug大部分都是显而易见的小bug,开发者分分钟可以发现,而那些真正需要花时间发现的bug通常是在代码审查中发现的)
  2. 代码审查最大的好处是纯社会性的。(如果你编程的时候知道你的同事将要看你的代码,你的编程方式会不一样,你的代码会写的更整洁,注释更加清楚,组织得更好。因为你知道其他人会看你的代码,他们的意见是你需要关注的。如果没有审查,你虽然知道人们最后会去看你的代码,但是那样不会给你一种紧迫感,也不会给你同样的个人评判的感觉。)
  3. 还有一个更大的好处就是代码审查可以传播知识。(在很多开发小组里,每个人都负责某一个核心组件,专注于自己的这一块,只要其他同事的模块不会破坏自己的代码就不会去关注,这种模式导致一个模块只有一个人熟悉对应的代码,如果一个人请教或者离职,其他人对他负责的模块将一无所知。如果采用代码审查,那么至少有两个人熟悉代码-作者和审查者。审查者知道的代码不如作者多,但是他们都熟悉代码的设计和结构,这意义重大)

Code Review的前提

  1. 重视代码review
    (Code Review人员是否理解了Code Review的概念和Code Review将做什么如果做Code Review的人员不能理解Code Review对项目成败和代码质量的重要程度,他们的做法可能就会是应付了事。)
  2. 代码是否已经正确的build,build的目的使得代码已 经不存在基本语法错误
    (我们总不希望高级开发人员或是主管将时间浪费在检查连编译都通不过的代码上吧。 )
  3. 代码执行时功能是否正确
    (Code Review人员也不负责检查代码的功能是否正确,也就是说,需要复查的代码必须由开发人员或质量人员负责该代码的功能的正确性。 )
  4. 开发人员是否对代码做了单元测试
    (这一点也是为了保证Code Review前一些语法和功能问题已经得到解决,Code Review人员可以将精力集中在代码的质量上。 )

Code Review需要注意什么?

  1. 完整性检查(Completeness)
    • 代码是否完全实现了设计文档中提出的功能需求
    • 代码是否已按照设计文档进行了集成和Debug
    • 代码是否已创建了需要的数据库,包括正确的初始化数据
    • 代码中是否存在任何没有定义或没有引用到的变量、常数或数据类型
  2. 一致性检查(Consistency)
    • 代码的逻辑是否符合设计文档
    • 代码中使用的格式、符号、结构等风格是否保持一致
  3. 正确性检查(Correctness)
    • 所有的变量都被正确定义和使用
    • 所有的注释都是准确的
    • 所有的程序调用都使用了正确的参数个数
  4. 可修改性检查(Modifiability)
    • 代码涉及到的常量是否易于修改(如使用配置、定义为类常量、使用专门的常量类等)
    • 代码是否只有一个出口和一个入口(严重的异常处理除外)
  5. 健壮性检查(Robustness)
  6. 可理解性检查(Understandability)
    • 注释是否足够清晰的描述每个子程序
    • 是否使用到不明确或不必要的复杂代码,它们是否被清楚的注释
    • 使用一些统一的格式化技巧(如缩进、空白等)用来增强代码的清晰度
    • 是否在定义命名规则时采用了便于记忆,反映类型等方法
    • 每个变量都定义了合法的取值范围
    • 代码中的算法是否符合开发文档中描述的数学模型
  7. 可验证性检查(Verifiability)
    • 代码中的实现技术是否便于测试

Code Review经验检查项

1、 编码规范方面检查项

2、面向对象设计方面检查项

- 类设计和抽象是否合适

- 是否符合面向接口编程的思想

- 是否采用合适的设计模式

3、性能方面检查项

- 对hashtable,vector等集合类数据结构的选择和设置是否合适

- 有无滥用String对象的现象

- 是否采用通用的线程池、对象池模块等cache技术以提高性能

- I/O方面是否使用了合适的类或采用良好的方法以提高性能(如减少序列化,使用buffer类封装流等)

- 同步方法的使用是否得当,是否过度使用

4、数据库处理方面

- 数据库资源是否正常关闭和释放

- 数据库访问模块是否正确封装,便于管理和提高性能

- 是否采用合适的事务隔离级别

- 资源泄漏处理方面检查项 cursor

5、通讯方面检查项

- socket通讯是否存在长期阻塞问题

6、重复代码

7、其他

- 日志是否正常输出和控制

- 配置信息如何获得,是否有硬编码

怎么更有效的做Code Review

  1. 一次评审量要低于 200–400 行代码缺陷密度 就是每 1000 行代码之中所发现的错误(bug)数
    这里写图片描述
  2. 每小时低于 300–500 LOC 检查率的目标
    这里写图片描述
  3. 花足够的时间进行适当缓慢的评审,但是不要超过 60-90 分钟
    但反过来说,评审代码所花的时间不得低于五分钟,就算代码只有一行也是如此。通常来说,单行的代码也会影响到整个的系统,所以花上五分钟时间去检查更改可能造成的结果是值得的
  4. 确定在评审开始之前代码开发者已经注释源代码了
  5. 使用检查表,因为它能极大地影响代码开发者和评审者的结果
    另外一个有用的概念就是 个人检查表 。每个人一般都会犯 15-20 个错误(bug)。如果您注意到了一些典型的错误(bug),那么您就可以开发自己的个人检查表
  6. 确认缺陷得到了修复

最后,让Code Review成为一种习惯

转载:http://www.androidchina.net/4831.html

这篇关于正视代码Review,养成良好的编程习惯的文章就介绍到这儿,希望我们推荐的文章对编程师们有所帮助!



http://www.chinasem.cn/article/1066542

相关文章

java之Objects.nonNull用法代码解读

《java之Objects.nonNull用法代码解读》:本文主要介绍java之Objects.nonNull用法代码,具有很好的参考价值,希望对大家有所帮助,如有错误或未考虑完全的地方,望不吝赐... 目录Java之Objects.nonwww.chinasem.cnNull用法代码Objects.nonN

SpringBoot实现MD5加盐算法的示例代码

《SpringBoot实现MD5加盐算法的示例代码》加盐算法是一种用于增强密码安全性的技术,本文主要介绍了SpringBoot实现MD5加盐算法的示例代码,文中通过示例代码介绍的非常详细,对大家的学习... 目录一、什么是加盐算法二、如何实现加盐算法2.1 加盐算法代码实现2.2 注册页面中进行密码加盐2.

python+opencv处理颜色之将目标颜色转换实例代码

《python+opencv处理颜色之将目标颜色转换实例代码》OpenCV是一个的跨平台计算机视觉库,可以运行在Linux、Windows和MacOS操作系统上,:本文主要介绍python+ope... 目录下面是代码+ 效果 + 解释转HSV: 关于颜色总是要转HSV的掩膜再标注总结 目标:将红色的部分滤

在C#中调用Python代码的两种实现方式

《在C#中调用Python代码的两种实现方式》:本文主要介绍在C#中调用Python代码的两种实现方式,具有很好的参考价值,希望对大家有所帮助,如有错误或未考虑完全的地方,望不吝赐教... 目录C#调用python代码的方式1. 使用 Python.NET2. 使用外部进程调用 Python 脚本总结C#调

Java时间轮调度算法的代码实现

《Java时间轮调度算法的代码实现》时间轮是一种高效的定时调度算法,主要用于管理延时任务或周期性任务,它通过一个环形数组(时间轮)和指针来实现,将大量定时任务分摊到固定的时间槽中,极大地降低了时间复杂... 目录1、简述2、时间轮的原理3. 时间轮的实现步骤3.1 定义时间槽3.2 定义时间轮3.3 使用时

Java中&和&&以及|和||的区别、应用场景和代码示例

《Java中&和&&以及|和||的区别、应用场景和代码示例》:本文主要介绍Java中的逻辑运算符&、&&、|和||的区别,包括它们在布尔和整数类型上的应用,文中通过代码介绍的非常详细,需要的朋友可... 目录前言1. & 和 &&代码示例2. | 和 ||代码示例3. 为什么要使用 & 和 | 而不是总是使

Java强制转化示例代码详解

《Java强制转化示例代码详解》:本文主要介绍Java编程语言中的类型转换,包括基本类型之间的强制类型转换和引用类型的强制类型转换,文中通过代码介绍的非常详细,需要的朋友可以参考下... 目录引入基本类型强制转换1.数字之间2.数字字符之间引入引用类型的强制转换总结引入在Java编程语言中,类型转换(无论

Python异步编程中asyncio.gather的并发控制详解

《Python异步编程中asyncio.gather的并发控制详解》在Python异步编程生态中,asyncio.gather是并发任务调度的核心工具,本文将通过实际场景和代码示例,展示如何结合信号量... 目录一、asyncio.gather的原始行为解析二、信号量控制法:给并发装上"节流阀"三、进阶控制

Vue 调用摄像头扫描条码功能实现代码

《Vue调用摄像头扫描条码功能实现代码》本文介绍了如何使用Vue.js和jsQR库来实现调用摄像头并扫描条码的功能,通过安装依赖、获取摄像头视频流、解析条码等步骤,实现了从开始扫描到停止扫描的完整流... 目录实现步骤:代码实现1. 安装依赖2. vue 页面代码功能说明注意事项以下是一个基于 Vue.js

mybatis-plus 实现查询表名动态修改的示例代码

《mybatis-plus实现查询表名动态修改的示例代码》通过MyBatis-Plus实现表名的动态替换,根据配置或入参选择不同的表,本文主要介绍了mybatis-plus实现查询表名动态修改的示... 目录实现数据库初始化依赖包配置读取类设置 myBATis-plus 插件测试通过 mybatis-plu