十個JAVA程序員容易犯的錯誤


十個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));

關於這個方法更多的解釋,請參見我之前的一篇文章

http://www.cnblogs.com/tina-smile/p/5056174.html

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 這三個有很大區別,稍后我再做解釋。

6.訪問權限

很多開發者使用public修飾類的字段,這樣很容易獲取字段值,但這是一個糟糕的設計。經驗法則是給用戶盡可能低的訪問權限。

7.ArrayList vs. LinkedList

大部分的開發者並不知道ArrayList和LinkedList 的區別,所以經常使用ArrayList,這個看起來更熟悉。但是,在這兩者之間,有一個很大的性能差異。

簡單來說,在有很多插入和刪除操作時,同時有很少或者幾乎沒有隨機訪問的操作情況下,推薦使用LinkedList。其次,再使用ArrayList。

8.Mutable vs. Immutable(可變 VS 不可變)

不可變對象有很多優勢這樣簡單,安全,等等。但它對於每一個不同的變量,需要一個單獨的對象。太多的對象可能會導致高成本的垃圾收集。所以在選擇可變和不可變的時應該有個權衡。

一般來說,使用可變對象,以避免產生過多的中間對象。一個經典的例子就是連接大量的字符串。如果使用不可變的String,你會產生很多立馬要被垃圾收集回收的對象,這很消耗時間和精力。如果使用一個可變的,類似StringBuffer。

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

可變對象還有其他的例子:函數傳參的過程中,可以傳入可變對象,這樣可以得到多個結果。

9.父類和子類的構建

編譯出錯的原因主要是默認的Super類的構造函數並沒有被定義。在Java中,如果一個類沒有實現構造函數,編譯器默認會給這個類插入一個無參構造函數。如果在父類中已經有了構造函數,那么編譯器將不會再插入默認無參構造函數。

子類的構造函數,要么是帶參的要么是無參的,都會調用父類的無參構造。因為編譯器想在子類中,加入super(),但是父類的無參構造函數並不存在。所以,編譯器會報錯。

為了解決這個問題,有兩種解決辦法,第一,在Super類中,加入無參構造:

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

第二種方式,移除自定義的父類構造函數

第三種方式,子類的構造函數中加入super(value)

10."" or Constructor?

創建一個String 有兩種方式:

//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

本文是翻譯自

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

posted @ 2016-01-08 16:29  smile_tina  閱讀( 2349)  評論( 4編輯  收藏


免責聲明!

本站轉載的文章為個人學習借鑒使用,本站對版權不負任何法律責任。如果侵犯了您的隱私權益,請聯系本站郵箱yoyou2525@163.com刪除。



 
粵ICP備18138465號   © 2018-2025 CODEPRJ.COM