十個JAVA程序員容易犯的錯誤
1. Array 轉 ArrayList
一般開發者喜歡用:
List<String> list = Arrays.asList(arr);
Arrays.asList() 會返回一個ArrayList,這是Arrays里內嵌的一個私有靜態類,而並不是java.util.ArrayList
類java.util.Arrays.ArrayList 有set(), get(), contains()方法,但並支持添加元素,所以大小是固定的,想要創建一個真正的ArrayList,你應該:
ArrayList<String> arrayList = new ArrayList<String>(Arrays.asList(arr));
關於這個方法更多的解釋,請參見我之前的一篇文章
2.檢測一個Array是否包含一個元素
一般開發者喜歡用:
Set<String> set = new HashSet<String>(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<String> list = new ArrayList<String>(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<String> list = new ArrayList<String>(Arrays.asList("a", "b", "c", "d"));
for (String s : list) {
if (s.equals("a"))
list.remove(s);
}
會報錯 ConcurrentModificationException.
正確的應該是:
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();
}
}
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<String> list = new ArrayList<String>();
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