代码Review那些事

===========

  1. 在代码提交之前如果有很多双眼睛盯着看可以发现bug,这是代码审查最广为人知的好处。(人们的确可以在代码审查中发现bug,但是这些bug大部分都是显而易见的小bug,开发者分分钟可以发现,而那些真正需要花时间发现的bug通常是在代码审查中发现的)

  2. 代码审查最大的好处是纯社会性的。(如果你编程的时候知道你的同事将要看你的代码,你的编程方式会不一样,你的代码会写的更整洁,注释更加清楚,组织得更好。因为你知道其他人会看你的代码,他们的意见是你需要关注的。如果没有审查,你虽然知道人们最后会去看你的代码,但是那样不会给你一种紧迫感,也不会给你同样的个人评判的感觉。)

  3. 还有一个更大的好处就是代码审查可以传播知识。(在很多开发小组里,每个人都负责某一个核心组件,专注于自己的这一块,只要其他同事的模块不会破坏自己的代码就不会去关注,这种模式导致一个模块只有一个人熟悉对应的代码,如果一个人请教或者离职,其他人对他负责的模块将一无所知。如果采用代码审查,那么至少有两个人熟悉代码-作者和审查者。审查者知道的代码不如作者多,但是他们都熟悉代码的设计和结构,这意义重大)

Code Review的前提

==============

  1. 重视代码review

(Code Review人员是否理解了Code Review的概念和Code Review将做什么如果做Code Review的人员不能理解Code Review对项目成败和代码质量的重要程度,他们的做法可能就会是应付了事。)

  1. 代码是否已经正确的build,build的目的使得代码已 经不存在基本语法错误

(我们总不希望高级开发人员或是主管将时间浪费在检查连编译都通不过的代码上吧。 )

  1. 代码执行时功能是否正确

(Code Review人员也不负责检查代码的功能是否正确,也就是说,需要复查的代码必须由开发人员或质量人员负责该代码的功能的正确性。 )

  1. 开发人员是否对代码做了单元测试

(这一点也是为了保证Code Review前一些语法和功能问题已经得到解决,Code Review人员可以将精力集中在代码的质量上。 )

Code Review需要注意什么?

==================

  1. 完整性检查(Completeness)
  • 代码是否完全实现了设计文档中提出的功能需求

  • 代码是否已按照设计文档进行了集成和Debug

  • 代码是否已创建了需要的数据库,包括正确的初始化数据

  • 代码中是否存在任何没有定义或没有引用到的变量、常数或数据类型

  1. 一致性检查(Consistency)
  • 代码的逻辑是否符合设计文档

  • 代码中使用的格式、符号、结构等风格是否保持一致

  1. 正确性检查(Correctness)
  • 所有的变量都被正确定义和使用

  • 所有的注释都是准确的

  • 所有的程序调用都使用了正确的参数个数

  1. 可修改性检查(Modifiability)
  • 代码涉及到的常量是否易于修改(如使用配置、定义为类常量、使用专门的常量类等)

  • 代码是否只有一个出口和一个入口(严重的异常处理除外)

  1. 健壮性检查(Robustness)

  2. 可理解性检查(Understandability)

  • 注释是否足够清晰的描述每个子程序

  • 是否使用到不明确或不必要的复杂代码,它们是否被清楚的注释

  • 使用一些统一的格式化技巧(如缩进、空白等)用来增强代码的清晰度

  • 是否在定义命名规则时采用了便于记忆,反映类型等方法

  • 每个变量都定义了合法的取值范围

  • 代码中的算法是否符合开发文档中描述的数学模型

  1. 可验证性检查(Verifiability)
  • 代码中的实现技术是否便于测试

Code Review经验检查项

================

1、 编码规范方面检查项

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

- 类设计和抽象是否合适

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

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

3、性能方面检查项

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

- 有无滥用String对象的现象

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

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

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

4、数据库处理方面

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

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

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

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

5、通讯方面检查项

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

6、重复代码

7、其他

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

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

怎么更有效的做Code Review

==================
自我介绍一下,小编13年上海交大毕业,曾经在小公司待过,也去过华为、OPPO等大厂,18年进入阿里一直到现在。

深知大多数初中级Android工程师,想要提升技能,往往是自己摸索成长或者是报班学习,但对于培训机构动则近万的学费,着实压力不小。自己不成体系的自学效果低效又漫长,而且极易碰到天花板技术停滞不前!

因此收集整理了一份《2024年Android移动开发全套学习资料》,初衷也很简单,就是希望能够帮助到想自学提升又不知道该从何学起的朋友,同时减轻大家的负担。

img

img

img

img

既有适合小白学习的零基础资料,也有适合3年以上经验的小伙伴深入学习提升的进阶课程,基本涵盖了95%以上Android开发知识点,真正体系化!

由于文件比较大,这里只是将部分目录截图出来,每个节点里面都包含大厂面经、学习笔记、源码讲义、实战项目、讲解视频,并且会持续更新!

如果你觉得这些内容对你有帮助,可以扫码获取!!(备注:Android)

总结

本文讲解了我对Android开发现状的一些看法,也许有些人会觉得我的观点不对,但我认为没有绝对的对与错,一切交给时间去证明吧!愿与各位坚守的同胞们互相学习,共同进步!

《Android学习笔记总结+移动架构视频+大厂面试真题+项目实战源码》,点击传送门即可获取!

oPEF.jpg" />

总结

本文讲解了我对Android开发现状的一些看法,也许有些人会觉得我的观点不对,但我认为没有绝对的对与错,一切交给时间去证明吧!愿与各位坚守的同胞们互相学习,共同进步!

《Android学习笔记总结+移动架构视频+大厂面试真题+项目实战源码》,点击传送门即可获取!
  • 30
    点赞
  • 12
    收藏
    觉得还不错? 一键收藏
  • 0
    评论

“相关推荐”对你有帮助么?

  • 非常没帮助
  • 没帮助
  • 一般
  • 有帮助
  • 非常有帮助
提交
评论
添加红包

请填写红包祝福语或标题

红包个数最小为10个

红包金额最低5元

当前余额3.43前往充值 >
需支付:10.00
成就一亿技术人!
领取后你会自动成为博主和红包主的粉丝 规则
hope_wisdom
发出的红包
实付
使用余额支付
点击重新获取
扫码支付
钱包余额 0

抵扣说明:

1.余额是钱包充值的虚拟货币,按照1:1的比例进行支付金额的抵扣。
2.余额无法直接购买下载,可以购买VIP、付费专栏及课程。

余额充值