Re: Locator to Lat/Long code: works, but is inelegant

Looks adequately elegant to my eyes :) Maybe you wish to compare to these alternate implementations

Ham::Locator - Convert between Maidenhead locators and latitude/longitude.

Astro::Coord::ECI - Manipulate geocentric coordinates

Small style point, there is no need for

sub latlong2maidenhead;
sub maidenhead2latlong;
[download]

in your code. You might wish to compare your program to this template see (tye)Re: Stupid question (and see one discussion of that template Re^2: RFC: Creating unicursal stars)

Comment on Re: Locator to Lat/Long code: works, but is inelegant Download Code

Replies are listed 'Best First'.
Re^2: Locator to Lat/Long code: works, but is inelegant by Willard B. Trophy (Hermit) on Jul 03, 2011 at 14:26 UTC
Aargh! Forgot about the zero'th rule of Perl programming: check CPAN first! I just got into the habit of sub declaration, because I remember warnings being generated if you didn't. -- bowling trophy thieves, die!	[reply]
Re^3: Locator to Lat/Long code: works, but is inelegant by Anonymous Monk on Jul 03, 2011 at 17:36 UTC
I just got into the habit of sub declaration, because I remember warnings being generated if you didn't. Its all about parens. If you use parens, no warnings. You use parens :) `$ perl -le " f(1); sub f{warn@_} " 1 at -e line 1. $ perl -le " f 1 ; sub f{warn@_} " Number found where operator expected at -e line 1, near "f 1" (Do you need to predeclare f?) syntax error at -e line 1, near "f 1" Execution of -e aborted due to compilation errors. $ perl -le " sub f; f 1 ; sub f{warn@_} " 1 at -e line 1.` [download]	[reply] [d/l]
Re^4: Locator to Lat/Long code: works, but is inelegant by ForgotPasswordAgain (Vicar) on Jul 03, 2011 at 22:35 UTC
Defining a sub also "declares" it: `$ perl -le "sub f{warn@_} f 1" 1 at -e line 1.` [download]	[reply] [d/l]
Re^5: Locator to Lat/Long code: works, but is inelegant by Anonymous Monk on Jul 03, 2011 at 23:05 UTC