背景:#EDF0F5 #FAFBE6 #FFF2E2 #FDE6E0 #F3FFE1 #DAFAF3 #EAEAEF 默認  
閱讀內(nèi)容

審查Java代碼的一些常見錯誤

[日期:2008-07-20] 來源:  作者:九通科技 [字體: ]
  六、常見錯誤6#:檢查new 操作的結(jié)果是否為null

  Java編程新手有時候會檢查new操作的結(jié)果是否為null?赡艿臋z查代碼為:

Integer i = new Integer (400); 
if (i == null) 
throw new NullPointerException(); 

  檢查當(dāng)然沒什么錯誤,但卻不必要,if和throw這兩行代碼完全是浪費,他們的唯一功用是讓整個程序更臃腫,運行更慢。

  C/C++程序員在開始寫java程序的時候常常會這么做,這是由于檢查C中malloc()的返回結(jié)果是必要的,不這樣做就可能產(chǎn)生錯誤。檢查C++中new操作的結(jié)果可能是一個好的編程行為,這依賴于異常是否被使能(許多編譯器允許異常被禁止,在這種情況下new操作失敗就會返回null)。在java 中,new 操作不允許返回null,如果真的返回null,很可能是虛擬機崩潰了,這時候即便檢查返回結(jié)果也無濟于事。

  七、常見錯誤7#:用== 替代.equals

  在Java中,有兩種方式檢查兩個數(shù)據(jù)是否相等:通過使用==操作符,或者使用所有對象都實現(xiàn)的.equals方法。原子類型(int, flosat, char 等)不是對象,因此他們只能使用==操作符,如下所示:

int x = 4; 
int y = 5; 
if (x == y) 
   System.out.println ("Hi"); 
// This ’if’ test won’t compile. 
if (x.equals (y)) 
   System.out.println ("Hi"); 

  對象更復(fù)雜些,==操作符檢查兩個引用是否指向同一個對象,而equals方法則實現(xiàn)更專門的相等性檢查。

  更顯得混亂的是由java.lang.Object 所提供的缺省的equals方法的實現(xiàn)使用==來簡單的判斷被比較的兩個對象是否為同一個。

  許多類覆蓋了缺省的equals方法以便更有用些,比如String類,它的equals方法檢查兩個String對象是否包含同樣的字符串,而Integer的equals方法檢查所包含的int值是否相等。

  大部分時候,在檢查兩個對象是否相等的時候你應(yīng)該使用equals方法,而對于原子類型的數(shù)據(jù),你用該使用==操作符。

  八、常見錯誤8#: 混淆原子操作和非原子操作

  Java保證讀和寫32位數(shù)或者更小的值是原子操作,也就是說可以在一步完成,因而不可能被打斷,因此這樣的讀和寫不需要同步。以下的代碼是線程安全(thread safe)的:

public class Example{ 
  private int value; // More code here... 
  public void set (int x){ 
   // NOTE: No synchronized keyword 
   this.value = x; 
  } 
} 

  不過,這個保證僅限于讀和寫,下面的代碼不是線程安全的:

public void increment (){ 
  // This is effectively two or three instructions: 
  // 1) Read current setting of ’value’. 
  // 2) Increment that setting. 
  // 3) Write the new setting back. 
  ++this.value; 
} 

  在測試的時候,你可能不會捕獲到這個錯誤。首先,測試與線程有關(guān)的錯誤是很難的,而且很耗時間。其次,在有些機器上,這些代碼可能會被翻譯成一條指令,因此工作正常,只有當(dāng)在其它的虛擬機上測試的時候這個錯誤才可能顯現(xiàn)。因此最好在開始的時候就正確地同步代碼:

public synchronized void increment (){ 
  ++this.value; 
} 

  九、常見錯誤9#:在catch 塊中作清除工作

  一段在catch塊中作清除工作的代碼如下所示:

OutputStream os = null; 
try{ 
  os = new OutputStream (); 
  // Do something with os here. 
  os.close(); 
}catch (Exception e){ 
  if (os != null) 
  os.close(); 
} 

  盡管這段代碼在幾個方面都是有問題的,但是在測試中很容易漏掉這個錯誤。下面列出了這段代碼所存在的三個問題:

  1.語句os.close()在兩處出現(xiàn),多此一舉,而且會帶來維護方面的麻煩。

  2.上面的代碼僅僅處理了Exception,而沒有涉及到Error。但是當(dāng)try塊運行出現(xiàn)了Error,流也應(yīng)該被關(guān)閉。

  3.close()可能會拋出異常。

  上面代碼的一個更優(yōu)版本為:

OutputStream os = null; 
try{ 
  os = new OutputStream (); 
  // Do something with os here. 
}finally{ 
  if (os != null) 
   os.close(); 
} 

  這個版本消除了上面所提到的兩個問題:代碼不再重復(fù),Error也可以被正確處理了。但是沒有好的方法來處理第三個問題,也許最好的方法是把close()語句單獨放在一個try/catch塊中。

  十、常見錯誤10#: 增加不必要的catch 塊

  一些開發(fā)者聽到try/catch塊這個名字后,就會想當(dāng)然的以為所有的try塊必須要有與之匹配的catch塊。

  C++程序員尤其是會這樣想,因為在C++中不存在finally塊的概念,而且try塊存在的唯一理由只不過是為了與catch塊相配對。

  增加不必要的catch塊的代碼就象下面的樣子,捕獲到的異常又立即被拋出:

try{ 
  // Nifty code here 
}catch(Exception e){ 
  throw e; 
}finally{ 
  // Cleanup code here 
} 

  不必要的catch塊被刪除后,上面的代碼就縮短為:

try{ 
  // Nifty code here 
}finally{ 
  // Cleanup code here 
} 

  常見錯誤11#;沒有正確實現(xiàn)equals,hashCode,或者clone 等方法

  方法equals,hashCode,和clone 由java.lang.Object提供的缺省實現(xiàn)是正確的。不幸地是,這些缺省實現(xiàn)在大部分時候毫無用處,因此許多類覆蓋其中的若干個方法以提供更有用的功能。但是,問題又來了,當(dāng)繼承一個覆蓋了若干個這些方法的父類的時候,子類通常也需要覆蓋這些方法。在進行代碼審查時,應(yīng)該確保如果父類實現(xiàn)了equals,hashCode,或者clone等方法,那么子類也必須正確。正確的實現(xiàn)equals,hashCode,和clone需要一些技巧。

  小結(jié)

  我在代碼審查的時候至少遇到過一次這些錯誤,我自己也犯過其中的幾個錯誤。好消息是只要你知道你在找什么錯誤,那么代碼審查就很容易管理,錯誤也很容易被發(fā)現(xiàn)和修改。即便你找不到時間來進行正規(guī)的代碼審查,以自審的方式把這些錯誤從你的代碼中根除會大大節(jié)省你的調(diào)試時間。花時間在代碼審查上是值得的。

上一頁12  GO
推薦 】 【 打印
相關(guān)新聞      

本文評論       全部評論

發(fā)表評論
  • 尊重網(wǎng)上道德,遵守中華人民共和國的各項有關(guān)法律法規(guī)
  • 承擔(dān)一切因您的行為而直接或間接導(dǎo)致的民事或刑事法律責(zé)任
  • 本站管理人員有權(quán)保留或刪除其管轄留言中的任意內(nèi)容
  • 本站有權(quán)在網(wǎng)站內(nèi)轉(zhuǎn)載或引用您的評論
  • 參與本評論即表明您已經(jīng)閱讀并接受上述條款


點評: 字?jǐn)?shù)
姓名:
內(nèi)容查詢