Java开发人员犯的10大错误

一、将数组转换为ArrayList
要将数组转换为ArrayList，开发人员通常会这样做：

List list = Arrays.asList(arr);
Arrays.asList()将返回 ArrayList私有静态类的Arrays，而不是java.util.ArrayList类。该java.util.Arrays.ArrayList有set()，get()，contains()方法，但没有添加元素的任何方法，所以它的大小是固定的。要创建一个real ArrayList，您应该执行以下操作：

ArrayList arrayList = new ArrayList(Arrays.asList(arr));
构造函数 ArrayList可以接收 Collection类型，它也是超类型 java.util.Arrays.ArrayList。

二、检查数组是否包含值
开发人员经常这样做：

Set set = new HashSet(Arrays.asList(arr));
return set.contains(targetValue);
该代码有效，但是无需先转换列表即可设置。将列表转换为集合需要额外的时间。它可以很简单：

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

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

三、从循环内的列表中删除元素
考虑以下代码，该代码在迭代期间删除元素：

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，因为它去检查list前后的大小的时候，发现不相等就会跑错，具体源码中有体现。相反，可以执行以下操作：

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()的源代码：

...

public Iterator iterator() {
return new Itr();
}

/**
* AbstractList.Itr的优化版本
*/
private class Itr implements Iterator {
int cursor; // 下一元素的索引返回
int lastRet = -1; // 返回的最后一个元素的索引，如果没有这为 -1
int expectedModCount = modCount;

public boolean hasNext() {
return cursor != size;
}

@SuppressWarnings("unchecked")
public E next() {
checkForComodification();
int i = cursor;
if (i >= size)
throw new NoSuchElementException();
Object[] elementData = ArrayList.this.elementData;
if (i >= elementData.length)
throw new ConcurrentModificationException();
cursor = i + 1;
return (E) elementData[lastRet = i];
}

public void remove() {
if (lastRet < 0)
throw new IllegalStateException();
checkForComodification();

try {
ArrayList.this.remove(lastRet);
cursor = lastRet;
lastRet = -1;
expectedModCount = modCount;
} catch (IndexOutOfBoundsException ex) {
throw new ConcurrentModificationException();
}
}

final void checkForComodification() {
if (modCount != expectedModCount)
throw new ConcurrentModificationException();
}
}

...
四、HashTable 、HashMap、LinkedHashMap、TreeMap
HashMap 和 HashTable之间的主要区别是 HashTable同步（所有的读写等操作都进行了锁（synchronized）保护，在多线程环境下没有安全问题。但是锁保护也是有代价的，会对读写的效率产生较大影响）。因此，通常建议不使用 HashTable，而使用 HashMap。


Java SE中有4种常用的Map实现-HashMap，TreeMap，Hashtable和LinkedHashMap。如果我们仅使用一个句子来描述每个实现，则将是以下内容：这就是如果程序是线程安全的，则应使用 HashMap的原因。
【1】HashMap被实现为哈希表，并且键或值没有排序。
【2】TreeMap是基于红黑树结构实现的，并通过 key进行排序。
【3】LinkedHashMap保留插入顺序
【4】与 HashMap相比，Hashtable是同步的。同步有开销。
如果 HashMap的键是自定义对象，则需要遵循equals（）和hashCode（）协定。

class Dog {
String color;

Dog(String c) {
color = c;
}
public String toString(){
return color + " dog";
}
}

public class TestHashMap {
public static void main(String[] args) {
HashMap hashMap = new HashMap();
Dog d1 = new Dog("red");
Dog d2 = new Dog("black");
Dog d3 = new Dog("white");
Dog d4 = new Dog("white");

hashMap.put(d1, 10);
hashMap.put(d2, 15);
hashMap.put(d3, 5);
hashMap.put(d4, 20);

//print size
System.out.println(hashMap.size());

//loop HashMap
for (Entry entry : hashMap.entrySet()) {
System.out.println(entry.getKey().toString() + " - " + entry.getValue());
}
}
}
【输出结果】 ：

4
white dog - 5
black dog - 15
red dog - 10
white dog - 20

注意这里，我们错误地两次添加了“white dog”，但是 HashMap接受了它。这没有道理，因为现在我们对真正有多少只 white dog感到困惑。Dog类应定义如下：

class Dog {
String color;

Dog(String c) {
color = c;
}

public boolean equals(Object o) {
return ((Dog) o).color.equals(this.color);
}

public int hashCode() {
return color.length();
}

public String toString(){
return color + " dog";
}
}
【输出结果】 ：

3
red dog - 10
white dog - 20
black dog - 15

原因是 HashMap不允许两个相同的元素。默认情况下，使用在 Object类中实现的hashCode()和equals()方法。默认的hashCode()方法为不同的对象提供不同的整数，而equals()方法仅在两个引用引用同一对象时才返回true。所以hashCode()和equals()方法校验结果不相同。如果重写了此方法，就会返回true，过滤掉多余的 white dog 。

五、使用原始集合类型
在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);
}
此代码将引发异常：

线程“主”中的异常java.lang.ClassCastException：无法将java.lang.Integer强制转换为java.lang.String
...
使用原始类型集合很危险，因为原始类型集合会跳过泛型类型检查并且不安全。之间存在巨大差异Set，Set和Set