天天看點

靈活代碼審查指南

  我強烈贊同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/