Java程序员容易犯的错误

标签：方法   程序   rem   add   ast   equal   迭代器   java程序员   开发者   

List list = Arrays.asList(arr);

Arrays.asList() 会返回一个ArrayList，这是Arrays里内嵌的一个私有静态类，而并不是java.util.ArrayList类java.util.Arrays.ArrayList 有set(), get(), contains()方法，但并支持添加元素，所以大小是固定的，想要创建一个真正的ArrayList，你应该：

ArrayList arrayList = new ArrayList(Arrays.asList(arr));

2.检测一个Array是否包含一个元素

一般开发者喜欢用：

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

return set.contains(targetValue);

这段代码能够工作，但并没有先转将list转为set，list转为set需要额外时间，可以用下面更简单的：

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

或者

for(String s: arr){

if(s.equals(targetValue))

return true;

}

return false;

第一种方法比第二种的可读性更好。

3. 在一个循环中移除list中的元素

考虑下下面这种方法：

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]

但这种方法有一个严重的问题，当一个元素被移除的时候，list的大小就缩减了，索引也发生了改变。所以如果想要在一个循环中使用索引移除多个元素，这是不可行的。

或许你又想用迭代器的方式解决这个问题，但实际上也不可行，考虑下下面的代码：

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();

}

}

4. Hashtable vs HashMap

在算法中，哈希表是数据结构的名称。但在Java中，数据结构的名字是HashMap中。其中一个Hashtable和HashMap的之间的主要区别是，Hashtable是同步的。所以，很多时候，你并不需要哈希表，HashMap就够用了。

5.使用原始类型的集合

在Java中，原始类型和***通配符类型很容易混在一起。以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, and Set

Java程序员容易犯的错误

标签：方法   程序   rem   add   ast   equal   迭代器   java程序员   开发者   

原文地址：https://blog.51cto.com/14512197/2443845