SPUG: Regexp::Assemble question

Mark Mertel mark.mertel at yahoo.com
Wed Feb 25 09:20:47 PST 2009


Mr. Sweger,

I'm no expert, but it seems to me that Mr. Krahn's comments are all perfectly valid, and I can detect no ill intent, or slander towards the original poster. His are the types of comments to be expected during a professional code review. Were these directed at code I had written, I would welcome them rather than take them personally.
 ---
Mark Mertel





________________________________
From: Andrew Sweger <andrew at sweger.net>
To: John W. Krahn <jwkrahn at shaw.ca>
Cc: SPUG <spug-list at mail.pm.org>
Sent: Wednesday, February 25, 2009 8:55:13 AM
Subject: Re: SPUG: Regexp::Assemble question

On Tue, 24 Feb 2009, John W. Krahn wrote:

> Emails do not have colours, they are just plain text.

Mine doesn't. But I know some do. Regardless, Amit's intent was still
understood. 

> Is this your actual code?  If so, you don't need the subroutines at all, 
> you just need the keys.  And why use a hash reference instead of a hash?

Your tone suggests that all Perl programmers should use it in the same way
(your's aparently). Where I come from, Perl is spoken by people of every
skill level; from baby-perl-talk to uber-l33t-golpher-perl. If the code
works, it works. Style and convention mostly become a matter of teamwork.
This list is about learning and sharing Perl. I suspect you might not have
realized this (considering your reply to Amit and your previous reply to
this list).

> Why all the duplicate code?
> 
>      my $mode = qx(file $filename) =~ /gzip/i ? '<:gzip' : '<';
>      open MYINPUTFILE, $mode, $filename or $Logger->logdie( "Error 
> opening file: $!" );

Again, TIMTOWTDI.

> No need for this variable as Perl provides the $. built-in variable that 
> keeps track of the current line number.

Maybe so, but $lines is a lot more obvious than $. when it comes time to
review or update the code. Perhaps $INPUT_LINE_NUMBER (via English.pm)
could be even more obvious (and mean *exactly* the same thing as $.,
whereas $lines could actually mean something slightly different!).

> Those people who think they know everything are a great
> annoyance to those of us who do.        -- Isaac Asimov

Wow. Again?

Let me put it simply: knock it off. If you want to bash people about how
they write Perl, go hang out on Usenet or some obscure IRC channel. It's
not welcome here.

-- 
Andrew B. Sweger -- The great thing about multitasking is that several
                                things can go wrong at once.





_____________________________________________________________
Seattle Perl Users Group Mailing List
    POST TO: spug-list at pm.org
SUBSCRIPTION: http://mail.pm.org/mailman/listinfo/spug-list
    MEETINGS: 3rd Tuesdays
    WEB PAGE: http://seattleperl.org/



      
-------------- next part --------------
An HTML attachment was scrubbed...
URL: <http://mail.pm.org/pipermail/spug-list/attachments/20090225/82f90994/attachment-0001.html>


More information about the spug-list mailing list