在沉浸于編碼一段時間以后,你就會漸漸明白任何事情有可能出錯,這就是為什么我們要采用“防御性編程”,即一些偏執習慣的原因。下面是動力節點的Java老師為大家整理的大牛眼中的10個非常有用但偏執的Java編程技術。一起來看一看吧:
1、將String字符串放在zui前面
為了防止偶發性的NullPointerException異常,我們通常將String放置在equals()函數的左邊來實現字符串比較,如下代碼:
//Bad
if(variable.equals("literal")){...}
//Good
if("literal".equals(variable)){...}
這是隨便用腦子想想就可以做的事,從Bad版本的代碼改寫表達式到Good版本的代碼,這中間并不會丟失任何東西。歡迎不同的觀點…
2、不要相信早期的JDKAPI
在Java早期,編程是一件非常痛苦的事情。那些API仍然很不成熟,也許你已經碰到過下面的代碼塊:
String[]files=file.list();
//Watchout
if(files!=null){
for(inti=0;i<files.length;i++){
...
}
}
看上去很偏執?也許吧,但請看Javadoc:
如果這個虛擬路徑不表示一個文件夾目錄,則此方法返回null。否則將會返回一個字符串數組,每一個字符串表示目錄中的文件或文件夾。
對,沒錯。我們可以添加一些校驗:
if(file.isDirectory()){
String[]files=file.list();
//Watchout
if(files!=null){
for(inti=0;i<files.length;i++){
...
}
}
}
3、不要相信“-1”
我知道這是偏執的,但Javadoc中對String.indexOf()方法明確指出:對象內第一次出現指定字符的位置索引,如果為-1則表示該字符不在字符序列中。
所以使用-1是理所當然的,對嗎?我說不對,請看以下代碼:
//Bad
if(string.indexOf(character)!=-1){...}
//Good
if(string.indexOf(character)>=0){...}
誰知道呢。也許到時候他們改變了編碼方式,對字符串并不區分大小寫,也許更好的方式是返回-2?誰知道呢。
4、避免意外賦值
是的。這種事情也許經常會發生。
//Ooops
if(variable=5){...}
//Better(becausecausesanerror)
if(5=variable){...}
//Intent(remember.ParanoidJavaScript:===)
if(5===variable){...}
所以你可以將比較常量放置在左側,這樣就不會發生意外賦值的錯誤了。
5、檢查Null和Length
無論如何,只要你有一個集合、數組等,請確保它存在,并且不為空。
//Bad
if(array.length>0){...}
//Good
if(array!=null&&array.length>0){...}
你并不知道這些數組從哪里來,也許是來自早期版本的JDKAPI,誰知道呢。
6、所有的方法都是final的
你也許會告訴我你的開/閉原則,但這都是胡說八道。我不相信你(正確繼承我這個父類的所有子類),我也不相信我自己(不小心繼承我這個父類的所有子類)。所以對于那些意義明確的方法要嚴格用final標識。
//Bad
publicvoidboom(){...}
//Good.Don'ttouch.
publicfinalvoiddontTouch(){...}
7、所有變量和參數都是final
//Bad
voidinput(StringimportantMessage){
Stringanswer="...";
answer=importantMessage="LOLaccident";
}
//Good
finalvoidinput(finalStringimportantMessage){
finalStringanswer="...";
}
8、重載時不要相信泛型
是,它可以發生。你相信你寫的超級好看的API,它很直觀,隨之而來的,一些用戶誰只是將原始類型轉換成Object類型,直到那該死的編譯器停止發牢騷,并且突然他們會鏈接錯誤的方法,以為這是你的錯誤。
看下面的代碼:
//Bad
<T>voidbad(Tvalue){
bad(Collections.singletonList(value));
}
<T>voidbad(List<T>values){
...
}
//Good
final<T>voidgood(finalTvalue){
if(valueinstanceofList)
good((List<?>)value);
else
good(Collections.singletonList(value));
}
final<T>voidgood(finalList<T>values){
...
}
因為,你知道……你的用戶,他們就像
//Thislibrarysucks
@SuppressWarnings("all")
Objectt=(Object)(List)Arrays.asList("abc");
bad(t);
相信我。這一切我都看到過。包括下面的
這種偏執還是不錯的。
9、總是在Switch語句的Default中拋出異常
Switch語句……它們其中一個可笑的語句我不知道該對它敬畏還是哭泣,但無論如何,既然我們堅持用switch,那我們不妨將它用得優美,看下面的代碼:
//Bad
switch(value){
case1:foo();break;
case2:bar();break;
}
//Good
switch(value){
case1:foo();break;
case2:bar();break;
default:
thrownewThreadDeath("That'llteachthem");
}
當value==3時,將會出現無法找到的提示,而不會讓人不知所謂。
10、Switch語句帶花括號
事實上,switch是最邪惡的語句,像是一些喝醉了或者賭輸了的人在寫代碼一樣,看下面的例子:
//Bad,doesn'tcompile
switch(value){
case1:intj=1;break;
case2:intj=2;break;
}
//Good
switch(value){
case1:{
finalintj=1;
break;
}
case2:{
finalintj=2;
break;
}
//Remember:
default:
thrownewThreadDeath("That'llteachthem");
}
在switch語句中,每一個case語句的范圍只有一行語句,事實上,這些case語句甚至不是真正的語句,他們就像goto語句中的跳轉標記一樣。
結論
偏執編程看起來似乎不可思議,有時,因為代碼經常被證明是更詳細一點,但并不是需求需要。你可能會想,“哦,這是不會發生的”,但正如動力節點Java老師所說。經過20年左右的時間編程,你不希望只修改這些愚蠢的bug。