99%的Java程序员都会写这样的垃圾代码

在软件开发领域，编程经验往往与设计、编码、重构和测试的能力相辅相成。随着时间的推移，这些技能的提升使您能够在日常工作中脱颖而出。然而，有时候我们可能陷入固定的编程模式，导致编码习惯停滞不前。在这篇文章中，我们将介绍10个Java编程习惯，它们可以帮助您提高编码技能，写出更加干净、健壮的Java代码。

1、调用equals()方法时使用String字面值或已知对象

这条肯定中！很多人以为由于equals()方法是对称的，因此调用 a.equals(b) 与调用b.equals(a)相同的，所以习惯性的这样写：

if (givenString.equals("YES")){
    // 执行一些操作
}

尽管这种写法在可读性上有优势，但它并不安全。如果givenString为null，那么这段代码将抛出NullPointerException。为了避免这种情况，我们应该将equals方法调用放在已知对象的一侧，如下所示：

"YES".equals(givenString)

这样，如果givenString为null，它将返回false，而不会抛出异常。这是一种更加安全和健壮的编码习惯，同时也是避免NullPointerException的一种流行方式。

2、使用entrySet遍历HashMap

在遍历HashMap时，我们通常会使用键集合（key set）来获取键，并通过键获取对应的值。例如：

Set keySet = map.keySet();
for (Key k : keySet){
    Value v = map.get(k);
    System.out.println(k + ": " + v);
}

然而，这种方法需要进行两次查找操作，可能会导致性能下降。如果我们需要同时访问键和值，更好的方式是使用entrySet，如下：

Set entrySet = map.entrySet();
for (Map.Entry entry : entrySet){
    Key k = entry.getKey();
    Value v = entry.getValue();
    System.out.println(k + ": " + v);
}

这种方式效率更高，因为它直接从entry对象中获取值，而不需要再次查找。

3、使用枚举作为单例

想象一下，您需要创建一个线程安全的单例模式。以前，这可能需要大量的代码和同步操作。但现在，您可以使用Java的枚举类型，仅需一行代码即可创建一个线程安全的单例：

public enum Singleton{
  INSTANCE;
}

这个枚举实例在多线程环境下也能保证只有一个实例存在，即使在序列化和反序列化过程中也是如此。这是一种简洁而强大的单例模式的实现方式。

4、使用Arrays.asList()或List.of()初始化集合

在Java中，初始化集合时，我们通常会逐个添加元素。例如：

List listOfCurrencies = new ArrayList();
listOfCurrencies.add("USD/AUD");
listOfCurrencies.add("USD/JPY");
listOfCurrencies.add("USD/INR");

这种方法虽然有效，但相对繁琐。您可以使用Arrays.asList()方法以更简洁的方式初始化集合，如下：

List listOfPairs = new ArrayList(Arrays.asList("USD/AUD", "USD/JPY", "USD/INR"));

此外，从Java 9开始，您还可以使用List.of()和Set.of()方法来创建不可变的列表和集合。这些方法提供了更好的不可变性保证。

List newList = List.of("One", "Two", "Infinity");

5、在循环中检查wait()条件

当我们使用wait()、notify()和notifyAll()方法进行多线程通信时，通常会在if语句中检查等待条件，然后调用wait()。例如：

synchronized(queue) {
  if(queue.isFull()){
    queue.wait();
  }
}

然而，这种写法存在一个问题，即可能会发生虚假通知（spurious notification）。为了解决这个问题，我们应该将检查等待条件的操作放在一个while循环内，如下：

synchronized(queue) {
  while(queue.isFull()){
    queue.wait();
  }
}

这样，即使在通知之前等待条件再次被满足，我们的代码也可以正确地工作。

6、捕获CloneNotSupportedException并返回子类实例

在Java中，对象克隆的实现机制常常受到批评，因为它的性能不佳。如果您需要实现clone()方法，可以使用以下习惯来减轻这种痛苦：

public Course clone() {
   Course c = null;
   try {
     c = (Course)super.clone();
   } catch (CloneNotSupportedException e) {} // 不会发生
   return c;
}

这个习惯利用了clone()方法实际上不会抛出CloneNotSupportedException的事实，只要类实现了Cloneable接口。这种方式返回了子类的实例，被称为协变方法覆盖（covariant method overriding），从Java 5开始支持。这可以减少客户端代码中的类型强制转换，使代码更加清晰。

7、在可能的情况下使用接口

在定义方法的返回类型、变量类型或方法参数类型时，应尽量使用接口而不是具体

的类。例如，不要这样写：

ArrayList listOfNumbers = new ArrayList();
public ArrayList getNumbers(){
    return listOfNumbers;
}
public void setNumbers(ArrayList numbers){
    listOfNumbers = numbers;
}

而应该这样写：

List listOfNumbers;
public List getNumbers(){
    return listOfNumbers;
}
public void setNumbers(List numbers){
    listOfNumbers = numbers;
}

这种方式提供了更大的灵活性，可以传递不同的集合实现。您还可以使用泛型中的通配符扩展功能，进一步提高灵活性。

public void processList(List