Java程序员常犯的十个错误

#1 把Array转变成ArrayList

为了把Array转变成ArrayList，程序员经常这样做：

List list = Arrays.asList(arr);

Arrays.asList返回的是Arrays内部的一个静态私有的ArrayList类，他不是Java.util.ArrayList类。Java.util.Arrays.ArrayList类有set(), get(), contains()这些方法，但是他没有任何加入元素的方法，所以他的大小是固定的。要创建一个真的ArrayList，你需要这样做：

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

ArrayList的构造函数可以接收一个Collection的类型，这个类型是java.util.Arrays.ArrayList的父类。

#2 检查一个Array是否包含一个值

程序员经常这样写：

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

这个代码是可以工作的，但是我们没有必要把一个list转变成set。转变成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]

这个代码有个眼中的问题，就是当一个元素被remove了，他的大小也变化了，同时index也发生了变化。所以，假如你想在循环中删除一个元素的话，那么使用index可能会出问题。

你也许会说，使用迭代是一个很好的在循环中删除元素的方式，你也许会觉得foreach在Java中其实就和迭代类似，其实事实上并不是这样，我们来看下面这个代码：

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();
    }
}

.next()必须在.remove()之前调用。在foreach循环中，编译器会在移除元素之后调用.next()，这样就会导致ConcurrentModificationException。具体你可以看看ArrayList.iterator()的源码。

#4 Hanshtable 对比 HashMap

从算法定义的角度来说，hashtable是数据结构的名字。但是在Java中，数据结构的名字是HashMap。Hashtable和HashMap的最大的不同在于Hashtable是同步的。所以，很多时候，你并不需要使用Hashtable，HashMap就足够了。

具体可以参加这两篇文章：

HashMap vs. TreeMap vs. Hashtable vs. LinkedHashMap

Top 10 questions about Map

#5 使用Collection的原生类型

在Java中，原生类型和无界通配符很容易混淆。用Set来举例，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：

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

使用原生类型是非常危险的，因为他没有类型的检测并且不安全。Set, Set以及Set