Java开发人员最常犯的9个错误

1.Array转ArrayList

当需要把Array转成ArrayList的时候，开发人员经常这样做：

String [] strArray = {"0" , "1" , "2"};
List<String> list = Arrays.asList(intArray);

Arrays.asList会返回一个ArrayList，但是要特别注意，这个ArrayList是Arrays类的静态内部类，并不是java.util.ArrayList类。java.util.Arrays.ArrayList类实现了set，get，contains方法，但是并没有实现增加元素的方法（事实上是可以调用add方法，但是没有具体实现，仅仅抛出UnsupportedOperationException异常），因此它的大小也是固定不变的。

Caused by: java.lang.UnsupportedOperationException
        at java.util.AbstractList.add(AbstractList.java:404)
        at java.util.AbstractList.add(AbstractList.java:425)
        at com.example.testcommon.MainActivity.onCreate(MainActivity.java:20)

为了创建一个真正的java.util.ArrayList，你应该这样做：

String [] strArray = {"0" , "1" , "2"};
ArrayList<String> list = new ArrayList<>(Arrays.asList(strArray));

ArrayList的构造方法可以接收一个Collection类型。
而java.util.Arrays.ArrayList已经实现了该接口。

2.判断一个数组是否包含某个值

开发人员经常这样做：

public boolean isContains(String [] strArray , String targetValue){
        Set<String> set = new HashSet<>(Arrays.asList(strArray));
        return set.contains(targetValue);
}

以上代码可以正常工作，但是没有必要将其转换成set集合，将一个List转成Set需要额外的时间，其实我们可以简单的使用如下方法即可：

public boolean isContains(String [] strArray , String targetValue){
        return Arrays.asList(strArray).contains(targetValue);
}

或者：

    public boolean isContains(String [] strArray , String targetValue){
        for (String value : strArray) {
            if (value.equals(targetValue)){
                return true;
            }
        }
        return false;
    }

3.在循环内部删除List中的一个元素

考虑如下代码，在迭代期间删除元素：

    public List<String> removeObject(){
        ArrayList<String> arrayList = new ArrayList<>(Arrays.asList("a" , "b" , "c" , "d"));
        for (int i = 0; i < arrayList.size(); i++) {
            arrayList.remove(i);
        }
        return arrayList;
    }

结果打印：[b, d]

在上面这个方法中有一系列的问题，当一个元素被删除的时候，list大小减小，然后原先索引指向了其它元素。所以如果你想在循环里通过索引来删除多个元素，将不会正确工作。

你也许知道使用迭代器是在循环里删除元素的正确方式，或许你也知道foreach循环跟迭代器很类似，但事实情况却不是这样，如下代码：

    public List<String> removeObject(){
        ArrayList<String> arrayList = new ArrayList<>(Arrays.asList("a" , "b" , "c" , "d"));
        for (String value : arrayList) {
            if (value.equals("a")){
                arrayList.remove(value);
            }
        }
        return arrayList;
    }

但是却会抛出异常：

     Caused by: java.util.ConcurrentModificationException
        at java.util.ArrayList$ArrayListIterator.next(ArrayList.java:573)
        at com.example.testcommon.MainActivity.removeObject(MainActivity.java:37)
        at com.example.testcommon.MainActivity.onCreate(MainActivity.java:23)

然而接下来的代码却是OK的：

    public List<String> removeObject(){
        ArrayList<String> arrayList = new ArrayList<>(Arrays.asList("a" , "b" , "c" , "d"));
        Iterator<String> iterator = arrayList.iterator();
        while (iterator.hasNext()){
            String value = iterator.next();
            if (value.equals("a")){
                iterator.remove();
            }
        }
        return arrayList;
    }

next方法需要在remove方法之前被调用，在foreach循环里，编译器会在删除元素操作后调用next方法，这导致了ConcurrentModificationException异常。

4.HashTable与HashMap

从算法的角度来讲，HashTable是一种数据结构名称。但是在Java中，这种数据结构叫做HashMap。HashTable与HashMap的一个主要的区别是HashTable是同步的，所以，通常来说，你会使用HashMap，而不是Hashtable。

5.使用集合原始类型（raw type）

在Java中，原始类型（raw type）和无界通配符类型很容易让人混淆。举个Set的例子，Set是原始类型，而Set是无界通配符类型。

请看如下代码，add方法使用了一个原始类型的List作为入参：

    public void add(List list , Object value){
        list.add(value);
    }

    public void test(){
        List<String> list = new ArrayList<>();
        add(list , 0);
        String s = list.get(0);
    }

运行test函数会抛出异常：

Caused by: java.lang.ClassCastException: java.lang.Integer cannot be cast to java.lang.String
        at com.example.testcommon.MainActivity.test(MainActivity.java:57)
        at com.example.testcommon.MainActivity.onCreate(MainActivity.java:26)

使用原始类型集合非常危险，因为它跳过了泛型类型检查，是不安全的。另外，Set, Set, 和Set这三个有很大的不同。

6.访问级别

开发人员经常使用public修饰类字段，虽然这很容易让别人直接通过引用获取该字段的值，但这是一个不好的设计。根据经验，应该尽可能的降低成员属性的访问级别。

7.ArrayList和LinkedList

为什么开发人员经常使用ArrayList和LinkedList，却不知道他们之间的区别，因为它们看起来很像。然而它们之间有着巨大的性能差异。简单的说，如果有大量的增加删除操作并且没有很多的随机访问元素的操作，应该首选LinkedList。否则反之。

8.可变与不可变

不可变对象有很多优点，如简单、安全等。但是对于每个不同的值都需要一个单独的对象，太多的对象会引起大量垃圾回收，因此在选择可变与不可变的时候，需要有一个平衡。

通常，可变对象用于避免产生大量的中间对象，一个经典的例子是大量字符串的拼接。如果你使用一个不可变对象，将会马上产生大量符合垃圾回收标准的对象，这浪费了CPU大量的时间和精力。使用可变对象是正确的解决方案（StringBuilder）；

另外，在有些其它情况下也是需要使用可变对象。例如往一个方法传入一个可变对象，然后收集多种结果，而不需要写太多的语法。另一个例子是排序和过滤：当然，你可以写一个方法来接收原始的集合，并且返回一个排好序的集合，但是那样对于大的集合就太浪费了。