10個(gè)實(shí)用的但偏執(zhí)的Java編程技術(shù)

作者：小峰 2015-08-31 10:46:05

偏執(zhí)編程看起來似乎不可思議，有時(shí)，因?yàn)榇a經(jīng)常被證明是更詳細(xì)一點(diǎn)，但并不是需求需要。你可能會想，“哦，這是絕不會發(fā)生的”，但正如我所說。經(jīng)過20年左右的時(shí)間編程，你不希望只修復(fù)這些愚蠢的bug，因?yàn)榫幊陶Z言是如此的陳舊的和有缺陷的。因?yàn)槟阒馈?/div>

[[147453]]

在沉浸于編碼一段時(shí)間以后（比如說我已經(jīng)投入近20年左右的時(shí)間在程序上了），你會漸漸對這些東西習(xí)以為常。因?yàn)椋阒赖?hellip;…

任何事情有可能出錯(cuò)，沒錯(cuò)，的確如此。

這就是為什么我們要采用“防御性編程”，即一些偏執(zhí)習(xí)慣的原因。下面是我個(gè)人認(rèn)為的10個(gè)最有用但偏執(zhí)的Java編程技術(shù)。一起來看一看吧：

1.將String字符串放在最前面

為了防止偶發(fā)性的NullPointerException 異常，我們通常將String放置在equals()函數(shù)的左邊來實(shí)現(xiàn)字符串比較，如下代碼：

// Bad 
if (variable.equals("literal")) { ... } 
// Good 
if ("literal".equals(variable)) { ... }

這是隨便用腦子想想就可以做的事，從Bad版本的代碼改寫表達(dá)式到Good版本的代碼，這中間并不會丟失任何東西。歡迎不同的觀點(diǎn)…

2.不要相信早期的JDK API

在Java早期，編程是一件非常痛苦的事情。那些API仍然很不成熟，也許你已經(jīng)碰到過下面的代碼塊：

String[] files = file.list(); 
// Watch out 
if (files != null) { 
    for (int i = 0; i < files.length; i++) { 
        ... 
    } 
}

看上去很偏執(zhí)？也許吧，但請看Javadoc：

如果這個(gè)虛擬路徑不表示一個(gè)文件夾目錄，則此方法返回null。否則將會返回一個(gè)字符串?dāng)?shù)組，每一個(gè)字符串表示目錄中的文件或文件夾。

對，沒錯(cuò)。我們可以添加一些校驗(yàn)：

if (file.isDirectory()) { 
    String[] files = file.list(); 
    // Watch out 
    if (files != null) { 
        for (int i = 0; i < files.length; i++) { 
            ... 
        } 
    } 
}

3.不要相信“-1”

我知道這是偏執(zhí)的，但Javadoc中對 String.indexOf()方法明確指出：對象內(nèi)***次出現(xiàn)指定字符的位置索引，如果為-1則表示該字符不在字符序列中。

所以使用-1是理所當(dāng)然的，對嗎？我說不對，請看以下代碼：

// Bad 
if (string.indexOf(character) != -1) { ... } 
// Good 
if (string.indexOf(character) >= 0) { ... }

誰知道呢。也許到時(shí)候他們改變了編碼方式，對字符串并不區(qū)分大小寫，也許更好的方式是返回-2？誰知道呢。

4.避免意外賦值

是的。這種事情也許經(jīng)常會發(fā)生。

// Ooops 
if (variable = 5) { ... } 
// Better (because causes an error) 
if (5 = variable) { ... } 
// Intent (remember. Paranoid JavaScript: ===) 
if (5 === variable) { ... }

所以你可以將比較常量放置在左側(cè)，這樣就不會發(fā)生意外賦值的錯(cuò)誤了。

5.檢查Null和Length

無論如何，只要你有一個(gè)集合、數(shù)組等，請確保它存在，并且不為空。

// Bad 
if (array.length > 0) { ... } 
// Good 
if (array != null && array.length > 0) { ... }

你并不知道這些數(shù)組從哪里來，也許是來自早期版本的JDK API，誰知道呢。

6.所有的方法都是final的

你也許會告訴我你的開/閉原則，但這都是胡說八道。我不相信你（正確繼承我這個(gè)父類的所有子類），我也不相信我自己（不小心繼承我這個(gè)父類的所有子類）。所以對于那些意義明確的方法要嚴(yán)格用final標(biāo)識。

// Bad 
public void boom() { ... } 
// Good. Don't touch. 
public final void dontTouch() { ... }

7.所有變量和參數(shù)都是final

就像我說的。我不相信我自己（不要意外覆蓋我的值）。話雖如此，我不相信我自己的是因?yàn)?hellip;

…這就是為什么所有的變量和參數(shù)都是final的原因。

// Bad 
void input(String importantMessage) { 
    String answer = "..."; 
    answer = importantMessage = "LOL accident"; 
} 
// Good 
final void input(final String importantMessage) { 
    final String answer = "..."; 
}

8.重載時(shí)不要相信泛型

是，它可以發(fā)生。你相信你寫的超級好看的API，它很直觀，隨之而來的，一些用戶誰只是將原始類型轉(zhuǎn)換成Object類型，直到那該死的編譯器停止發(fā)牢騷，并且突然他們會鏈接錯(cuò)誤的方法，以為這是你的錯(cuò)誤。

看下面的代碼：

// 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) { 
    ... 
}

因?yàn)椋阒?hellip;…你的用戶，他們就像

// This library sucks 
@SuppressWarnings("all") 
Object t = (Object) (List) Arrays.asList("abc"); 
bad(t);

相信我。這一切我都看到過。包括下面的

這種偏執(zhí)還是不錯(cuò)的。

9.總是在Switch語句的Default中拋出異常

Switch語句……它們其中一個(gè)可笑的語句我不知道該對它敬畏還是哭泣，但無論如何，既然我們堅(jiān)持用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"); 
}

當(dāng)value == 3時(shí)，將會出現(xiàn)無法找到的提示，而不會讓人不知所謂。

10.Switch語句帶花括號

事實(shí)上，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"); 
}