惊呆了！Java程序员最常犯的错竟然是这10个

和绝大多数的程序员一样，我也非常的宅。周末最奢侈的享受就是逛一逛技术型网站，比如说 programcreek，这个小网站上有一些非常有意思的主题。比如说：java 程序员最常犯的错竟然是这 10 个，像这类令人好奇心想害死猫的主题，非常值得扒出来给大家分享一下。

ps：别问我“为什么标题要加上‘惊呆了’？”问了答案就只有一个——吓唬人——总得勾起大家的阅读兴趣嘛（我容易吗我）。

01、把 array 转成 arraylist

说实在的，很多 java 程序员喜欢把 array 转成 arraylist：

list<string> list = arrays.aslist(arr);

但实际上，arrays.aslist() 返回的 arraylist 并不是 java.util.arraylist，而是 arrays 的内部私有类 java.util.arrays.arraylist。虽然名字完全相同，都是 arraylist，但两个类有着很大的不同。arrays.arraylist 虽然有 set()、get() 和 contains() 等方法，但却没有一个方法用来添加元素，因此它的大小是固定的。

如果想创建一个真正的 arraylist，需要这样做：

list<string> list = new arraylist<string>(arrays.aslist(arr));

arraylist 的构造方法可以接收一个 collection 类型的参数，而 arrays.arraylist 是其子类，所以可以这样转化。

02、通过 set 检查数组中是否包含某个值

之前我在写一篇文章《如何检查java数组中是否包含某个值 》中曾提到一种方法：

set<string> set = new hashset<string>(arrays.aslist(arr));
return set.contains(targetvalue);

这种方法确实可行，但却忽视了性能问题；为了能够尽快完成检查，可以这样做：

arrays.aslist(arr).contains(targetvalue);

或者使用普通的 for 循环或者 for-each。

03、通过 for 循环删除列表中的元素

新手特列喜欢使用 for 循环删除列表中的元素，就像这样：

list<string> list = new arraylist<string>(arrays.aslist("沉", "默", "王", "二"));
for (int i = 0; i < list.size(); i++) {
    list.remove(i);
}
system.out.println(list);

上面这段代码的目的是把列表中的元素全部删除，但结果呢：

[默, 二]

竟然还有两个元素没删除，why？

当 list 的元素被删除时，其 size() 会减小，元素的下标也会改变，所以想通过 for 循环删除元素是行不通的。

那 for-each 呢？

for(string s : list) {
    if ("沉".equals(s)) {
       list.remove(s);
    }
}

system.out.println(list);

竟然还抛出异常了：

exception in thread "main" java.util.concurrentmodificationexception
    at java.util.arraylist$itr.checkforcomodification(arraylist.java:909)
    at java.util.arraylist$itr.next(arraylist.java:859)
    at com.cmower.java_demo.programcreek.top10mistake.main(top10mistake.java:15)

抛出异常的原因，可以查看我之前写的文章《java，你告诉我 fail-fast 是什么鬼？》。

有经验的程序员应该已经知道答案了，使用 iterator：

iterator<string> iter = list.iterator();
while (iter.hasnext()) {
    string s = iter.next();

    if (s.equals("沉")) {
        iter.remove();
    }
}

system.out.println(list);

程序输出的结果如下：

[默, 王, 二]

04、使用 hashtable 而不是 hashmap

通常来说，哈希表应该是 hashtable，但在 java 中，哈希表通常指的是 hashmap。两者之间的区别之一是 hashtable 是线程安全的。如果没有特殊要求的话，哈希表应该使用 hashmap 而不是 hashtable。

05、使用原始类型

在 java 中，新手很容易混淆无限通配符和原始类型之间的差别。举例来说，list<?> list 为无限通配符，list 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, 18);
    add(list, "沉默王二");
    string s = list.get(0);
}

这段代码在运行时会抛出异常：

exception in thread "main" java.lang.classcastexception: java.lang.integer cannot be cast to java.lang.string
    at com.cmower.java_demo.programcreek.top10mistake.main(top10mistake.java:38)

使用原始类型非常的危险，因为跳过了泛型的检查。至于 list<object> 和 list 之间的区别，查看我写的另外一篇文章：《为什么不应该使用java的原始类型》。

06、使用 public 修饰字段

有些新手喜欢使用 public 修饰字段，因为不需要 getter/setter 方法就可以访问字段。但实际上，这是一个非常糟糕的设计；有经验的程序员更习惯于提供尽可能低的访问级别。

07、使用 arraylist 而不是 linkedlist

新手往往搞不清楚 arraylist 和 linkedlist 之间的区别，因此更倾向于使用 arraylist，因为比较面熟。但是呢，它们之间存在巨大的性能差异。简单的说吧，如果“添加/删除”的操作比较多，而“获取”的操作比较少，则应该首选 linkedlist。

08、使用过多的不可变对象

不可变对象有着不少的优点，比如说简单性和安全性。但是呢，如你所料，它也有一些难以抗拒的弊端：对于每一个不同的值，它都需要一个单独的对象来表示，这样的对象太多的话，很可能会导致大量的垃圾，回收的成本就变得特别高。

为了在可变与不可变之间保持平衡，通常会使用可变对象来避免产生太多中间对象。一个经典的例子就是使用 stringbuilder（可变对象） 来连接大量的字符串，否则的话，string（不可变对象）会产生很多要回收的垃圾。

反例：

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

正例：

stringbuilder result = new stringbuilder();
for (string s: strs) {
    result.append(s);
}

参考文章：为什么 java 字符串是不可变的？

09、父类没有默认的无参构造方法

在 java 中，如果父类没有定义构造方法，则编译器会默认插入一个无参的构造方法；但如果在父类中定义了构造方法，则编译器不会再插入无参构造方法。所以下面的代码会在编译时出错。

子类中的无参构造方法试图调用父类的无参构造方法，但父类中并未定义，因此编译出错了。解决方案就是在父类中定义无参构造方法。

10、使用构造方法创建字符串

创建字符串有两种方法：

1）使用双引号

string er = "沉默王二";

2）使用构造方法

string san = new string("沉默王三");

但是它们之间有着很大的不同（可以参照创建 java 字符串，用""还是构造函数），双引号被称为字符串常量，可以避免重复内容的字符串在内存中创建。

好了，读者朋友们，以上就是本文的全部内容了。可以掏心窝子地说，没有任何客观的数据来证明它们就是前十名，但绝对非常普遍。如果不认可其中的内容，请在留言区轻喷，好人有好报。如果觉得不过瘾，还想看到更多，可以 star 二哥的 github【】，本文已收录。

原创不易，如果觉得有点用的话，请不要吝啬你手中点赞的权力——这将是我最强的写作动力。