Beefy Boxes and Bandwidth Generously Provided by pair Networks
Problems? Is your data what you think it is?
 
PerlMonks  

Re: RFC: beginner level script improvement

by eyepopslikeamosquito (Chancellor)
on Sep 20, 2013 at 09:13 UTC ( #1054982=note: print w/replies, xml ) Need Help??


in reply to RFC: beginner level script improvement

A few random suggestions after skimming your code:

  • Prefer lower case for variable names (your code looks like it's shouting).
  • Learn Perl here documents.
  • Spend more time choosing good names (descriptive, explanatory, consistent, regular, ... spelt-correctly ;-).
  • Your main program is way too long. Decompose it into modules. Write unit tests for each module. Avoid big-arse functions.
  • Run your script through Perl::Critic.
  • Prefer three-argument open.
  • Insert $! in your error strings when a Perl function fails.
  • Prefer low precedence or to ||; for example, prefer or die to || die.
  • Don't use a leading & when calling functions.
  • Avoid C-style for loops. For example, prefer for my $item (@arr) {...$item...} to for(my $I=0;$I<@ARR;$I++) {...$ARR[$I]...}.
  • Prefer my variables to our variables. Declare variables at point of first use (do not declare a huge bunch of variables at the top of a subroutine). Minimize variable scope. Avoid global variables.

Log In?
Username:
Password:

What's my password?
Create A New User
Node Status?
node history
Node Type: note [id://1054982]
help
Chatterbox?
[LanX]: see soundfile https://en. wiktionary.org/ wiki/human# Pronunciation
[Your Mother]: Just Another Perl 'Acker.
[LanX]: I pronounce it 'juːmən'
[Your Mother]: Hey, let's not bring antisemitism into the conversation.
[LanX]: for centuries posh English speakers tried to approach French, that's where all the trouble starts
[Eily]: Just another Perl coeurs
[Eily]: (perl with hearts, 'Acker with a French accent is pronunced like coeurs)
[LanX]: IMHO 'ju' for 'u' happens in English where they can't pronounce the French u (or German )
[LanX]: and I'm sure I read that *hotel* used to be pronounced 'otel by the English upperclass
[Your Mother]: American English vowels are pretty lazy/mumbly in general.

How do I use this? | Other CB clients
Other Users?
Others contemplating the Monastery: (8)
As of 2017-03-27 17:13 GMT
Sections?
Information?
Find Nodes?
Leftovers?
    Voting Booth?
    Should Pluto Get Its Planethood Back?



    Results (320 votes). Check out past polls.