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

最新推荐文章于 2025-08-14 09:11:10 发布

翻译最新推荐文章于 2025-08-14 09:11:10 发布 · 523 阅读

文章标签：

#java

java 专栏收录该内容

26 篇文章

订阅专栏

本文列举了Java开发过程中常见的十个误区，包括如何正确地从数组转换到ArrayList、如何检测数组是否包含某个元素、如何在循环中移除List中的元素等。这些误区可能会导致代码效率低下或者出现运行时错误。

摘要生成于 C知道，由 DeepSeek-R1 满血版支持，前往体验 >

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

本文是翻译自

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

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.父类和子类的构建

编译出错的原因主要是默认的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