Элегантное решение для пользовательской замены java string

Question

Я решил проблему, указанную ниже, она отлично работает, но кажется громоздкой и не очень эффективной. Я ищу способы улучшить его и получить более элегантное решение, любые предложения, как я могу его улучшить? Любые советы оценили. Спасибо!

Задача: Приведенная строка возвращает строку, в которой каждый вид строчного слова «есть» заменен на «нет». Слово «есть» не должно предшествовать или сопровождаться буквой - так, например, «is» в «this» не учитывается.

Тесты:

notReplace("is test") → "is not test" 
    notReplace("is-is") → "is not-is not" 
    notReplace("This is right") → "This is not right" 
    notReplace("This is isabell") → "This is not isabell" 
    notReplace("")→ "" 
    notReplace("is") → "is not" 
    notReplace("isis") → "isis" 
    notReplace("Dis is bliss is") → "Dis is not bliss is not" 
    notReplace("is his") → "is not his"  
    notReplace("xis yis") → "xis yis" 
    notReplace("AAAis is") → "AAAis is not" 

Мое решение:

public static String notReplace(String str) { 
    String result=""; 
    int begin = 0; 
    if (str.equals("is")) 
     return "is not"; 
    int index = str.indexOf("is"); 
    if (index==-1) 
     return str; 
    while (index>-1){ 
      if (index+begin==0 && !Character.isLetter(str.charAt(index+2))){ 
       result += "is not"; 
      begin = index+2; 
      index = str.substring(begin).indexOf("is"); 
      } 
      else if (index+begin==0 && Character.isLetter(str.charAt(index+2))){ 
       result += str.substring(begin,begin+index)+"is"; 
       begin += index+2; 
       index = str.substring(begin).indexOf("is"); 
      } 
      else if (index+begin == str.length()-2 && !Character.isLetter(str.charAt(index+begin-1))){ 
       result += str.substring(begin, begin+index)+"is not"; 
       return result; 
      } 
      else if(!Character.isLetter(str.charAt(index+begin-1))&&!Character.isLetter(str.charAt(index+begin+2))){ 
       result += str.substring(begin,begin+index)+"is not"; 
       begin += index+2; 
       index = str.substring(begin).indexOf("is"); 
      } 
      else { 
       result += str.substring(begin,begin+index)+"is"; 
       begin += index+2; 
       index = str.substring(begin).indexOf("is"); 
      } 
     } 
     result += str.substring(begin); 
     return result; 
} 

Answer 1

Вы должны использовать Pattern.compile, а затем replaceAll. Я попытался написать regex, но я потерпел неудачу.

Так что вы должны сделать что-то вроде этого:

class Replacer { 
    static Pattern isPattern = Pattern.compile("...(is)..."); // here you have to figure out the right pattern 

    public static String notReplace(String input) { 
     return isPattern.matcher(input).replaceAll("is not"); 
    } 
} 

Я думаю, что это самое чистое решение, и гораздо быстрее, чем input.replaceAll, который компилирует Pattern каждый раз.

Как регулярное выражение, вероятно, вы должны использовать н как [^\\p{Alphanum}](is)[^\\p{Alphanum]

UPDATE

Вы должны использовать так называемые граница слова петь \b поэтому выражение должно выглядеть следующим образом: Pattern.compile("\\b(is)\\b") и проходит все ваших тестов :-)

Answer 2

Это решение работает для большинства ваших примеров:

public String notReplace(String str) { 
    // Add surrounding whitespace in case of an "is" at the beginning or end 
    str = " " + str + " "; 
    // Do replacement 
    String result = str.replaceAll(" is ", " is not "); 
    // Other replacements... 
    // result = result.replaceAll("", ""); 

    return result.trim(); // Remove added whitespaces again using trim() 
} 

Для примеров, которые не заменены с этим кодом вам нужно добавить дополнительные строки кода. Или загляните в регулярные выражения, - сказал он.

Надеюсь, это поможет.

Answer 3

Вы можете использовать регулярные выражения для этого. Ниже будет вернуть то, что вы указали в ваших примерах:

str.replaceAll("([^a-zA-Z]|^)is([^a-zA-Z]|$)", "$1is not$2"); 

Answer 4

С @ k0ner чьей помощи выяснили одно решение строки:

public String notReplace(String str) { 
    return str.replaceAll("\\bis\\b","is not"); 
} 

Answer 5

Все данные слова в вашем примере можно организовать с этим ответом и в случае вы получите что-то вроде «Это не Isabell является» то этим последним «является» будет заменен только вместо того, чтобы заменить «является» с «не».

String ss = word.replaceAll("\\bis\\b(?!=*.not\\b)", "is not"); 

Пример link