Beefy Boxes and Bandwidth Generously Provided by pair Networks
Problems? Is your data what you think it is?

Re: Ways of commenting subroutines

by Lexicon (Chaplain)
on Mar 29, 2001 at 05:50 UTC ( #68026=note: print w/replies, xml ) Need Help??

in reply to Ways of commenting subroutines

My current commenting style, although it needs work. You'll have to excuse it's JavaScript-iness, that's all I have handy.
// ======================================================== // COUNT OCCURRENCES ( // STR, // String to search through // SUB // String to search for // ) // -------------------------------------------------------- // Counts the instances of SUB in STR. Counts overlapping // occurrences. Count_Occurrences( 'rrrrr', 'rr') = 4. // -------------------------------------------------------- function Count_Occurrences ( STR, SUB ) { var last = 0; var count = 0; while ((last = STR.indexOf(SUB, last + 1)) > -1) { count++ } return count; }
Update: If you ever spend a year in a country where they don't speak your language, keep a spell checker handy. Alternatively, if my code is used it will only be by non-english speakers, who wouldn't know how to spell 'occurrence' anyway. ;)


Replies are listed 'Best First'.
watch you're speling
by grinder (Bishop) on Mar 30, 2001 at 15:56 UTC

    ouch! you just hit on one of my pet peeves... bad spelling.

    It's "occurrence", not "occurance". Your code may be the greatest thing in the world since sliced bread, but if routine names contain spelling mistakes people are going to believe that the code contains other shoddinesses, thus you wind up needlessly damaging your reputation.

    Worse, client programmers using your code are going to stumble and trip, because they're going to try and call count_occurrences and fail miserably.

    g r i n d e r

Log In?

What's my password?
Create A New User
Node Status?
node history
Node Type: note [id://68026]
and all is quiet...

How do I use this? | Other CB clients
Other Users?
Others about the Monastery: (4)
As of 2018-06-23 09:28 GMT
Find Nodes?
    Voting Booth?
    Should cpanminus be part of the standard Perl release?

    Results (125 votes). Check out past polls.