java时常犯错总结

向右看齐 2022-01-27 14:53 223阅读 0赞

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

List<String> list = Arrays.asList(arr);

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

ArrayList<String> arrayList = new ArrayList<String>(Arrays.asList(arr));

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

**判断一个数组是否包含某个值**  
开发人员经常这样做：

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

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

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

或者

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

第一种方法可读性更强。

**在循环内部删除List中的一个元素**  
考虑如下代码，在迭代期间删除元素：

ArrayList<String> list = new ArrayList<String>(Arrays.asList("a", "b", "c","d"));
    
        for (int i = 0; i < list.size(); i++) {
            list.remove(i);
        }
        System.out.println(list);

结果打印：  
\[b, d\]

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

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

ArrayList<String> list = new ArrayList<String>(Arrays.asList("a", "b", "c","d"));
    
        for (String s : list) {
            if (s.equals("a"))
                list.remove(s);
        }

将抛出`ConcurrentModificationException`异常。  
然而接下来的代码却是OK的：

ArrayList<String> list = new ArrayList<String>(Arrays.asList("a", "b", "c","d"));
    
        Iterator<String> iter = list.iterator();
    
        while (iter.hasNext()) {
            String s = iter.next();
            if (s.equals("a")) {
                iter.remove();
            }
        }

next()方法需要在remove()方法之前被调用，在foreach循环里，编译器会在删除元素操作化调用next方法，这导致了ConcurrentModificationException异常。更多详细信息，可以查看ArrayList.iterator()的源码。

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

更多信息：HashMap vs. TreeMap vs. Hashtable vs. LinkedHashMap Top 10 questions about Map

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

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

public static void add(List list, Object o){
            list.add(o);
        }
    
        public static void main(String[] args){
            List<String> list = new ArrayList<String>();
            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这三个有很大的不同，具体请看：类型擦除和Raw type vs. Unbounded wildcard。

[http://www.cnblogs.com/chenpi/p/5508177.html][http_www.cnblogs.com_chenpi_p_5508177.html]

[http://www.programcreek.com/2013/12/raw-type-set-vs-unbounded-wildcard-set/][http_www.programcreek.com_2013_12_raw-type-set-vs-unbounded-wildcard-set]

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

相关阅读：public, default, protected, and private

[http://www.cnblogs.com/chenpi/p/5488202.html][http_www.cnblogs.com_chenpi_p_5488202.html]

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