Beefy Boxes and Bandwidth Generously Provided by pair Networks
go ahead... be a heretic
 
PerlMonks  

Re: To script or not to script, that is the question...

by tilly (Archbishop)
on May 31, 2005 at 17:30 UTC ( #462135=note: print w/replies, xml ) Need Help??


in reply to To script or not to script, that is the question...

The Perl Dr saw a function named Update::update and thought this was fine??

You need that second opinion!

There are a million kinds of updates out there. If I run across a function named update, I have no idea what it does, and I'll curse the person who wrote it with no mercy for the maintainance programmer looking for context.

Please name it something that gives a hint what it does. Something like "update_download" or "make_download_current".

While you're at it, I would leave out the superfluous "return 1". (Are you using that return anywhere?)

  • Comment on Re: To script or not to script, that is the question...

Replies are listed 'Best First'.
Re^2: To script or not to script, that is the question...
by biosysadmin (Deacon) on May 31, 2005 at 22:01 UTC

    I concur with tilly. My diagnosis is that your module needs a more specifically named namespace, and possibly a more illustrative name for your subroutine. Weirdly enough, coming up with a perfect name for a subroutine/package/module sometimes takes me longer than the design login and implementation.

    So, take two rename update's and call us in the morning. :)

Re^2: To script or not to script, that is the question...
by tlm (Prior) on May 31, 2005 at 17:41 UTC

    The return value of Update::update is used by the calling code to decide whether there are newly updated primary data files that need reprocessing. This doesn't seem unreasonable to me, though I'm open to second opinions on how best to do this, too.

    the lowliest monk

      That seems reasonable then.

Log In?
Username:
Password:

What's my password?
Create A New User
Node Status?
node history
Node Type: note [id://462135]
help
Chatterbox?
[stevieb]: I'm checking some other versions. This is a little frightening
[pryrt]: Sorry, my bad: it was't done downloading when I did the sha1sum... now I match your 309ad... value
[stevieb]: I'll see how to get in touch with someone and advise that there's an issue. Thanks for helping me confiirm pryrt!

How do I use this? | Other CB clients
Other Users?
Others making s'mores by the fire in the courtyard of the Monastery: (12)
As of 2017-03-29 21:02 GMT
Sections?
Information?
Find Nodes?
Leftovers?
    Voting Booth?
    Should Pluto Get Its Planethood Back?



    Results (353 votes). Check out past polls.