moved initalizing arraylist from constructor to field

The initalizing arraylist in the constructor caused the arraylist to lose the objects it gained when units after Unit2 were created.

Moving the initalizing to the field where it is declared makes it initalize once and only once (since I added final), which prevents any issues with reinitalizing