Beefy Boxes and Bandwidth Generously Provided by pair Networks
We don't bite newbies here... much
 
PerlMonks  

Re: Perl Misconceptions

by Flame (Deacon)
on May 20, 2004 at 19:27 UTC ( [id://355052]=note: print w/replies, xml ) Need Help??


in reply to Perl Misconceptions
in thread Code Optimization v5.8.1

Ok, this is a completely unordered list, but I'll point out what I notice first pass:

  • You don't use strict, and you aren't localizing your variables with my or local
  • You initialize your scalar variables with empty sets. While this isn't really an error, it's not needed. When creating a variable, it is quite all right to simply declare its existance, perl initializes it to a default empty value.
  • You clear your variables the same way you're initializing them. If you truly wish to clear them, either set them to '', 0 or use undef. Not to say that your method is by definition wrong, just that it isn't commonly done.
  • In cases with nested loops and the loop control operators, it's usually best to name your loops just in case.
  • Around line 33 you use a chomp statement, but I suspect something got cut out, because you completely loose me there.


Nothing horrible, and easily fixed.
On a related note, I, personally, couldn't get your code to run, but I suspect that's just because of the way you entered it into the text box.


Edit: Minor correction, expanded on a statement.




My code doesn't have bugs, it just develops random features.

Flame

Log In?
Username:
Password:

What's my password?
Create A New User
Domain Nodelet?
Node Status?
node history
Node Type: note [id://355052]
help
Chatterbox?
and the web crawler heard nothing...

How do I use this?Last hourOther CB clients
Other Users?
Others goofing around in the Monastery: (6)
As of 2024-03-28 22:54 GMT
Sections?
Information?
Find Nodes?
Leftovers?
    Voting Booth?

    No recent polls found