Simon Cozens writes "I've been threatening to do it for a while, but now I've caved in, and set up the code review ladder mailing list." This list is for reviewing Perl code, and posting Perl code to be reviewed. Read on for more details.
"Part of the reason why I think some CPAN uploads are rubbish -- including some of my own -- is that people develop in isolation, and no one gets a chance to look over code before release. Other people can spot obvious mistakes or better ways to do something if they're a little further away from the action.
The idea behind the mailing list is that you can post your code to be reviewed, and if you get good feedback, you ought to feel some kind of moral obligation to help the next guy along. I've taken the concept from the go teaching ladder, where it seems to work quite well.
This isn't just for CPAN code, although improving CPAN is an intended goal. Post any code you'd like reviewed, and if we reach critical mass, you'll hopefully get some decent improvements on it."
Now, if you were going to create something that cool for Perl code reviews I'd be there today. But another mailing-list where some code will get posted for comments by the crowd? Where egos will flair and I'll get drawn into all kinds of silly wars over the particularities of Perl syntax? Thanks, but no thanks! PerlMonks, p5p and the mod_perl list more than fill my quota of that.
Still, cool idea!
-sam
Premature evaluation
Simon on 2003-09-03T09:22:41
But another mailing-list wheresome code will get posted for comments by the crowd? Where egos will flair and I'll get drawn into all kinds of silly wars overthe particularities of Perl syntax? Thanks, but no thanks!A hundred subscribers, and not a flame war yet! Who'da thunk it?
Re:Perl Mentor Program
uri on 2003-09-03T04:11:27
i have been doing code review for my current contract which is partly on site and partly telecommute. i find it much better and more effective to do it in person. when i show a way to rewrite some code i get immediate feedback if the coder(s) understand why/what i did. i can then explain it in depth and cover related topics (like if i use map, i will go into a map sidebar) at their level. i work with perl coders of various skill levels and so i also have to tune my rewrites and explanations to their level.so i doubt a online code review will be as effective. it couldn't hurt to try but as i am doing this already i don't think i could spare any tuits for this. i do some review on c.l.p.m and perlguru.com but not in quantity and for very short code pieces.
just my $.02
uri
Re:Perl Mentor Program
KM on 2003-09-03T04:36:24
Not many people do have tuits:) The idea behind the PMP (pronounced like 'pimp') was for someone how have only a few (or one) people to concentrate on at any time. Not random posts on a list or newsgroup. Someone they can really work with and spend as much time (duration) as the person needs. Like a Big Brother/Big Sister thing. And, isn't specifically to just to do a code review, give your $.02 on the code, and move on. Rather, to really help someone evolve as a programmer.
It's nice to do things face to face, but unless everyone is being paid by a large company to be flown in to review code... that's simply unreasonable in most cases. So, online is what most people have (also phone). But, online does help. I was one Perl Monks early on, and have been list-dad of the beginners lists since day 1. I have seen many people go from only asking questions to answering them (correctly)... all online.
Re:Perl Mentor Program
ziggy on 2003-09-04T17:30:51
It's not a matter of as effective as much as useful.so i doubt a online code review will be as effective.The list has been up a few days, and there's already been one module reviewed that didn't need to be written -- there's a similar module on CPAN already. There have been some other pieces of code reviewed with questionable usage that have been discussed and remediated.
Sounds like a good job so far. But certainly not a replacement for someone focused on doing code review on a large body of code.
This looks like an interesting idea. There are many areas which could benefit from this approach, such as:
I think this definitely a good idea and a good investment if you plan to write code that is understandable and hackable by others. Count me in
