Требуется заявление Java IF?

Question

у меня есть:

String str = "Hello, how, are, you"; 

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

private static String removeComma(String str){ 
    if(str.contains(",")){ 
     str = str.replaceAll(",",""); 
    } 
    return str; 
} 

ИЛИ

private static String removeComma(String str){ 
    str = str.replaceAll(",",""); 
    return str;  
} 

Похоже, мне не нужен оператор IF но может быть случай, когда я делаю. Если есть лучший способ, дайте мне знать.

Answer 1

Оба функционально эквивалентны, но первый более подробный и, вероятно, будет медленнее, поскольку он выполняет дополнительную операцию.

Также обратите внимание, что вам не нужен replaceAll (который принимает регулярное выражение): replace будет делать.

Так что я хотел бы пойти на:

private static String removeComma(String str){ 
    return str.replace(",", ""); 
} 

Answer 2

Оператор IF не является необходимым, если вы не обработки «больших» строки (мы говорим мегабайта или больше).

Если вы используете оператор IF, ваш код сначала будет искать первое заполнение запятой, а затем выполнить замену. Это может быть дорогостоящим, если запятая находится ближе к концу строки, а ваша строка большая, так как ее нужно будет пройти дважды.

Без оператора IF запятые будут заменены, если они существуют. Если ответ отрицательный, ваша строка будет не затронута.

Нижнее правило: используйте версию без утверждения IF.

Answer 3

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

Answer 4

Это вопрос о том, какая вероятность иметь строки с запятой в вашей вселенной строк.

Если у вас есть высокая вероятность, вызовите метод replaceAll, не проверив сначала. НО Если вы не используете чрезвычайно огромные строки, я думаю, вы не увидите никакой разницы в производительности.

Answer 5

Просто еще одно решение с временной сложностью O (п), космической сложности O (N):

public static String removeComma(String str){ 
    int length = str.length(); 
    StringBuffer sb = new StringBuffer(); 

    for (int i = 0; i < length; i++) { 
     char c = str.charAt(i); 
     if (c != ',') { 
      sb.append(c); 
     } 
    } 
    return sb.toString(); 
}