Looks good, though you could probably just change the `if length > 0` to `if length >= 0` above