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

Re: Re: Another commenting question,

by merlyn (Sage)
on Mar 17, 2001 at 00:28 UTC ( #65018=note: print w/replies, xml ) Need Help??


in reply to Re: Another commenting question,
in thread Another commenting question,

$x++; # keep track of how many users have logged in
Actually, I think that's a poor comment, unless the text around it makes it make more sense. I'd prefer to see something like:
$x++; # $x is number of users currently logged in, and we have a new o +ne
Hmm. On re-reading your comment, I can't tell whether you meant total number of users that have ever logged in, or whether it's a current count. That's probably why I already flagged it as bad. {grin}

Even better would be to never use $x for something like that:

$current_users++; # we have a new one
That's a lot better.

-- Randal L. Schwartz, Perl hacker

Replies are listed 'Best First'.
Better variable name
by tenya (Beadle) on Mar 21, 2001 at 02:53 UTC
    $current_user_count

    Is perhaps better (to make it obvious that it is a counter when it is not associated with the ++ operator).

Log In?
Username:
Password:

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

How do I use this? | Other CB clients
Other Users?
Others contemplating the Monastery: (7)
As of 2019-10-24 04:57 GMT
Sections?
Information?
Find Nodes?
Leftovers?
    Voting Booth?
    Notices?