java程序员们最常犯的10个错误 -买球官网平台

21顶
0踩

java程序员们最常犯的10个错误

2014-06-25 10:32 by 见习记者 评论(14) 有33871人浏览
1.将数组转化为列表

将数组转化为一个列表时,程序员们经常这样做:

list list = arrays.aslist(arr);

arrays.aslist()会返回一个arraylist对象,arraylist类是arrays的一个私有静态类,而不是java.util.arraylist类,java.util.arrays.arraylist类有set()、get()、contains()方法,但是没有增加元素的方法,所以它的大小是固定的,想要创建一个真正的arraylist类,你应该这样做:

arraylist arraylist = new arraylist(arrays.aslist(arr));

arraylist的构造方法可以接受一个集合类型,刚好它也是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;

第一种方法比第二种更容易读

3.在一个循环中删除一个列表中的元素

思考下面这一段在循环中删除多个元素的的代码

arraylist list = new arraylist(arrays.aslist("a","b","c","d"));
for(int i=0;i

输出结果是:

[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与hashmap的对比

就算法而言,哈希表是数据结构的一个名字。但是在java中,这个数据结构的名字是hashmap。hashtable与hashmap的一个重要不同点是hashtable是同步的。所以你经常不需要hashtable,相反hashmap经常会用到。

具体请看:

5.在集合中使用原始类型

在java中原始类型与无界通配符类型很容易混合在一起,拿set来说,set是一个原始类型,而set是无界的通配符类型。
考虑下面使用原始类型list作为参数的代码:

public static void add(list list,object o){
    list.add(o);
}
pulbic 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