十个JAVA程序员容易犯的错误

十个JAVA程序员容易犯的错误

1. Array 转 ArrayList

一般开发者喜欢用:

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

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

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

关于这个方法更多的解释,请参见我之前的一篇文章

http://www.cnblogs.com/tina-smile/p/5056174.html

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

一般开发者喜欢用:

Set<String> set = new HashSet<String>(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<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的大小就缩减了,索引也发生了改变。所以如果想要在一个循环中使用索引移除多个元素,这是不可行的。

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

ArrayList<String> list = new ArrayList<String>(Arrays.asList("a", "b", "c", "d"));

for (String s : list) {
    if (s.equals("a"))
        list.remove(s);
}   

会报错 ConcurrentModificationException.

正确的应该是:

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

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<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<?>, and Set 这三个有很大区别,稍后我再做解释。

6.访问权限

很多开发者使用public修饰类的字段,这样很容易获取字段值,但这是一个糟糕的设计。经验法则是给用户尽可能低的访问权限。

7.ArrayList vs. LinkedList

大部分的开发者并不知道ArrayList和LinkedList 的区别,所以经常使用ArrayList,这个看起来更熟悉。但是,在这两者之间,有一个很大的性能差异。

简单来说,在有很多插入和删除操作时,同时有很少或者几乎没有随机访问的操作情况下,推荐使用LinkedList。其次,再使用ArrayList。

8.Mutable vs. Immutable(可变 VS 不可变)

不可变对象有很多优势这样简单,安全,等等。但它对于每一个不同的变量,需要一个单独的对象。太多的对象可能会导致高成本的垃圾收集。所以在选择可变和不可变的时应该有个权衡。

一般来说,使用可变对象,以避免产生过多的中间对象。一个经典的例子就是连接大量的字符串。如果使用不可变的String,你会产生很多立马要被垃圾收集回收的对象,这很消耗时间和精力。如果使用一个可变的,类似StringBuffer。

String result="";
for(String s: arr){
    result = result + s;
}

可变对象还有其他的例子:函数传参的过程中,可以传入可变对象,这样可以得到多个结果。

9.父类和子类的构建

十个JAVA程序员容易犯的错误_第1张图片

编译出错的原因主要是默认的Super类的构造函数并没有被定义。在Java中,如果一个类没有实现构造函数,编译器默认会给这个类插入一个无参构造函数。如果在父类中已经有了构造函数,那么编译器将不会再插入默认无参构造函数。

子类的构造函数,要么是带参的要么是无参的,都会调用父类的无参构造。因为编译器想在子类中,加入super(),但是父类的无参构造函数并不存在。所以,编译器会报错。

为了解决这个问题,有两种解决办法,第一,在Super类中,加入无参构造:

public Super(){
    System.out.println("Super");
}

第二种方式,移除自定义的父类构造函数

第三种方式,子类的构造函数中加入super(value)

10."" or Constructor?

创建一个String 有两种方式:

//1. use double quotes
String x = "abc";
//2. use constructor
String y = new String("abc");

区别是什么?

String a = "abcd";
String b = "abcd";
System.out.println(a == b);  // True
System.out.println(a.equals(b)); // True
 
String c = new String("abcd");
String d = new String("abcd");
System.out.println(c == d);  // False
System.out.println(c.equals(d)); // True

本文是翻译自

http://www.programcreek.com/2014/05/top-10-mistakes-java-developers-make/

posted @ 2016-01-08 16:29 smile_tina 阅读( ...) 评论( ...) 编辑 收藏
刷新评论 刷新页面 返回顶部

你可能感兴趣的:(十个JAVA程序员容易犯的错误)