歡迎光臨
每天分享高質量文章

Java開發人員最常犯的10個錯誤

譯者:風一樣的碼農

鏈接:https://www.cnblogs.com/chenpi/p/5508949.html

原文:http://www.programcreek.com/2014/05/top-10-mistakes-java-developers-make/

這個串列總結了10個Java開發人員最常犯的錯誤。

Array轉ArrayList

當需要把Array轉成ArrayList的時候,開發人員經常這樣做:

List list = Arrays.asList(arr);

Arrays.asList()會傳回一個ArrayList,但是要特別註意,這個ArrayList是Arrays類的靜態內部類,並不是java.util.ArrayList類。java.util.Arrays.ArrayList類實現了set(), get(),contains()方法,但是並沒有實現增加元素的方法(事實上是可以呼叫add方法,但是沒有具體實現,僅僅丟擲UnsupportedOperationException異常),因此它的大小也是固定不變的。為了創建一個真正的java.util.ArrayList,你應該這樣做:

ArrayList<String> arrayList = new ArrayList<String>(Arrays.asList(arr));

ArrayList的構造方法可以接收一個Collection型別,而java.util.Arrays.ArrayList已經實現了該接口。

判斷一個陣列是否包含某個值

開發人員經常這樣做:

Set<Stringset = new HashSet<String>(Arrays.asList(arr));
return set.contains(targetValue);

以上代碼可以正常工作,但是沒有必要將其轉換成set集合,將一個List轉成Set需要額外的時間,其實我們可以簡單的使用如下方法即可:

Arrays.asList(arr).contains(targetValue);

或者

for(String s: arr){
    if(s.equals(targetValue))
        return true;
}
return false;

第一種方法可讀性更強。

在迴圈內部刪除List中的一個元素

考慮如下代碼,在迭代期間刪除元素:

ArrayList list = new ArrayList(Arrays.asList("a""b""c","d"));
for (int i = 0; i list.size(); i++) {
    list.remove(i);
}
System.out.println(list);

結果打印:
[b, d]

在上面這個方法中有一系列的問題,當一個元素被刪除的時候,list大小減小,然後原先索引指向了其它元素。所以如果你想在迴圈裡通過索引來刪除多個元素,將不會正確工作。

你也許知道使用迭代器是在迴圈裡刪除元素的正確方式,或許你也知道foreach迴圈跟迭代器很類似,但事實情況卻不是這樣,如下代碼:

ArrayList list = new ArrayList(Arrays.asList("a""b""c","d"));
for (String s : list) {
    if (s.equals("a"))
        list.remove(s);
}

將丟擲ConcurrentModificationException異常。

然而接下來的代碼卻是OK的:

ArrayList<String> list = new ArrayList<String>(Arrays.asList("a""b""c","d"));
Iterator<String> iter = list.iterator();
while (iter.hasNext()) {
    String s = iter.next();
    if (s.equals("a")) {
        iter.remove();
    }
}

next()方法需要在remove()方法之前被呼叫,在foreach迴圈裡,編譯器會在刪除元素操作化呼叫next方法,這導致了ConcurrentModificationException異常。更多詳細信息,可以查看ArrayList.iterator()的原始碼。

HashTable與HashMap

從演算法的角度來講,HashTable是一種資料結構名稱。但是在Java中,這種資料結構叫做HashMap。HashTable與HashMap的一個主要的區別是HashTable是同步的,所以,通常來說,你會使用HashMap,而不是Hashtable。

更多信息:HashMap vs. TreeMap vs. Hashtable vs. LinkedHashMap Top 10 questions about Map

使用集合原始型別(raw type)

在Java中,原始型別(raw type)和無界通配符型別很容易讓人混淆。舉個Set的例子,Set是原始型別,而Set是無界通配符型別。

請看如下代碼,add方法使用了一個原始型別的List作為入參:

public static void add(List list, Object o){
    list.add(o);
}
public static void main(String[] args){
    List list = new ArrayList();
    add(list10);
    String s = list.get(0);
}

運行以上代碼將會丟擲異常:

Exception in thread "mainjava.lang.ClassCastExceptionjava.lang.Integer cannot be cast to java.lang.String
at ...

使用原始型別集合非常危險,因為它跳過了泛型型別檢查,是不安全的。另外,Set, Set, 和Set這三個有很大的不同,具體請看:型別擦除和Raw type vs. Unbounded wildcard。

訪問級別

開發人員經常使用public修飾類欄位,雖然這很容易讓別人直接通過取用獲取該欄位的值,但這是一個不好的設計。根據經驗,應該盡可能的降低成員屬性的訪問級別。

相關閱讀:public, default, protected, and private

ArrayList和LinkedList

為什麼開發人員經常使用ArrayList和LinkedList,卻不知道他們之間的區別,因為它們看起來很像。然而它們之間有著巨大的性能差異。簡單的說,如果有大量的增加刪除操作並且沒有很多的隨機訪問元素的操作,應該首選LinkedList。

相關閱讀:ArrayList vs. LinkedList

可變與不可變

不可變物件有很多優點,如簡單、安全等。但是對於每個不同的值都需要一個單獨的物件,太多的物件會引起大量垃圾回收,因此在選擇可變與不可變的時候,需要有一個平衡。

通常,可變物件用於避免產生大量的中間物件,一個經典的例子是大量字串的拼接。如果你使用一個不可變物件,將會馬上產生大量符合垃圾回收標準的物件,這浪費了CPU大量的時間和精力。使用可變物件是正確的解決方案(StringBuilder);

String result="";
for(String s: arr){
    result = result + s;
}

另外,在有些其它情況下也是需要使用可變物件。例如往一個方法傳入一個可變物件,然後收集多種結果,而不需要寫太多的語法。另一個例子是排序和過濾:當然,你可以寫一個方法來接收原始的集合,並且傳回一個排好序的集合,但是那樣對於大的集合就太浪費了。

更多閱讀:Why String is Immutable??

Why we need mutable classes?

父類和子類的構造方法

之所以出現這個編譯錯誤,是因為父類的預設構造方法未定義。在Java中,如果一個類沒有定義構造方法,編譯器會預設插入一個無引數的構造方法;但是如果一個構造方法在父類中已定義,在這種情況,編譯器是不會自動插入一個預設的無參構造方法,這正是以上demo的情況;

對於子類來說,不管是無參構造方法還是有參構造方法,都會預設呼叫父類的無參構造方法;當編譯器嘗試在子類中往這兩個構造方法插入super()方法時,因為父類沒有一個預設的無參構造方法,所以編譯器報錯;

要修複這個錯誤,很簡單:

1、在父類手動定義一個無參構造方法:

public Super(){
    System.out.println("Super");
}

2、移除父類中自定義的構造方法

3、在子類中自己寫上父類構造方法的呼叫;如super(value);

“”還是構造方法

有兩種創建字串的方式:

//1. use double quotes
String x = "abc";
//2. use constructor
String y = new String("abc");

它們之間有什麼區別呢?

以下代碼提供了一個快速回答:

String a = "abcd";
String b = "abcd";
System.out.println(a == b); // True
System.out.println(a.equals(b)); // True
String c = new String("abcd");
String d = new String("abcd");
System.out.println(c == d); // False
System.out.println(c.equals(d)); // True

更多關於它們記憶體分配的信息,請參考Create Java String Using ” ” or Constructor??

未來工作

這個串列是我基於大量的github上的開源專案,Stack overflow上的問題,還有一些流行的google搜索的分析。沒有明顯示的評估證明它們是前10,但它們絕對是很常見的。如果您不同意任一部分,請 留下您的評論。如果您能提出其它一些常見的錯誤,我將會非常感激。

已同步到看一看
赞(0)

分享創造快樂