Use empty for Collection#toArray([])
pixee:java/use-empty-for-toarray​
| Importance | Review Guidance | Requires Scanning Tool |
|---|---|---|
| LOW | Merge Without Review | No |
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]);
processTokens(tokenArray);