Java开发人员经常犯的10大错误

我们在这里总结了Java开发人员经常犯的十大错误，看看你中了几个？

1、将Array转换为ArrayList

要将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));

构造函数ArrayList可以接受Collection类型，它也是一个超类型java.util.Arrays.ArrayList。

2、检查数组是否包含指定值

开发人员经常这样做：

Set<String> set = new HashSet<String>(Arrays.asList(arr));
return set.contains(targetValue);

代码可以工作，但不需要先将列表转换为set。将列表转换为集合需要额外的时间。它可以很简单：

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。

4、Hashtable vs HashMap

根据算法中的约定，Hashtable是数据结构的名称。但在Java中，数据结构的名称是HashMap。Hashtable和HashMap之间的关键区别之一是Hashtable是同步。所以你经常不需要Hashtable，而HashMap经常使用。

5、使用原始类型的集合

在Java中，原始类型 和 无界通配符类型 很容易混合在一起。以Set为例，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<?>和Set<Object>。

6、访问权限

开发人员经常将public用于所有的类成员变量。通过直接引用很容易获得字段值，但这是一个非常糟糕的设计。经验法则是为成员提供尽可能低的访问级别。

7、ArrayList与LinkedList

当开发人员不知道ArrayList和LinkedList之间的区别时，他们经常使用ArrayList，因为它看起来很熟悉。但是，它们之间存在巨大的性能差异。简而言之，如果存在大量的添加/删除操作并且没有大量随机访问操作，则应该首选LinkedList。

8、可变与不可变

不可变对象具有许多优点，例如简单性，安全性等。但是对于每个不同的值，它需要单独的对象，并且太多的对象可能导致高的垃圾收集成本。在可变和不可变之间进行选择时应该保持平衡。

通常，可变对象用于避免产生太多中间对象。一个典型的例子是连接大量字符串。如果使用不可变字符串，则会产生大量符合垃圾收集条件的对象。这会浪费CPU上的时间和精力，使用可变对象正确的解决方案（例如StringBuilder）。

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

在需要可变对象时还有其他情况。例如，将可变对象传递给方法可以让您收集多个结果，而不会跳过太多语法箍。另一个例子是排序和过滤：当然，您可以创建一个获取原始集合的方法，并返回一个已排序的方法，但这对于较大的集合来说会变得非常浪费。

9、Super和Sub的构造函数

发生此编译错误，因为默认的超级构造函数是未定义的。在Java中，如果类没有定义构造函数，编译器将默认为该类插入默认的无参数构造函数。如果构造函数是在Super类中定义的，在本例中是Super（String s），编译器将不会插入默认的无参数构造函数。这是上面超级类的情况。

Sub类的构造函数（带参数或无参数）将调用无参数的超级构造函数。由于编译器尝试将super() 插入Sub类中的2个构造函数，但未定义Super的默认构造函数，因此编译器会报告错误消息。

要解决这个问题，只需1）将Super() 构造函数添加到Super类中即可

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

，或2）删除自定义的超级构造函数，或3）添加super(value)到子构造函数。

10、" "或构造函数？

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

//1. 使用双引号
String x = "abc";
//2. 使用构造函数
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

最后

这是我基于对GitHub上的大量开源项目、Stack Overflow问题的分析。没有评估和证明它们恰好是前10名，但绝对是非常普遍的。

如果您不赞同部分内容，请留下您的评论。如果你能指出一些更常见的其他错误，我将非常感谢。

“不积跬步，无以至千里”，希望未来的你能：有梦为马 随处可栖！加油，少年！

---

关注公众号:「Java 知己」，每天更新Java知识哦，期待你的到来！

• 发送「1024」，免费领取 30 本经典编程书籍。
• 发送「Group」，与 10 万程序员一起进步。
• 发送「面试」，领取BATJ面试资料、面试视频攻略。
• 发送「JavaEE 实战」，领取《JavaEE 实战》系列视频教程。
• 发送「玩转算法」，领取《玩转算法》系列视频教程。