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

Re: RFC: Proofread POD for my main random module, please? (more)

by Marshall (Abbot)
on Jun 01, 2017 at 21:49 UTC ( #1191899=note: print w/replies, xml ) Need Help??


in reply to RFC: Proofread POD for my main random module, please? (more)

Hi Lady_Aleena,

I started by downloading your .pl code and then generating the HTML. I only looked at the HTML doc, not your code in this review. Looking at the code would perhaps solve lots of questions, but that is not the point of a doc review.

I don't understand the main purpose of this module other than it has something to do with randomizing things. Randomize what? And why? How does your module do much more than obvious uses of the standard Perl rand functions?

Why is there a separate tiny_rand() from instant_rand()? Why the need for 2 functions? As for naming why wouldn't a "tiny" one be "instant"?

As a naive person to your topic, I just do not understand what this thing is supposed to do.
I am not trying to "rain on your parade". But, I think the doc's could be improved. superfluous comment deleted.

update: Starting to look at the actual code and I agree with Re: RFC: Proofread POD for my main random module, please? (more) from pyrt in regards to "tiny_rand()" and "instant_rand()". I would think random_from_list() as pyrt suggests is more understandable. I don't see a reason to expose the 2 parameter version of "instant_rand()" as "tiny_rand()" unless there is some significant performance reason to do so.

  • Comment on Re: RFC: Proofread POD for my main random module, please? (more)

Log In?
Username:
Password:

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

How do I use this? | Other CB clients
Other Users?
Others taking refuge in the Monastery: (5)
As of 2019-10-18 04:05 GMT
Sections?
Information?
Find Nodes?
Leftovers?
    Voting Booth?
    Notices?