Java開發人員最常犯的10個錯誤
閱讀目錄
-
Array轉ArrayList
-
判斷一個數組是否包含某個值
-
在迴圈內部刪除List中的一個元素
-
HashTable與HashMap
-
使用集合原始型別(raw type)
-
訪問級別
-
ArrayList和LinkedList
-
可變與不可變
-
父類和子類的構造方法
-
“”還是構造方法
-
未來工作
這個列表總結了10個Java開發人員最常犯的錯誤。
1、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已經實現了該介面。
2、判斷一個數組是否包含某個值
開發人員經常這樣做:
Set<String> set = 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;
第一種方法可讀性更強。
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大小減小,然後原先索引指向了其它元素。
所以如果你想在迴圈裡通過索引來刪除多個元素,將不會正確工作。
你也許知道使用迭代器是在迴圈裡刪除元素的正確方式,或許你也知道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()的原始碼。
4、HashTable與HashMap
從演算法的角度來講,HashTable是一種資料結構名稱。但是在Java中,這種資料結構叫做HashMap。
HashTable與HashMap的一個主要的區別是HashTable是同步的,所以,通常來說,你會使用HashMap,而不是Hashtable。
5、使用集合原始型別(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(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, 和Set這三個有很大的不同。
6、訪問級別
開發人員經常使用public修飾類欄位,雖然這很容易讓別人直接通過引用獲取該欄位的值,但這是一個不好的設計。
根據經驗,應該儘可能的降低成員屬性的訪問級別。
7、ArrayList和LinkedList
為什麼開發人員經常使用ArrayList和LinkedList,卻不知道他們之間的區別,因為它們看起來很像。然而它們之間有著巨大的效能差異。
簡單的說,如果有大量的增加刪除操作並且沒有很多的隨機訪問元素的操作,應該首選LinkedList。
8、可變與不可變
不可變物件有很多優點,如簡單、安全等。但是對於每個不同的值都需要一個單獨的物件,太多的物件會引起大量垃圾回收,因此在選擇可變與不可變的時候,需要有一個平衡。
通常,可變物件用於避免產生大量的中間物件,一個經典的例子是大量字串的拼接。
如果你使用一個不可變物件,將會馬上產生大量符合垃圾回收標準的物件,這浪費了CPU大量的時間和精力。使用可變物件是正確的解決方案(StringBuilder);
String result="";
for(String s: arr){
result = result + s;
}
另外,在有些其它情況下也是需要使用可變物件。例如往一個方法傳入一個可變物件,然後收集多種結果,而不需要寫太多的語法。
另一個例子是排序和過濾:當然,你可以寫一個方法來接收原始的集合,並且返回一個排好序的集合,但是那樣對於大的集合就太浪費了。
9、父類和子類的構造方法
之所以出現這個編譯錯誤,是因為父類的預設構造方法未定義。
在Java中,如果一個類沒有定義構造方法,編譯器會預設插入一個無引數的構造方法;
但是如果一個構造方法在父類中已定義,在這種情況,編譯器是不會自動插入一個預設的無參構造方法,這正是以上demo的情況;
對於子類來說,不管是無參構造方法還是有參構造方法,都會預設呼叫父類的無參構造方法;
當編譯器嘗試在子類中往這兩個構造方法插入super()方法時,因為父類沒有一個預設的無參構造方法,所以編譯器報錯;
要修復這個錯誤,很簡單:
1、在父類手動定義一個無參構造方法:
public Super(){
System.out.println("Super");
}
2、移除父類中自定義的構造方法
3、在子類中自己寫上父類構造方法的呼叫;如super(value);
10、“”還是構造方法
有兩種建立字串的方式:
//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
未來工作
這個列表是我基於大量的github上的開源專案,Stack overflow上的問題,還有一些流行的google搜尋的分析。
沒有明顯示的評估證明它們是前10,但它們絕對是很常見的。如果您不同意任一部分,請留下您的評論。如果您能提出其它一些常見的錯誤,我將會非常感激。