String index out of range with replace all

Question

How can I replace mapDir surrounded by <> to a certain string?

  String mapDir = "D:\\mapping\\specialists\\ts_gpc\\";
  String test = "foo: <mapDir> -bar";
  println(test.replaceAll("<mapDir>", mapDir));

The above gives me a StringIndexOutOfBoundsException.

This code below for me, but I think pure java has to work as well.

static String replaceWord(String original, String find, String replacement) {
    int i = original.indexOf(find);
    if (i < 0) {
        return original;  // return original if 'find' is not in it.
    }

    String partBefore = original.substring(0, i);
    String partAfter  = original.substring(i + find.length());

    return partBefore + replacement + partAfter;
}

You need to quote (Pattern.quote) the first argument of replaceAll, as it is a regex. — Gábor Bakos
– Gábor Bakos, Commented Nov 22, 2014 at 13:17
Check out this post stackoverflow.com/questions/18875852/… — gashu
– gashu, Commented Nov 22, 2014 at 13:38

SMA · Accepted Answer · 2014-11-22 13:20:34Z

6

You dont need replaceAll method as you are not using regex. Instead you could work with replace api like below:

String mapDir = "D:\\mapping\\specialists\\ts_gpc\\";
String test = "foo: <mapDir> -bar";
System.out.println(test.replace("<mapDir>", mapDir));

answered Nov 22, 2014 at 13:20

SMA

37.2k8 gold badges53 silver badges76 bronze badges

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

1 Comment

Tom Over a year ago

Exactly. That all in replaceAll does not mean, that it replaces all occurrences of the searched string, whereas replace replaces only the first one. They both replace all occurrences.

ebo · Accepted Answer · 2014-11-22 13:25:52Z

3

replaceAll in String uses a regex, as specified in the documentation:

Note that backslashes () and dollar signs ($) in the replacement string may cause the results to be different than if it were being treated as a literal replacement string; see Matcher.replaceAll. Use Matcher.quoteReplacement(java.lang.String) to suppress the special meaning of these characters, if desired.

Thus, you should escape your replacement string like this:

String mapDir = "D:\\mapping\\specialists\\ts_gpc\\";
String test = "foo: <mapDir> -bar";
System.out.println(test.replaceAll("<mapDir>", Matcher.quoteReplacement(mapDir)));

which gives the output:

foo: D:\mapping\specialists\ts_gpc\ -bar

answered Nov 22, 2014 at 13:25

ebo

2,7471 gold badge27 silver badges22 bronze badges

Comments

Community · Accepted Answer · 2017-05-23 11:59:20Z

0

Since replaceAll works with regex, you need to re-escape the backslashes:

String mapDir = "D:\\\\mapping\\\\specialists\\\\ts_gpc\\\\";
String test = "foo: <mapDir> -bar";
System.out.println(test.replaceAll("<mapDir>", mapDir));

Quoting this answer:

by the time the regex compiler sees the pattern you've given it, it sees only a single backslash (since Java's lexer has turned the double backwhack into a single one)

edited May 23, 2017 at 11:59

CommunityBot

11 silver badge

answered Nov 22, 2014 at 13:25

alterfox

1,7053 gold badges22 silver badges37 bronze badges

Collectives™ on Stack Overflow

String index out of range with replace all

3 Answers 3

1 Comment

Comments

Comments

Your Answer

Linked

Hot Network Questions

Collectives™ on Stack Overflow

3 Answers 3

1 Comment

Comments

Comments

Your Answer

Sign up or log in

Post as a guest

Linked

Related