Java程序员常犯的10个错误

原文出自：http://www.programcreek.com/2014/05/top-10-mistakes-java-developers-make/

本文总结了Java程序员常犯的10个错误。

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()方法，但是不能够添加新的元素。它的大小是固定的。如果你想要一个java.util.ArrayList，正确的方法是:

ArrayList arrayList = new ArrayList (Arrays.asList(arr));
java.util.ArrayList的构造函数可以接受一个集合类型。java.util.Arrays.ArrayList也继承了集合类型，所以可以作用参数使用。

2. 检查数组是否包含一个值

开发人员经常做的是：

Set set = new HashSet (Arrays.asList(arr));
return set.contains(targetValue);
这个代码是工作的，但没有没有效率。把列表转换成set没有必要，需要额外的时间。正确的方法是：

Arrays.asList(arr).contains(targetValue);
或者，一个简单的loop：

for(String s: arr){
if(s.equals(targetValue))
return true;
}
return false;
第一种比第二种更具有可读性。

3. 在循环中删除一个列表元素

考虑下面的代码，迭代过程中删除元素：

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]
这个方法有一个严重的问题。当元素被移除，该列表的大小缩减，元素索引也随之发生了变化。所以，如果你想通过使用索引来删除一个循环内的多个元素，就会导致错误的结果。

你可能猜到可以使用iterator来删除循环中的元素。在Java中的foreach循环的工作原理就像一个iterator。 但是在这里也会发生错误。请看下面的代码：

ArrayList list = new ArrayList (Arrays.asList("a", "b", "c", "d"));

for (String s : list) {
if (s.equals("a"))
list.remove(s);
}
上面的foreach loop代码会抛出一个异常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();
}

}
通过分析ArrayList.iterator()的原代码，我们可以发现next()方法必须要在remove()方法前被调用。在foreach loop中，编译器产生的代码会先调用next()方法，从而产生异常ConcurrentModificationException。请查看ArrayList.iterator()的原代码。

4. Hashtable 与 HashMap

按照算法惯例，Hashtable是数据结构的名称。但在Java中，数据结构的名称是HashMap。Hashtable是同步的版本。所以很多时候你并不需要Hashtable，而是HashMap。 这两篇文章详细介绍了各种Map的区别和常见的问题： HashMap vs. TreeMap vs. Hashtable vs. LinkedHashMap, Map常见10大问题,

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 in thread "main" java.lang.ClassCastException: java.lang.Integer cannot be cast to java.lang.String at ... 使用原始类型的Collection是危险的，因为原始类型的Collection跳过类型检查。另外值得一提的是Set, Set>, Set