Use of uninitialized value $file in concatenation

[Anonymous Monk]

use strict; 

opendir(DIR, "./") or die $!;  # open current directory
my @files;
while (my $file = readdir(DIR)) {
    # Use a regular expression to ignore files beginning with a period
    push(@files, lc($file));
}
closedir(DIR);

# read the prompt.
my $promptfile = shift;
print STDERR "reading $promptfile\n";

open(PROMPT, $promptfile);
my $i = 0;
while (my $prompt = <PROMPT>) { 
    $i++;
    print STDERR $prompt . "\n";
    $prompt =~ s/^\d+\.\s//; # remove "93. ";
    $prompt = lc($prompt);
    
    # concat a list of .wav files corresponding to the prompt
    my $files = "";
    foreach my $word (split(/\s+/, $prompt)) {
        # choose a random file from the files, that beings with the gi
+ven $word.
        my @grepped = grep(/^$word/, @files);
        my $file = $grepped[ int(rand(scalar @grepped)) ]; # choose a 
+random file
        $files .= $file . " ";
    }
    my $output = sprintf("S%05d",$i);
    my $command = "sox $files $output.wav";
    print STDERR $command . "\n"; # TODO: add system call.
    system($command);
    print STDOUT "$output.wav $output.mfc\n";
}
close(PROMPT);

AFTER RUN THIS I GOT

Use of uninitialized value $file in concatenation (.) or string at scr
+ipts/conca
tprompt.pl line 29, <PROMPT> line 100.
sox           S00100.wav
sox FAIL sox: Not enough input filenames specified

AND COMMAND IS LIKE:

perl -w scripts/concatprompt.pl trainprompts > codetrain.scp

PLEASE TELL ME HOW TO REMOVE THIS ERROR...

[toolic]

Do you expect $file to be undefined? If so, start printing variable values (Tip #2 from the Basic debugging checklist).

Also, read Writeup Formatting Tips again, and post readable code in "code" tags.

[chromatic]

What happens when @grepped is empty?

[aaron_baugher]

Are you seeing what you expect when you print $prompt and $promptfile to STDERR? If not, figure out why. (One hint: you probably want to chomp your lines from your input file. A trailing newline will mess up the last filename on the line, at least.) If those are okay, put similar print statements inside your foreach loop to print variables like $word and $file, to see when you stop getting the results you expect.