Stories
Slash Boxes
Comments
NOTE: use Perl; is on undef hiatus. You can read content, but you can't post it. More info will be forthcoming forthcomingly.

All the Perl that's Practical to Extract and Report

use Perl Log In

Log In

[ Create a new account ]

Where can one get peer-review?

posted by pudge on 2000.12.10 16:32   Printer-friendly
sheriff_p writes "I've just finished writing a perl module, and knowing that my perl skills are far from perfect, I wanted to ask people what they thought the most effective way of getting peer-reviews on code was? For anyone who's interested, the module is at http://www.shoutingmyke.com/Sequin.pm"
The Fine Print: The following comments are owned by whoever posted them. We are not responsible for them in any way.
 Full
 Abbreviated
 Hidden
More | Login | Reply
Loading... please wait.
  • This is as good a place as any, I suspect, to ask for that sort of review. I, for one, would be interested in seeing some general writeup at the top of the module about its purpose, usage and suchlike.
  • Try posting on comp.lang.perl.modules. There are three things for which you could seek review
    1. The module name. Proposed module names should also be submitted to modules@perl.org for review.
    2. The API. Best way to do this is to post the POD.
    3. The code itself. If it is big, consider posting a URL to the sources, rather than the sources themselves. You might also give some indication of what aspects you want feedback on, e.g. general coding style, algorithms, integration with other facilities, specific techinica

  • I posted my last module on Perl Monks [perlmonks.org] with good response. A handful of other people have solicited feedback there. (The best response so far was to Filter::FileHandle, I believe. Good module, good discussion.)
  • I might as well toss a link here for folks to look at then. I've been working on a word game in perl (the guts are in a module so they can be used to produce multiple front ends). You can find the whole ball of wax at thecia.net/users/pate/ [thecia.net]. If you just want to see the module [thecia.net], you can do that to.