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