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
gav (2710)
gav
(email not shown publicly)
http://www.estey.com/
AOL IM: flufflegavin (Add Buddy, Send Message)
Hacker in NYC.
(email not shown publicly)
http://www.estey.com/
AOL IM: flufflegavin (Add Buddy, Send Message)
Hacker in NYC.
Friday February 28, 2003
02:25 PM
what were thinking?
Had to do some updates to a small cgi-script written by a co-worker and I found:
- Doesn't use CGI.pm
- Has the same if-elsif-else block copy and pasted four times but with slightly different numbers
- The very redundant $shipping += 0; statement
- Uses interesting quoting repeatedly - 'We\'re'
- Has code that doesn't match the comments:
##Perform Gift Cert Calc.##
##Free GiftCert Shipping with Stanard##
$shipping += GIFTCERTSHIP * $GCcount; - Gets the result wrong by forgetting that a line item can have a quantity
- Has a 96 line subroutine
I was going to excuse him if it was written a while back, but it's only 3 months old.
What do you do with a co-worker who seems to have no desire to improve his level of programming?
The Fine Print: The following comments are owned by whoever posted them. We are not responsible for them in any way.
--Larry Wall in stab.c from the perl source code
Stories, comments, journals, and other submissions on use Perl; are Copyright 1998-2006, their respective owners.
C4 (Score:1)
Seriously, talk to the manager. Tell them that the code is unmaintainable. Let them know.
-Dom