in reply to Need help deleting *.bak files

use strict; use warnings; use File::Find qw( find ); my $search_dir = '/emc/cccadm/scripts/perl/etc-test'; { print "Are you sure you would like to delete all *.bak ". "files that exist in: $search_dir [yes/no] "; chomp( my $answer = lc(<STDIN>) ); if ($answer ne 'y' && $answer ne 'yes') { die("You did not enter [yes]. Exiting program.\n"); } my @bak = find_bak_files($search_dir); if (!@bak) { # Why is this worthy of being said? warn("No files were found\n"); } for (@bak) { unlink($_) or warn("Can't delete $_: $!\n"); } } sub find_bak_files { my ($dir) = @_; my @bak; find( sub { push @bak, $File::Find::name if /\.bak\z/ && -f; }, $dir); return @bak; }

If it wasn't for the useless warning, you could shorten the above to

use strict; use warnings; use File::Find qw( find ); my $search_dir = '/emc/cccadm/scripts/perl/etc-test'; { print "Are you sure you would like to delete all *.bak ". "files that exist in: $search_dir [yes/no] "; chomp( my $answer = lc(<STDIN>) ); if ($answer ne 'y' && $answer ne 'yes') { die("You did not enter [yes]. Exiting program.\n"); } exit(1) if !delete_bak_files($search_dir); } sub delete_bak_files { my ($dir) = @_; my $success = 1; find( sub { if (/\.bak\z/ && -f) { if (!unlink($_)) { warn("Can't delete $File::Find::name: $!\n"); $success = 0; } } }, $dir); return $success; }