代码走查(Code Review)25条疑问

        这25条疑问就是代码规范的修炼手册,这25条疑问是别人总结出来的,记录在此用于提醒自己。养成在实现业务需求后,及时走查代码,重构完善。

  1. 代码的注释与代码是否一致?注释是否是多余的?
  2. 是否存在超过3层嵌套的循环与/或判断?
  3. 变量的命名是否代表了其作用?
  4. 所有的循环边界是否正确?
  5. 所有的判断条件边界是否正确?
  6. 输入参数的异常是否处理了?
  7. 程序中所有的异常是否处理了?
  8. 是否存在重复的代码?
  9. 是否存在超过25行的方法?
  10. 是否存在超过7个方法的类?
  11. 方法的参数是否超过3个?
  12. 是否有多种原因导致修改某个类?
  13. 当发生某个功能变化时,是否需要修改多个类?
  14. 代码中的常量是否合适?
  15. 一个方法是否访问了其他类的多个属性?
  16. 某几项数据是否总是同时出现,而又不是一个类的属性?
  17. switch语句是否可以用类来替代?
  18. 是否有一类的职责很少?
  19. 是否有一个类的某些属性或者方法没有被其他类所使用?
  20. 在类的方法中是否存在如下的调用形式:a.b().c()?
  21. 是否某个类的方法总是调用另外一个类的同名方法?
  22. 是否某个类总是访问另外一个类的属性与方法?
  23. 是否两个类完成了类似的工作,使用了不同的方法名,却没有拥有同一个父类?
  24. 是否某个类仅有字段和简单的赋值方法与取值方法构成?
  25. 是否某个子类仅使用了父类的部分属性或方法?

        通过代码走查可以提高代码的质量,同时减少Bug出现的几率。但是在小公司中并没有代码走查的过程在这里总结和记录一些代码走查的要求,时刻提醒自己注重代码质量。

  • 1
    点赞
  • 3
    收藏
    觉得还不错? 一键收藏
  • 0
    评论

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

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

请填写红包祝福语或标题

红包个数最小为10个

红包金额最低5元

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

抵扣说明:

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

余额充值