Re: STDIN help for a new Perl coder

[linuxer]

in reply to STDIN help for a new Perl coder

Welcome here and to Perl.

If you read User input like that, you must know, that the linebreak will also be stored in your variable. So you must remove it; see chomp for details.

chomp( my $dir = <STDIN> );

Each statement should be completed with a ';', your chdir misses one.

You should check if chdir was successful and do something if it wasn't ($! contains the system's error message).

chdir( $dir ) or die "$dir: $!\n";

If you already changed into $dir, you must not specify it again in the glob pattern.

As you are using strict, you need to declare each new variable. You did it right with $dir, but not with @files.

[perlJourney]

Thank you for the advice... I forgot about declaring the @files but will do so. I also was not aware of the linebreak being stored... I did not know Perl did this. I will look into the chomp command.

I will also add the system error message to the chdir line to give the message like you suggest.