Java編程新手有時候會檢查new操作的結果是否為null�？赡艿臋z查代碼為：
Integer i = new Integer (400); if (i == null) throw new NullPointerException(); |
檢查當然沒什么錯誤，但卻不必要，if和throw這兩行代碼完全是浪費，他們的唯一功用是讓整個程序更臃腫，運行更慢。
C/C++程序員在開始寫java程序的時候常常會這么做，這是由于檢查C中malloc()的返回結果是必要的，不這樣做就可能產(chǎn)生錯誤。檢查C++中new操作的結果可能是一個好的編程行為，這依賴于異常是否被使能(許多編譯器允許異常被禁止，在這種情況下new操作失敗就會返回null)。在java 中，new 操作不允許返回null，如果真的返回null，很可能是虛擬機崩潰了，這時候即便檢查返回結果也無濟于事。
七、常見錯誤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");
|
對象更復雜些，==操作符檢查兩個引用是否指向同一個對象，而equals方法則實現(xiàn)更專門的相等性檢查。
更顯得混亂的是由java.lang.Object 所提供的缺省的equals方法的實現(xiàn)使用==來簡單的判斷被比較的兩個對象是否為同一個。
許多類覆蓋了缺省的equals方法以便更有用些，比如String類，它的equals方法檢查兩個String對象是否包含同樣的字符串，而Integer的equals方法檢查所包含的int值是否相等。
大部分時候，在檢查兩個對象是否相等的時候你應該使用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;
}
|
在測試的時候，你可能不會捕獲到這個錯誤。首先，測試與線程有關的錯誤是很難的，而且很耗時間。其次，在有些機器上，這些代碼可能會被翻譯成一條指令，因此工作正常，只有當在其它的虛擬機上測試的時候這個錯誤才可能顯現(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。但是當try塊運行出現(xiàn)了Error，流也應該被關閉。
3．close()可能會拋出異常。
上面代碼的一個更優(yōu)版本為：
OutputStream os = null;
try{
os = new OutputStream ();
// Do something with os here.
}finally{
if (os != null)
os.close();
}
|
這個版本消除了上面所提到的兩個問題：代碼不再重復，Error也可以被正確處理了。但是沒有好的方法來處理第三個問題，也許最好的方法是把close()語句單獨放在一個try/catch塊中。
十、常見錯誤10#： 增加不必要的catch 塊
一些開發(fā)者聽到try/catch塊這個名字后，就會想當然的以為所有的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)在大部分時候毫無用處，因此許多類覆蓋其中的若干個方法以提供更有用的功能。但是，問題又來了，當繼承一個覆蓋了若干個這些方法的父類的時候，子類通常也需要覆蓋這些方法。在進行代碼審查時，應該確保如果父類實現(xiàn)了equals，hashCode，或者clone等方法，那么子類也必須正確。正確的實現(xiàn)equals，hashCode，和clone需要一些技巧。
小結
我在代碼審查的時候至少遇到過一次這些錯誤，我自己也犯過其中的幾個錯誤。好消息是只要你知道你在找什么錯誤，那么代碼審查就很容易管理，錯誤也很容易被發(fā)現(xiàn)和修改。即便你找不到時間來進行正規(guī)的代碼審查，以自審的方式把這些錯誤從你的代碼中根除會大大節(jié)省你的調(diào)試時間。花時間在代碼審查上是值得的。
