Re^2: Search array of file names in directory structure

If you are going for an autogenned regex off a fixed file list, why bother with the grouping? By the time you've gotten to the file extension, you've already passed the more rigorous constraint. Plus, yours can fail positive due to substring match and your filter doesn't allow for checking .htm files, as in the original regex. Simpler and more recyclable:

my $regex = do {
    my @escaped;
    push @escaped, quotemeta for @filelist;
    my $joined = join '|', @escaped;
    qr/^(?:$joined)$/;
};
[download]

or, if you like things done in 1 shot,

my $regex = '^(?:' . join('|', map quotemeta, @filelist) . ')$';
[download]

or, for the overly clever, my $regex = qr/^(?:@{[join '|', map quotemeta, @filelist]})$/;

#11929 First ask yourself `How would I do this without a computer?' Then have the computer do it the same way.

Comment on Re^2: Search array of file names in directory structure Select or Download Code

Replies are listed 'Best First'.
Re^3: Search array of file names in directory structure by Kenosis (Priest) on Oct 01, 2012 at 18:56 UTC
Great comments! Actually, I completely missed the `l?` in the original regex. Unless I'm mistaken, your regex filter also doesn't allow for checking `.htm` files. Have updated my posting. Appreciate you bringing these issues to my attention.	[reply] [d/l] [select]
Re^4: Search array of file names in directory structure by kennethk (Abbot) on Oct 01, 2012 at 20:09 UTC
It doesn't, but only insofar is it only checks the literals of the passed list. At some point, of course, all this becomes academic, since we don't actually know the entire use case; TIMTOWTDI. #11929 First ask yourself `How would I do this without a computer?' Then have the computer do it the same way.	[reply]
Re^5: Search array of file names in directory structure by Kenosis (Priest) on Oct 01, 2012 at 20:23 UTC
Thanks, kennethk. I appreciate your insights about this...	[reply]