help using Java Matcher для изменения группы

Question

Я хочу совместить регулярное выражение и изменить соответствие. вот моя функция. прямо сейчас мой метод вообще не меняет ввод. что не так? Благодарю.

Matcher abbrev_matcher = abbrev_p.matcher(buffer); 
    StringBuffer result = new StringBuffer();//must use stringbuffer here! 
    while (abbrev_matcher.find()){ 
     //System.out.println("match found"); 
     abbrev_matcher.appendReplacement(result, getReplacement(abbrev_matcher)); 
    } 
    abbrev_matcher.appendTail(result); 

private static String getReplacement(Matcher aMatcher){ 
    StringBuilder temp = new StringBuilder(aMatcher.group(0)); 
    for (int i = 0; i < temp.length(); i++){ 
     if (temp.charAt(i) == '.'){ 
      temp.deleteCharAt(i); 
     } 

    } 
    return temp.toString(); 
} 

Answer 1

Вы просто хотите удалить все точки из согласованного текста? Здесь:

StringBuffer result = new StringBuffer(); 
while (abbrev_matcher.find()) { 
    abbrev_matcher.appendReplacement(result, ""); 
    result.append(abbrev_matcher.group().replaceAll("\\.", "")); 
} 
abbrev_matcher.appendTail(result); 

Причина в том, что appendReplacement(result, "")appendReplacement ищет $1, $2 и т.д., так что он может заменить их с группами захвата. Если вы не передаете строковые литералы или другие строковые константы для этого метода, лучше избегать этого этапа обработки и вместо этого использовать метод StringBuffer append. В противном случае он будет стремиться взорваться, если в заменяющей строке есть какие-либо знаки доллара или обратные слэши.

Что касается вашего метода getReplacement, в моих тестах он делает, чтобы изменить согласованную строку, но она не делает это правильно. Например, если строка равна ...blah..., она возвращает .blah.. Это потому, что каждый раз, когда вы вызываете deletecharAt(i) на StringBuilder, вы меняете индексы всех последующих символов. Вам нужно будет перебирать строку назад, чтобы этот подход работал, но это не стоит; просто начните с пустым StringBuilder и создайте строку append -ing вместо удаления. Это намного эффективнее, а также легче управлять.

Теперь, когда я думаю об этом еще, по этой причине вы не видите каких-либо изменений может быть, что ваш код бросает StringIndexOutOfBoundsException, что вы не видите, потому что код работает в try блоке и соответствующий catch блок пуст (классический Empty Catch Block анти-шаблон). N'est-ce pas?