String.replaceAll() не работает

Question

Я редактирую письмо, полученное от tesseract ocr.

Вот мой код:

 if (email != null) {
        email = email.replaceAll(" ", "");
        email = email.replaceAll("caneer", "career");
        email = email.replaceAll("canaer", "career");
        email = email.replaceAll("canear", "career");
        email = email.replaceAll("caraer", "career");
        email = email.replaceAll("carear", "career");
        email = email.replace("|", "l");
        email = email.replaceAll("}", "j");
        email = email.replaceAll("j3b", "job");
        email = email.replaceAll("gmaii.com", "gmail.com");
        email = email.replaceAll("hotmaii.com", "hotmail.com");
        email = email.replaceAll(".c0m", ".com");
        email = email.replaceAll(".coin", ".com");
        email = email.replaceAll("consuit", "consult");
    }
    return email;

Но вывод не правильный.

Вход:

amrut=ac.hrworks@g mai|.com

Выход:

lalcl.lhlrlwlolrlklsl@lglmlalil|l.lclolml

Но когда я присваивал результат новой строке после каждой замены, он работал нормально. Почему непрерывное назначение в одной и той же строке не работает?

Answer 1

В Javadoc для String.replaceAll() вы заметите, что первый аргумент - это регулярное выражение.

Период (.) имеет особое значение там, как и труба (|) как и фигурная скобка (}). Вы должны избежать их всех, таких как:

email = email.replaceAll("gmaii\\.com", "gmail.com");

Answer 2

(Это Java?)

Обратите внимание, что в Java replaceAll принимает регулярное выражение и точка соответствует любому символу. Вам нужно уйти от точки или использовать

somestring.replaceAll(Pattern.quote("gmail.com"), "replacement");

Также обратите внимание на опечатку здесь:

email = emai.replaceAll("canear", "career");

должно быть

email = email.replaceAll("canear", "career");

Answer 3

Ты должен сбежать . от \\.как следующее:

if (email != null) {
    email = email.replaceAll(" ", "");
    email = email.replaceAll("caneer", "career");
    email = email.replaceAll("canaer", "career");
    email = email.replaceAll("canear", "career");
    email = email.replaceAll("caraer", "career");
    email = email.replaceAll("carear", "career");
    email = email.replace("|", "l");
    email = email.replaceAll("}", "j");
    email = email.replaceAll("j3b", "job");
    email = email.replaceAll("gmaii\\.com", "gmail.com");
    email = email.replaceAll("hotmaii\\.com", "hotmail.com");
    email = email.replaceAll("\\.c0m", "com");
    email = email.replaceAll("\\.coin", "com");
    email = email.replaceAll("consuit", "consult");
}
return email;

Answer 4

Понимая, что replaceAll() Первый аргумент regex вы можете сделать свои сравнения гораздо меньше

Например, вы можете проверить на возможные ошибки в написании слова career по следующим regex

email = email.replaceAll("ca[n|r][e|a][e|a]r", "career"));

Answer 5

Я думаю, что вы не знаете, что первый параметр replaceAll это регулярное выражение

., |, } может быть истолковано иначе, чем вы ожидаете.

.   Any character (may or may not match line terminators)

http://docs.oracle.com/javase/6/docs/api/java/util/regex/Pattern.html

Для космоса лучше использовать

\s  A whitespace character: [ \t\n\x0B\f\r]

и избежать других специальных символов с ведущим \\

Answer 6

Вы используете несколько символов регулярных выражений.

Пожалуйста, избегайте их, используя \ или с помощью Pattern.quote метод