note
nysus
Well, when the module continutally tried to search http:/www.nytimes.com instead of http://www.nytimes.com, that was a pretty big tip off that the code wasn't up to snuff. The problem is that he rolled his own instead of using the URI module. Also, when I looked at the guts of the code, the 'new' constructor was 90% of the body of the code. That didn't smell quite right to me either. Then, to pass arugments, he would use code like <p>
<code>sub _save_img { my ($self,$url,$img) = (shift,shift,shift,shift);</code>
<p>
Finally the code was littered with numerous <code>#hack</code> comments.
<p>
I'm no expert mechanic true. But I do know that if the body is rusted and the interior is a filth pit (not that this code was THAT bad, mind you) there's a good chance the engine isn't worth repairing. I've seen enough professional code before and I know it when I see it. That doesn't mean I can write it.
<font size=2><p>
$PM = "Perl Monk's";<br>
$MCF = "Most Clueless <strike>Friar</strike> <strike>Abbot</strike> <strike>Bishop</strike> Pontiff";<br>
$nysus = $PM . $MCF;<br>
<a href="http://perlmonks.org/index.pl?node=Offering%20Plate">Click here if you love Perl Monks</a><br></font>
272046
272481