天天看点

敏捷代码审查指南

  我强烈赞同code review 是软件生命周期管理中重要的一部分,因为它能帮助我们交付高质量代码、合格作品。

敏捷代码审查指南

  code review的好处:

  俗话说三个臭皮匠赛过诸葛亮,code review 更易于发现代码bug等问题

  3、保证代码质量以及提高代码可读性

  2、团队之间建立信任

  1、指导初级程序员

  编码标准是独立于语言的,对于java 程序员来说,我想从以下几个范围来做code review

  java code review的标准:

  合适的变量声明;如:实例变量还是静态变量、常量等

  9、性能问题;如:当没有线程安全问题时使用arraylist,hashmap替代vector,hashtable

  8、内存问题;如:本应使用对象重用或者对象池时却被不恰当的初始化,没有在finally块中关闭昂贵的资源。

  6、线程安全问题;如:java api类像simpledateformat,calendar,decimalformat等不是线程完全的,在jsp中声明变量也不是安全的,存储状态信息在struts action类中或者多线程servlet也不是线程安全的。

  4、system.out常被log4j替换

  3、设计问题:没有重用代码,没有清晰的责任分离。如:业务逻辑嵌套在servlet中,而没有使用业务逻辑层,可视化元素(如html,css)嵌入在后台。

  2、代码文档:没有注释,没有头文件等

  1、从给定的框架中遵循最佳实践:如spring3中注解替代xml文件对于ioc, 对于每一个简单的部署使用外部属性替代硬编码值等

  工欲善其事必先利其器

  code review 工具:

  1、checkstyle: 并不只是一个code review 工具,更是一个开发工具确保开发者的代码遵循标准,在每一次code review中节省时间。

====================================分割线================================

最新内容请见作者的github页:http://qaseven.github.io/