91

I want to replace the word "blue" with "red" in all text files named as 1_classification.dat, 2_classification.dat and so on. I want to edit the same file so I tried the following code, but it does not work. Where am I going wrong?

@files = glob("*_classification.dat");
foreach my $file (@files)
{
    open(IN,$file) or die $!;
    <IN>;
    while(<IN>)
    {
        $_ = '~s/blue/red/g';
        print IN $file;
    }
    close(IN)
}

4 Answers 4

187

Use a one-liner:

$ perl -pi.bak -e 's/blue/red/g' *_classification.dat

Explanation

  • -p processes, then prints <> line by line
  • -i activates in-place editing. Files are backed up using the .bak extension
  • The regex substitution acts on the implicit variable, which are the contents of the file, line-by-line
8
  • Yeah, or no quotes at all, if the code doesn't contain spaces.
    – bart
    Commented Aug 9, 2011 at 10:55
  • 6
    Using the * globbing in arguments does not seem to work in windows.
    – TLP
    Commented Aug 9, 2011 at 12:01
  • 3
    I notied that on windows I had to used double quotes around the regex
    – Tan Rezaei
    Commented Apr 15, 2016 at 0:31
  • glob hack to support wildcard command line arguments (*.dat) for Windows users: BEGIN { @ARGV = map +glob, @ARGV }
    – Zaid
    Commented Nov 20, 2019 at 16:27
  • 1
    @nr5 The problem is that bash variables don't expand within single quoted strings. Use double quotes.
    – drevicko
    Commented Jun 3, 2022 at 17:26
22

None of the existing answers here have provided a complete example of how to do this from within a script (not a one-liner). Here is what I did:

rename($file, $file . '.bak');
open(IN, '<' . $file . '.bak') or die $!;
open(OUT, '>' . $file) or die $!;
while(<IN>)
{
    $_ =~ s/blue/red/g;
    print OUT $_;
}
close(IN);
close(OUT);
1
  • 4
    ugly misuse of $_
    – iDoc
    Commented May 19, 2019 at 7:50
14

$_='~s/blue/red/g';

Uh, what??

Just

s/blue/red/g;

or, if you insist on using a variable (which is not necessary when using $_, but I just want to show the right syntax):

$_ =~ s/blue/red/g;
4

It can be done using a single line:

perl -pi.back -e 's/oldString/newString/g;' inputFileName

Pay attention that oldString is processed as a Regular Expression.
In case the string contains any of {}[]()^$.|*+? (The special characters for Regular Expression syntax) make sure to escape them unless you want it to be processed as a regular expression.
Escaping it is done by \, so \[.

Not the answer you're looking for? Browse other questions tagged or ask your own question.