Question

Let’s say I have this:

public class Whatever {
   private ArrayList<String> myList = new ArrayList<String>();
   // more code goes here
}

or let’s say I have this:

public class Whatever {
   private ArrayList<String> myList = null;

   public Whatever() {
     myList = new ArrayList<String>();
   }
}

What’s the difference between these two initialisations of myList? Would it be wrong to preffer the first variant?

Answer 1

The first variant will always instantiate the array list, the second one only when calling the default constructor. Meaning for the second solution you will have to call the default constructor for any additional constructor you add e.g.

public class Whatever {
   private final List<String> myList;

   public Whatever() {
     myList = new ArrayList<String>();
   }

   public Whatever(String name) {
     this();
     // Other stuff done
   }

   public Whatever(List<String> myList) {
     this.myList = myList;
   }
}

The (second) “lazy” initialization method might be better if you don’t always use the list (e.g. if you set the list in another constructor directly like in my example) and want to avoid creating unnecessary objects. (EDIT: I changed the ArrayList to an interface and set it final. It wasn’t part of the question but it is – as mentioned in the comments – the best way to use List collections).