Beefy Boxes and Bandwidth Generously Provided by pair Networks
Syntactic Confectionery Delight
 
PerlMonks  

comment on

( [id://3333]=superdoc: print w/replies, xml ) Need Help??
Just a couple of comments on coding style
  • You don't really *need* to comment what the purpose of your variables are if they're clearly labeled (they were pretty clear to me at least), although this is more of a personal preference than anything.
  • It's good practice to keep your line-length under 80 columns for several reasons
    • it shows that your expression is too long and probably needs to be simplified (as always there are exceptions)
    • 80 cols is the standard width for terms which leads on to ...
    • code is a lot easier to read when the lines are nice'n'short (again this is personal opinion)
  • Here's a good rule of thumb for the amount arguments to pass to functions - if you've got 7+ arguments you probably don't have enough[1]. I think you could reduce the amount arguments you pass about by taking advantage of perl's aggregate data types.
  • Ack - no use strict in sight! This is pretty much a must for anything but the smallest of perl programs, and the warnings pragma is also muchly helpful.
  • Is there any reason why you're passing some of your scalar variables as references? While this is helpful and sometimes necessary when dealing with hashes and arrays, scalars are passed by value so don't needed to be passed by reference (unless you've got a good reason to of course).
And do make sure you get onto using CGI and watching out for potential security holes (see. opening a pipe to sendmail).
HTH

_________
broquaint

[1] this is indeed sarcasm but unfortunately I cannot take credit for this masterful piece of wit[2] which was introduced to me by a co-worker
[2] that, however, is mine ;-)


In reply to Re: Some suggestions on coding style - a chance to critique by broquaint
in thread Some suggestions on coding style - a chance to critique by emilford

Title:
Use:  <p> text here (a paragraph) </p>
and:  <code> code here </code>
to format your post; it's "PerlMonks-approved HTML":



  • Are you posting in the right place? Check out Where do I post X? to know for sure.
  • Posts may use any of the Perl Monks Approved HTML tags. Currently these include the following:
    <code> <a> <b> <big> <blockquote> <br /> <dd> <dl> <dt> <em> <font> <h1> <h2> <h3> <h4> <h5> <h6> <hr /> <i> <li> <nbsp> <ol> <p> <small> <strike> <strong> <sub> <sup> <table> <td> <th> <tr> <tt> <u> <ul>
  • Snippets of code should be wrapped in <code> tags not <pre> tags. In fact, <pre> tags should generally be avoided. If they must be used, extreme care should be taken to ensure that their contents do not have long lines (<70 chars), in order to prevent horizontal scrolling (and possible janitor intervention).
  • Want more info? How to link or How to display code and escape characters are good places to start.
Log In?
Username:
Password:

What's my password?
Create A New User
Domain Nodelet?
Chatterbox?
and the web crawler heard nothing...

How do I use this?Last hourOther CB clients
Other Users?
Others romping around the Monastery: (8)
As of 2024-04-18 16:06 GMT
Sections?
Information?
Find Nodes?
Leftovers?
    Voting Booth?

    No recent polls found