Question

I implemented the replaceAll() method with matcher, which replaces all punctuations with “”. But it always throws an exception: “java.lang.StringIndexOutOfBoundsException: String index out of range: 6“

private static StringBuilder filterPunctuation(StringBuilder sb){
    Pattern pattern =  Pattern.compile("(\\.)");
    Matcher matcher = pattern.matcher(sb);
    while(matcher.find()){
        sb.replace(matcher.start(), matcher.end(), "");  
// if sb.replace(matcher.start(),matcher.end()," "), it wil be right, but I want replace all punction with ""
    }
    return sb;
}

public static void main(String[] args){
    System.out.println(filterPunctuation(new StringBuilder("test.,.")));
}

Answer 1

If you are going to change the StringBuilder (especially its length by removing characters) inside of the loop, you are going to need to get a new Matcher (because the old one will continue to look at the original buffer or an inconsistent mix of both).

Take a look at how Jon Skeet would do it.