unless (/^[.]{1,2}$/){
open IN,'<','../input/'.$_
or die "Could not open $_ for input : $!";
You skip `.' and `..' with the unless check, but don't do any checks after that, so if DIR contains any other directories, the script will die prematurely. That might not be an issue, depending on the OP's situation, but it's also easily avoided by replacing the unless with:
next if -d; # Skip directories.. OR:
# next unless -f; # Skip anything that isn't a file
(You can then outdent what is currently in the unless { } block.)
Second, there is pretty much never a good reason not to use indirect filehandles to avoid potential namespace clashes with handles like DIR, IN, and OUT:
open my $in, '<', '../input/' . $_;
Hope this helps. |