Skip to main content

Use empty for Collection#toArray([])


ImportanceReview GuidanceRequires Scanning Tool
LOWMerge Without ReviewNo

This change updates new array creation with Collection#toArray(T[]) to use an empty array argument, which is better for performance.

The point of the argument is provide an array to hold the objects and be returned, according to the documentation:

If the collection fits in the specified array, it is returned therein.

Although it's not intuitive, allocating a right-sized array ahead of time to pass to the API appears to be generally worse for performance according to benchmarking and JVM developers due to a number of implementation details in both Java and the virtual machine.

For a real world example, consider this issue in H2 where significant gains were achieved by switching to an empty array instead of a right-sized one.

Our changes look something like this:

- int[] tokenArray = tokens.toArray(new int[tokens.size()]);
+ int[] tokenArray = tokens.toArray(new int[0]);