10.1Java开发人员犯的十大错误

本列表总结了Java开发人员经常犯的十大错误。

1.转换数组为ArrayList

将数组转成一个ArrayList，开发者经常这样做：
List list = Arrays.asList(arr);

Arrays.asList()将返回一个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. 检查数组是否包含值

开发人员经常这样做：

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

代码的可以工作，但没有必要先转换列表。 将列表转换为集合需要额外的时间。 它可以像下面这样简单：

Arrays.asList(arr).contains(targetValue);

或者

for(String s: arr){
    if(s.equals(targetValue))
        return true;
}
return false;

代码1的可读性更好，特别注意其实这个包含检测其实非常耗时间的。

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]

这种方法存在一个严重的问题。 当一个元素被删除，列表的大小缩小，索引改变。 所以，如果你想通过使用索引删除一个循环内的多个元素，这将无法正常工作。

您可能知道使用迭代器是删除循环内部元素的正确方法，并且您知道Java中的foreach循环像迭代器一样工作，但事实上并非如此。 考虑下面的代码：

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. Hashtable vs HashMap

按照算法中的约定，Hashtable是数据结构的名称。 但是在Java中，数据结构的名字是HashMap。 Hashtable和HashMap之间的一个主要区别是Hashtable是同步的。 所以很多时候你不需要Hashtable，而应该使用HashMap。

HashMap vs. TreeMap vs. Hashtable vs. LinkedHashMap
Top 10 questions about Map

5. 使用原始类型的集合

在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 ...

使用原始类型集合是危险的，因为原始类型集合跳过了通用类型检查而不安全。 Set，Set <？>和Set