Though my version was not incorrect, I agree this is probably a better alternative here.

Generally, I dislike the ungainly $#some_array syntax and it's rarely needed. For example:

$some_array[$#some_array]
[download]

is better written as:

$some_array[-1]
[download]

because, apart from the unpleasant-to-read $#some_array, the second form avoids the duplication of some_array.