Beefy Boxes and Bandwidth Generously Provided by pair Networks
No such thing as a small change
 
PerlMonks  

Re: Useless use of private variable

by Sandy (Deacon)
on Jul 15, 2009 at 02:49 UTC ( #780138=note: print w/ replies, xml ) Need Help??


in reply to Useless use of private variable

The "useless use of private variable" refers to the $s in your for loop because the variable is never being used.

A more perlish way of looping over a bit of code a fixed number of times (one of many ways actually) is

foreach (1 .. $s_no)

Sandy


Comment on Re: Useless use of private variable
Select or Download Code
Re^2: Useless use of private variable
by Sun751 (Beadle) on Jul 15, 2009 at 03:10 UTC
    as I am using the value of $s as index number for array,
    my $command = $CFG::CFG[$s]->{'CMD'}
    using foreach I don't see any possibility if I want to iterate over my array? Any suggestion?? Cheers.
      $s should be ok here. I suspect that $$p ( $$p=$$p-1;)could be the cause here? I'm not sure as I didn't get that error in my simple testing. I don't see much point in using a ref to $p. I would pass p itself, change this $$p line to $p--; and then return($p); at the end of the sub. I suppose its also possible that you did pass $p instead of \$p and this error message is a referencing error.

      Update: Looked again. $CFG::CFG[$s]->{'CMD'};
      also looks a bit odd. I'd like to see some more code and explanation of what you think that this CFG data structure looks like. Maybe you intend an array of Hash? I'm not sure from this code. $CFG isn't defined anywhere that I can see. What I said above shouldn't actually matter from functionality or compile errors. If you are going to use a ref, passing a ref to this $CFG thing is the place. Normally passing a ref to a scalar is not necessary or desirable as Perl can return multiple lvalues.

      Update Again: I didn't see the missing $s=0 in the for loop. Even so this code sans $CFG::CFG[$s]->{'CMD'}; compiled and ran on my Perl 5.10 machine.

Log In?
Username:
Password:

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

How do I use this? | Other CB clients
Other Users?
Others exploiting the Monastery: (17)
As of 2014-09-23 17:07 GMT
Sections?
Information?
Find Nodes?
Leftovers?
    Voting Booth?

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











    Results (232 votes), past polls