That looks like a sensible suggestion apart from the following injunction from the "known issues" section of the FindBin documentation:

Which also means that you should avoid using FindBin in modules that you plan to put on CPAN.

However, given that the issue is with a script rather than a module it seems reasonable that that advice does not apply.