Re: redrafted code

[lyklev]

Move the subroutine definition to somewhere else, for example to the top of your code, before the main part of the code starts. Then call the subroutine at the point in the code where you have now defined it.

Also note that when you split a line:

• the first token starts at index 0, so your latitude will be $tokens[4] and your longitute will be $tokens[5];
• the last token will contain the newline at the end of the line, so before splitting, you might want to chomp the line.

[chromatic]

Move the subroutine definition to somewhere else, for example to the top of your code, before the main part of the code starts.

Perl 5 parses the entire code before running it, so this is only relevant if your sub declaration relies on runtime effects.