Beefy Boxes and Bandwidth Generously Provided by pair Networks
laziness, impatience, and hubris
 
PerlMonks  

2Re: Reviewed code reviewed again

by jeffa (Bishop)
on Apr 16, 2004 at 04:55 UTC ( [id://345638]=note: print w/replies, xml ) Need Help??


in reply to Re: Reviewed code reviewed again
in thread Determine files added/removed (code for magazine article)

Call me crazy, but i would rather use a simple:

print OUT "For unique files in OldInventory.txt: IGNORING FILES WITH Tmp OR Temp IN PATHNAME. IGNORING FILES IN THE RECYCLE BIN. IGNORING ALL HISTORY FILES. LOWERCASING ALL FILE AND PATHNAMES BEFORE COMPARING. ";
for something as simple as that. But i would rather maintain your example than multiple print lines any day of the week. ;)

jeffa

L-LL-L--L-LL-L--L-LL-L--
-R--R-RR-R--R-RR-R--R-RR
B--B--B--B--B--B--B--B--
H---H---H---H---H---H---
(the triplet paradiddle with high-hat)

Replies are listed 'Best First'.
Re: 2Re: Reviewed code reviewed again
by McMahon (Chaplain) on Apr 16, 2004 at 22:15 UTC
    Thanks jeffa, and you're right.

    The production version(s) of this code have *many* more rules and are much more complicated. Also, I frequently get requests to modify the exclusion rules.

    I hope that my audience actually uses this code-- in which case they will want to experiment with their own regexes and I did want to make it easy to hack the output, too.

    Thanks for the comment!

Log In?
Username:
Password:

What's my password?
Create A New User
Domain Nodelet?
Node Status?
node history
Node Type: note [id://345638]
help
Chatterbox?
and the web crawler heard nothing...

How do I use this?Last hourOther CB clients
Other Users?
Others having an uproarious good time at the Monastery: (7)
As of 2024-04-19 10:52 GMT
Sections?
Information?
Find Nodes?
Leftovers?
    Voting Booth?

    No recent polls found