How to replace several Strings in a file at once using HashMap in Java

Question

I'm trying to replace several strings in one method using a HashMap but currently I can just get it to work with the first string in Map.

Previously I was using a "replaceAll" method for each String, which is hard to maintain because I'm trying to replace a changing list of Strings.

Can you give me any ideas?

Here's my code:

private static void string_change(String args[])
 {
 try
 {
 File file = new File("input_file.txt");
 BufferedReader reader = new BufferedReader(new FileReader(file));
 String line = "", oldtext = "";
 while((line = reader.readLine()) != null)
     {
     oldtext += line + "\r\n";
 }
 reader.close();

 Map<String, String> ReplacementMap = new HashMap<String, String>();
 ReplacementMap.put("STRING1", "TEXT1");
 ReplacementMap.put("STRING2", "TEXT2");
 ReplacementMap.put("STRING3", "TEXT3");

 String originalString = oldtext;

 for (Map.Entry<String, String> entry : ReplacementMap.entrySet()) {
   StringBuilder builder =  new StringBuilder(originalString.replaceAll(entry.getKey(), entry.getValue()));

   String newtext = builder.toString();

 FileWriter writer = new FileWriter("output_file.txt");
 writer.write(newtext);
 writer.close();
 }

 }
 catch (IOException ioe)
 {
 ioe.printStackTrace();
 }
}

So it only replaces "STRING1" with "TEXT1", and not "STRING2" and "STRING3"? — Atif
– Atif, Commented Sep 24, 2013 at 22:01

foxylion · Accepted Answer · 2016-02-23 17:00:15Z

4

The problem is, that you try to write the file for every entry in the map. So only the last entry is beeing applied to the target file. The better solution would be to apply all replacements before you write the file.

File file = new File("input_file.txt");
BufferedReader reader = new BufferedReader(new FileReader(file));
StringBuffer buffer = new StringBuffer();
String line;
while((line = reader.readLine()) != null) {
    buffer.append(line);
    buffer.append("\r\n");
}
reader.close();

Map<String, String> replacementMap = new HashMap<String, String>();
replacementMap.put("STRING1", "TEXT1");
replacementMap.put("STRING2", "TEXT2");
replacementMap.put("STRING3", "TEXT3");

String toWrite = buffer.toString();
for (Map.Entry<String, String> entry : replacementMap.entrySet()) {
    toWrite = toWrite.replaceAll(entry.getKey(), entry.getValue()));
}

FileWriter writer = new FileWriter("output_file.txt");
writer.write(toWrite);
writer.close();

edited Feb 23, 2016 at 17:00

answered Sep 24, 2013 at 22:01

foxylion

3931 silver badge13 bronze badges

Sign up to request clarification or add additional context in comments.

2 Comments

Boolean_Integer Over a year ago

Hey Thanks! It works perfectly! Do you think this code could perform better on large files than the string by string approach I was using before?

foxylion Over a year ago

Yes, that increases performance. When using a loop the compiler can't automatically replace the string concatenation with the StringBuilder. StringBuilder performs better when appending large amounts of strings before you really need the value itself.

Mark Elliot · Accepted Answer · 2013-09-24 22:08:39Z

0

You're overwriting your output file each time through the loop. How about this?

String temp = originalString;
for (Map.Entry<String, String> entry : ReplacementMap.entrySet()) {
    temp = temp.replaceAll(entry.getKey(), entry.getValue()));
}
FileWriter writer = new FileWriter("output_file.txt");
writer.append(temp);
writer.close();

edited Sep 24, 2013 at 22:08

answered Sep 24, 2013 at 22:02

Mark Elliot

77.4k23 gold badges147 silver badges162 bronze badges

1 Comment

Boolean_Integer Over a year ago

Thanks for your help! I've tried with your code but for some reason the output_file is blank. I'll try to tweak it a little bit to see if I can get the desired output.

Collectives™ on Stack Overflow

How to replace several Strings in a file at once using HashMap in Java

2 Answers 2

2 Comments

1 Comment

Your Answer

Hot Network Questions

Collectives™ on Stack Overflow

2 Answers 2

2 Comments

1 Comment

Your Answer

Sign up or log in

Post as a guest

Related