Matching words in a list against a stringarray in java

Question

I need to find and replace a list of words given by the user. My application reads line by line in an HTML file and I want to verify if there is a word from the list and replace it with a blank space. This is what I have until now but I think I will have to modify my hole code in order to get what I want.

    private static void PrintFile(File source) throws IOException {
    String s;
    FileReader fr = new FileReader(source);
    @SuppressWarnings("resource")
    BufferedReader br = new BufferedReader(fr);

    @SuppressWarnings("resource")
    PrintWriter pw = new PrintWriter("Results.txt");
    while ((s=br.readLine())!=null) {
        pw.println(s.replaceAll("&#160;", "") //Words to be replaced.
                .replaceAll("<br>", "")
                .replaceAll("&amp;", "")
                .replaceAll("</p>", "")
                .replaceAll("</body>","")
                .replaceAll("</html>", "")
                .replaceAll("<remote object=\"#DEFAULT\">&gt;", ""));
    }
    System.out.println("Done!");
}

I accept any suggestions, the list idea may not be the best option.

score 1 · Accepted Answer · answered Jul 23 '15 at 22:14

1

You can remove HTML markup as simple as this with Jsoup:

public static String html2text(String html) {
  return Jsoup.parse(html).text();
}

Also have a look at Cleaner and Whitelist to sanatize documents individually.

answered Jul 23 '15 at 22:14

user1438038

5,821
6
60
94

Thanks, Jsoup helped a lot, in fact I improved mi solution by filtering words I needed once my html file was parsed. – Josué Almonasi Aug 12 '15 at 13:19

score 0 · Answer 2 · answered Jul 23 '15 at 22:11

Because String.replaceAll(String regex, String replacement) takes a regexp as its first parameter I would suggest to use String.replace(CharSequence, CharSequence replacement) instead to avoid undesired behavior.

Other than that I can't see a big problem in your code.

score -1 · Answer 3 · answered Jul 23 '15 at 22:14

-1

If you don't mind including apache commons lang into your project, you can use StringUtils.replaceEach and be done with it.

answered Jul 23 '15 at 22:14

Buhb

7,088
3
24
38

Matching words in a list against a stringarray in java

3 Answers3