Beefy Boxes and Bandwidth Generously Provided by pair Networks
Clear questions and runnable code
get the best and fastest answer
 
PerlMonks  

Re (tilly) 1: Improving the efficiency of this piece of code

by tilly (Archbishop)
on Feb 25, 2001 at 06:14 UTC ( #60712=note: print w/ replies, xml ) Need Help??


in reply to Improving the efficiency of this piece of code

First of all please don't play games with the font. The usual one is fine.

Secondly rather than calling functions using:   &index;
it is better to call it with    index();
so you don't accidentally pass in the wrong parameters. (The first form reuses your current parameters.)

Third I would suggest that you have a final case that reports what the parameter was and that it was not understood. This will help debugging when (in my experience not if) things go sour.

Fourth in chatter you got the suggestion to call param('page') once and put that in a variable. That idea is a good one.

Fifth, you are using strict, good, but I think you are using too much from CGI. Try :standard until that runs into problems. (Or using the OO form.)

I could add more, but this is probably more than enough to digest at one go...


Comment on Re (tilly) 1: Improving the efficiency of this piece of code
Select or Download Code
Re: 2: Improving the efficiency of this piece of code (&FUNC; vs. FUNC(); vs. &FUNC(); ? )
by ybiC (Prior) on Feb 25, 2001 at 08:44 UTC
    I've made a habit of calling functions as &FUNC(); to make them more readily visible.

    The following explanations would seem to approve my approach.   Are there other factors I should take into account?   Reason(s) why I should forego the preceeding "&"?   Particular situations, perhaps?

    perlman::perlsub tells me:
    "& is optional with parentheses" and that "& makes current @_ visible to called subroutine".

    perlman::perlvar says:
    "within a subroutine the array @_ contains the parameters passed to that subroutine. See the perlsub manpage".
        thanks,
        Don
        striving for Perl Adept
        (it's pronounced "why-bick")

    Update: After reading tye's post that Albannach linked below, and re-reading tilly's post above, I *think* I'm OK in continuing to &FUNC().

      Coincidentally I ran across this node today in which tye offers a nice summary of calling options.

      --
      I'd like to be able to assign to an luser

Log In?
Username:
Password:

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

How do I use this? | Other CB clients
Other Users?
Others drinking their drinks and smoking their pipes about the Monastery: (9)
As of 2014-09-17 10:30 GMT
Sections?
Information?
Find Nodes?
Leftovers?
    Voting Booth?

    How do you remember the number of days in each month?











    Results (72 votes), past polls