Java程序員容易犯的錯誤

1. Array 轉 ArrayList
   一般開發者喜歡用：

List list = Arrays.asList(arr);

Arrays.asList() 會返回一個ArrayList，這是Arrays里內嵌的一個私有靜態類，而并不是java.util.ArrayList類java.util.Arrays.ArrayList 有set(), get(), contains()方法，但并支持添加元素，所以大小是固定的，想要創建一個真正的ArrayList，你應該：

ArrayList arrayList = new ArrayList(Arrays.asList(arr));

2.檢測一個Array是否包含一個元素

一般開發者喜歡用：

Set set = new HashSet(Arrays.asList(arr));

return set.contains(targetValue);

這段代碼能夠工作，但并沒有先轉將list轉為set，list轉為set需要額外時間，可以用下面更簡單的：

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

或者

for(String s: arr){

if(s.equals(targetValue))

return true;

}

return false;

第一種方法比第二種的可讀性更好。

3. 在一個循環中移除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的大小就縮減了，索引也發生了改變。所以如果想要在一個循環中使用索引移除多個元素，這是不可行的。

或許你又想用迭代器的方式解決這個問題，但實際上也不可行，考慮下下面的代碼：

ArrayList list = new ArrayList(Arrays.asList("a", "b", "c", "d"));

for (String s : list) {

if (s.equals("a"))

list.remove(s);

}

會報錯 ConcurrentModificationException.

正確的應該是：

ArrayList list = new ArrayList(Arrays.asList("a", "b", "c", "d"));

Iterator iter = list.iterator();

while (iter.hasNext()) {

String s = iter.next();

if (s.equals("a")) {

iter.remove();

}

}

4. Hashtable vs HashMap

在算法中，哈希表是數據結構的名稱。但在Java中，數據結構的名字是HashMap中。其中一個Hashtable和HashMap的之間的主要區別是，Hashtable是同步的。所以，很多時候，你并不需要哈希表，HashMap就夠用了。

5.使用原始類型的集合

在Java中，原始類型和***通配符類型很容易混在一起。以Set為例，設定為原始類型，同時set是***的通配符類型。

使用原始類型list，考慮下面的情況：

public static void add(List list, Object o){

list.add(o);

}

public static void main(String[] args){

List list = new ArrayList();

add(list, 10);

String s = list.get(0);

}

這個代碼將會拋出錯誤：

Exception in thread "main" java.lang.ClassCastException: java.lang.Integer cannot be cast to java.lang.String

at ...

使用原始類型集合是很危險的一件事，因為它跳過了類屬性檢測并且是不安全的。Set, Set, and Set