go ahead... be a heretic | |
PerlMonks |
comment on |
( [id://3333]=superdoc: print w/replies, xml ) | Need Help?? |
Great idea. Couple comments though:
You mentioned +2 for excessive comments. I'd personally -- for excessive comments. Excessive means that it's more than is needed, which to me means something like: I think the loop is self commenting enough (not that you'd ever use that). It bothers me when people over comment. You spend more time reading about what each line does than you do reading the program. Maybe something subjective like "effective use of commenting"? Also, I think beyond the cut and dry "points", there should be a human critique on the style, flow, and as always, what could be better.
Just my nickle. In reply to Re: Code review on script site
by rchiav
|
|