在沉浸於編碼一段時間以後,你會漸漸對這些東西習以為常。因為,你知道的…
任何事情有可能出錯,沒錯,的確如此。
這就是為什麼我們要採用“防禦性編程”,即一些偏執習慣的原因。以下是我個人認為的10個最有用但偏執的Java程式設計技術。一起來看看:
1.將String字串放在最前面
為了防止偶發性的NullPointerException 異常,我們通常將String放置在equals()函數的左邊來實現字串比較,如下程式碼:
// Bad if (variable.equals("literal")) { ... } // Good if ("literal".equals(variable)) { ... }
這是隨便用腦子想想就可以做的事,從Bad版本的程式碼改寫表達式到Good版本的程式碼,這中間並不會遺失任何東西。歡迎不同的觀點…
2.不要相信早期的JDK API
在Java早期,程式設計是一件非常痛苦的事情。那些API仍然很不成熟,也許你已經碰到過下面的程式碼區塊:
String[] files = file.list(); // Watch out if (files != null) { for (int i = 0; i < files.length; i++) { ... } }
看上去很偏執?也許吧,但請看Javadoc:
如果這個虛擬路徑不表示一個資料夾目錄,則此方法傳回null。否則將會傳回一個字串數組,每一個字串表示目錄中的檔案或資料夾。
對,沒錯。我們可以加入一些校驗:
if (file.isDirectory()) { String[] files = file.list(); // Watch out if (files != null) { for (int i = 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 (because causes an error) if (5 = variable) { ... } // Intent (remember. Paranoid JavaScript: ===) if (5 === variable) { ... }
所以你可以將比較常數放置在左側,這樣就不會發生意外賦值的錯誤了。
5.檢查Null和Length
無論如何,只要你有一個集合、數組等,請確保它存在,並且不為空。
// Bad if (array.length > 0) { ... } // Good if (array != null && array.length > 0) { ... }
你並不知道這些陣列從哪裡來,也許是來自早期版本的JDK API,誰知道呢。
6.所有的方法都是final的
你也許會告訴我你的開/閉原則,但這都是胡說八道。我不相信你(正確繼承我這個父類別的所有子類別),我也不相信我自己(不小心繼承我這個父類別的所有子類別)。所以對於那些意義明確的方法要嚴格地用final來標識。
// Bad public void boom() { ... } // Good. Don't touch. public final void dontTouch() { ... }
7.所有變數和參數都是final
就像我說的。我不相信我自己(不要意外覆蓋我的值)。話雖如此,我不相信我自己的是因為…
…这就是为什么所有的变量和参数都是final的原因。
// Bad void input(String importantMessage) { String answer = "..."; answer = importantMessage = "LOL accident"; } // Good final void input(final String importantMessage) { final String answer = "..."; }
8.重载时不要相信泛型
是,它可以发生。你相信你写的超级好看的API,它很直观,随之而来的,一些用户谁只是将原始类型转换成Object类型,直到那该死的编译器停止发牢骚,并且突然他们会链接错误的方法,以为这是你的错误。
看下面的代码:
// Bad <T> void bad(T value) { bad(Collections.singletonList(value)); } <T> void bad(List<T> values) { ... } // Good final <T> void good(final T value) { if (value instanceof List) good((List<?>) value); else good(Collections.singletonList(value)); } final <T> void good(final List<T> values) { ... }
因为,你知道……你的用户,他们就像
// This library sucks @SuppressWarnings("all") Object t = (Object) (List) Arrays.asList("abc"); bad(t);
相信我。这一切我都看到过。包括下面的
这种偏执还是不错的。
9.总是在Switch语句的Default中抛出异常
Switch语句……它们其中一个可笑的语句我不知道该对它敬畏还是哭泣,但无论如何,既然我们坚持用switch,那我们不妨将它用得完美,看下面的代码:
// Bad switch (value) { case 1: foo(); break; case 2: bar(); break; } // Good switch (value) { case 1: foo(); break; case 2: bar(); break; default: throw new ThreadDeath("That'll teach them"); }
当value == 3时,将会出现无法找到的提示,而不会让人不知所谓。
10.Switch语句带花括号
事实上,switch是最邪恶的语句,像是一些喝醉了或者赌输了的人在写代码一样,看下面的例子:
// Bad, doesn't compile switch (value) { case 1: int j = 1; break; case 2: int j = 2; break; } // Good switch (value) { case 1: { final int j = 1; break; } case 2: { final int j = 2; break; } // Remember: default: throw new ThreadDeath("That'll teach them"); }
在switch语句中,每一个case语句的范围只有一行语句,事实上,这些case语句甚至不是真正的语句,他们就像goto语句中的跳转标记一样。
在switch语句中,每一个case语句的范围只有一行语句,事实上,这些case语句甚至不是真正的语句,他们就像goto语句中的跳转标记一样。
结论
偏执编程看起来似乎不可思议,有时,因为代码经常被证明是更详细一点,但并不是需求需要。你可能会想,“哦,这是绝不会发生的”,但正如我所说。经过20年左右的时间编程,你不希望只修复这些愚蠢的bug,因为编程语言是如此的陈旧的和有缺陷的。因为你知道…