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

21顶
0踩

2014-06-25 10:32 by 见习记者 tianxiataguo 评论(14) 有34293人浏览

java 程序员编程经验

1.将数组转化为列表

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

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

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

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

ArrayList的构造方法可以接受一个集合类型，刚好它也是java.util.Arrays.ArrayList的超类。

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

程序员们经常这样做：

Set<String> set = new HashSet<String>(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<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]

在这个方法中有一个严重的错误。当一个元素被删除时，列表的大小缩小并且下标变化，所以当你想要在一个循环中用下标删除多个元素的时候，它并不会正常的生效。

你也许知道在循环中正确的删除多个元素的方法是使用迭代，并且你知道java中的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异常。

相反下面的显示正常：

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()的源代码。

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<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<Object>中有很大的不同，具体请看Raw type vs. Unbounded wildcard和Type Erasure。

6.访问级别

程序员们经常使用public作为类字段的修饰符，可以很简单的通过引用得到值，但这是一个坏的设计，按照经验，分配给成员变量的访问级别应该尽可能的低。

具体请看：public, default, protected, and private

7.ArrayList与LinkedList的对比

当程序员们不知道ArrayList与LinkedList的区别时，他们经常使用ArrayList，因为它看起来比较熟悉。然而，它们之前有巨大的性能差别。简而言之，如果有大量的增加删除操作并且没有很多的随机访问元素的操作，应该首先LinkedList。如果你刚接触它们，请查看ArrayList vs. LinkedList来获得更多关于它们性能的信息。

8.可变与不可变

不可变对象有许多的优点，比如简单，安全等等。但是对于每一个不同的值都需要一个独立的对象，太多的对象可能会造成大量的垃圾回收。当选择可变与不可变时应该有一个平衡。
一般的，可变对象用来避免产生大量的中间对象。一个典型的例子是连接大量的字符串。如果你用一个不可变的字符串，你会产生很多需要进行垃圾回收的对象。这很浪费CPU的时间，使用可变对象是正确的解决方案(比如StringBuilder)。

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

有时在某些情况下也是需要可变对象的，比如将可变对象作为参数传入方法，你不用使用很多语句便可以得到多个结果。另外一个例子是排序和过滤：当然，你可以写一个方法来接收原始的集合，并且返回一个排好序的集合，但是那样对于大的集合就太浪费了。(来自StackOverFlow的dasblinkenlight’s的答案)

具体请看：Why String is Immutable?

9.父类与子类的构造函数

这个编译期错误的出现是父类默认的构造方法未定义，在java中，如果一个类没有定义构造方法，编译器会默认的为这个类添加一个无参的构造方法。如果在父类中定义了构造方法，在这个例子中是Super(String s),编译器就不会添加默认的无参构造方法，这就是上面这个父类的情形。

子类的构造器，不管是无参还有有参，都会调用父类的无参构造器。因为编译器试图在子类的两个构造方法中添加super()方法。但是父类默认的构造方法未定义，编译器就会报出这个错误信息。

想要修复这个问题，可以简单的通过1)在父类中添加一个Super()构造方法，像下面这样：

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

或者2)移除父类自定义的构造方法，或者3)在子类的构造方法中调用父类的super(value)方法。

具体请看：Constructor of Super and Stub

10.使用" "还是构造器

有两种方式可以创建字符串：

//1.使用字符串
String x = "abc";
//2.使用构造器
String y = new String("abc");

有什么区别？

下面的例子会给出一个快速的答案：

String a = "abc";
String b = "abc";
System.out.println(a==b);//true
System.out.println(a.equals(b));//true
 
String c = new String("abc");
String d = new String("abc");
System.out.println(a==b);//false
System.out.println(a.equals(b));//true

关于它们内存分配的更多信息，请参考Create Java String Using ” ” or Constructor?.

将来的工作

这个列表是我基于大量的github上的开源项目，Stack overflow上的问题，还有一些流行的google搜索的分析。没有明显示的评估证明它们是前10，但它们绝对是很常见的。如果您不同意任一部分，请留下您的评论。如果您能提出其它一些常见的错误，我将会非常感激。

原文链接： programcreek 翻译： ImportNew.com - 林林
译文链接： http://www.importnew.com/12074.html

查看图片附件

来自: ImportNew

分享到：

21
顶

0
踩

评论共 14 条请登录后发表评论

14 楼 ayanami001 2015-06-29 17:53

ylxg12345 写道

最后那段比较的
String c = new String("abc");
String d = new String("abc");
System.out.println(a==b);//false
System.out.println(a.equals(b));//true

望修改！

不要在意那些细节

13 楼 ylxg12345 2014-07-07 14:45

最后那段比较的
String c = new String("abc");
String d = new String("abc");
System.out.println(a==b);//false
System.out.println(a.equals(b));//true

望修改！

12 楼 cbyyni 2014-07-04 14:14

11 楼 kidneyball 2014-07-04 07:59

mfkvfn 写道

第2个。for循环查找比转换为List然后调用contains好。
第3个可以用for循环（从后往前循环）来删除元素。

第2个，其实用Array.asList来wrap一下并不会损耗什么性能（参考第1点，Array.asList返回的是一个固定长度的List，直接把所有list操作代理到原本的数组上了）。而使用Array.asList返回的List来查找正确处理了null值的情况：

        public int indexOf(Object o) {
            if (o==null) {
                for (int i=0; i<a.length; i++)
                    if (a[i]==null)
                        return i;
            } else {
                for (int i=0; i<a.length; i++)
                    if (o.equals(a[i]))
                        return i;
            }
            return -1;
        }

        public boolean contains(Object o) {
            return indexOf(o) != -1;
        }

文中的简单循环在数组里有null值时会抛NPE的。

10 楼 lvwenwen 2014-07-03 11:31

总结得挺好

9 楼 bluky999 2014-07-02 13:38

10.

String c = new String("abc");
String d = new String("abc");
System.out.println(c==d);//false
System.out.println(c.equals(d));//true

a->c
b->d

8 楼 fengjianquan9527 2014-06-30 12:27

7 楼 liuxmi 2014-06-29 19:34

凉风suzukaze 写道

o_o_0 写道

Integer a = 1;
Integer b = 1;
System.out.println(a==b);//true

Integer c = 200;
Integer d = 200;
System.out.println(c==d);//false

这是Integer类的缓存问题，一般开发中不怎么用得到吧~

对象类型比较统一用equals方法，只有初学者才会用==来比较

6 楼凉风suzukaze 2014-06-28 15:19

o_o_0 写道

Integer a = 1;
Integer b = 1;
System.out.println(a==b);//true

Integer c = 200;
Integer d = 200;
System.out.println(c==d);//false

这是Integer类的缓存问题，一般开发中不怎么用得到吧~

5 楼 o_o_0 2014-06-28 11:32

Integer a = 1;
Integer b = 1;
System.out.println(a==b);//true

Integer c = 200;
Integer d = 200;
System.out.println(c==d);//false

4 楼 wussrc 2014-06-27 13:29

写的非常好，平常开发中确实没有注意过这些细节问题。

3 楼 shadowlin 2014-06-26 09:53

mfkvfn 写道

第2个。for循环查找比转换为List然后调用contains好。
第3个可以用for循环（从后往前循环）来删除元素。

还是用迭代器保险一点

2 楼 operating...mydream 2014-06-26 08:45

总结得挺好的LZ get

1 楼 mfkvfn 2014-06-25 11:56

第2个。for循环查找比转换为List然后调用contains好。
第3个可以用for循环（从后往前循环）来删除元素。

发表评论

您还没有登录,请您登录后再发表评论

21顶0踩