十个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

posted @ 2016-01-08 16:29  smile_tina  阅读(2787)  评论(4编辑  收藏  举报