skyler has asked for the wisdom of the Perl Monks concerning the following question:

I'm trying to debug this code but I'm hitting a wall do you have any ideas. I keep getting an error message "Can't stat 1 : No such file or directory at line 43" 
#! perl -w

use strict;
use File::Find;
use File::Copy;

my $infile  = 'c:/test.txt';

   open IN, "<$infile"  or die "Couldn't open $infile, $!";
   
   while(<IN>) {
   chomp;
   
   my @fields = split /,/;
   my $path_str = $fields[6];

   do { warn "Empty field 7"; next } unless $path_str;

   my @path = split /\\/, $path_str;
   
   # assuming you want to remove a few directories
   my @directory = join "\\", @path[0,5,6];

   my $srcfile;
   my $dir = @directory;

    sub process_file
{
    if ($File::Find::dir ne $dir)
    {
        $File::Find::prune = 1;
        return 0;
    }

    return 0 if ($_ !~ /\.rtf$/);

    copy($File::Find::name, "C:\\testfiles\\$_") or die "Failed to cop
+y $_: $!\n";
    return 1;
}

find(\&process_file, $dir);
     
}
exit;

[download]

Replies are listed 'Best First'.
Re: copy files
by steves (Curate) on Feb 24, 2003 at 20:29 UTC

    These lines are the problem: 

    my @directory = join "\\", @path[0,5,6];

my $srcfile;
my $dir = @directory;

    [download]
    You're assigning a path to an array, which will put it in the first array element. You're then assigning that array to $dir in scalar context, which is assigning the size of the array (1) to $dir.

    The use of an array for @directory doesn't quite make sense to me here unless you later intend to process a list of directories, which I don't see code for.

[reply]
[d/l]
      Steves I hope you are there! I complety understand what are you saying? I have added shift to @directory but it only copies one file???? so you are right!! how could I recode this script so it looks at every directory contain within the array. Any suggestions??? I'll appreciate your help. 
      #! perl -w

use strict;
use File::Find;
use File::Copy;

my $infile  = 'c:/doclist1.chr';
# my $outfile = 'c:/doclist1.txt';

   open IN, "<$infile"  or die "Couldn't open $infile, $!";
#   open OUT, ">$outfile" or die "Couldn't open $outfile, $!"; 
   
   while(<IN>) {
   chomp;
   
   my @fields = split /,/;
   my $path_str = $fields[6];

   do { warn "Empty field 7"; next } unless $path_str;

   my @path = split /\\/, $path_str;
   
   # assuming you want to remove a few directories
   my @directory = join "\\", @path[0,5,6];

   my $srcfile;
   my $dir = shift @directory;

    sub process_file
{
    if ($File::Find::dir ne $dir)
    {
        $File::Find::prune = 1;
        return 0;
    }

    return 0 if ($_ !~ /\.rtf$/);

    copy($File::Find::name, "C:\\testfiles\\$_") or die "Failed to cop
+y $_: $!\n";
    return 1;
}

find(\&process_file, $dir);

    
     # print OUT "\n@directory";
     
}
exit;

      [download]
[reply]
[d/l]
Re: copy files
by jasonk (Parson) on Feb 24, 2003 at 20:26 UTC 
    my $dir = @directory;

    [download]

    Evaluating an array as a scalar returns the number of elements in the array, so $dir will always get set to the number of elements in @directory, rather than the contents of @directory. You probably want something like my $dir = shift @directory; instead.

[reply]
[d/l]
[select]

Back to Seekers of Perl Wisdom