лучший способ для создания строки в Java

У меня есть строка выглядит следующим образом:лучший способ для создания строки в Java

«Это @awesome @dude»

Из этой строки я хочу, чтобы извлечь удивительный и чувак и создать строку

output==> "awesome,dude"

Так что мой код, как следующее:

Matcher matcher = Pattern.compile("(?<[email protected])\\w+").matcher(textStr); 
while (matcher.find()){ 
    mergedStr += matcher.group() +","; 
}

Но это создает артефакт в ванной d

output==> "awesome,dude," //<-- egghh comma.. in the end

Что является лучшим способом решить эту проблему.

источник

2013-06-04 Fraz

Вы добавляете дополнительную запятую с объединеннымStr + = matcher.group() + ","; линия. – rajah9

Проблема не в регулярном выражении, а только на том, как вы строите результат. –

Ну, вы можете либо обрезать постороннюю запятую, либо никогда не добавлять ее в первую очередь. Один из способов сделать последнее - сделать первое слово вне цикла, а затем добавить запятую вверху цикла. –

Это один вариант:

Matcher matcher = Pattern.compile("(?<[email protected])\\w+").matcher(textStr); 
while (matcher.find()){ 
    if (!mergedStr.isEmpty()) 
     mergedStr += ","; 
    mergedStr += matcher.group(); 
}

источник

2013-06-04 21:04:55 Dukeling

+1 для этого простого решения, но, пожалуйста, избегайте операторов без фигурных скобок. Это довольно плохая практика. –

Это решает проблему, но я думаю, что это был бы лучший ответ, если бы вы упомянули, что он должен использовать 'StringBuilder'. :-) – ruakh

@adenoyelle Я думаю, что буду хранить свои 1-строчные if-утверждения без фигурных скобок, я вообще против фигурных скобок в любом месте. Но все это просто мнения. – Dukeling

Вот еще один общий подход:

Matcher matcher = Pattern.compile("(?<[email protected])\\w+").matcher(textStr); 
StringBuilder sb = new StringBuilder(); 
while (matcher.find()){ 
    sb.append(matcher.group()).append(","); 
} 
return sb.toString().replaceAll(",$", "");

Если вы не хотите использовать регулярное выражение, вы можете сделать это следующим образом:

Matcher matcher = Pattern.compile("(?<[email protected])\\w+").matcher(textStr); 
StringBuilder sb = new StringBuilder(); 
while (matcher.find()){ 
    sb.append(matcher.group()).append(","); 
} 
if (sb.length() == 0) { 
    return ""; 
} 
else { 
    return sb.toString().substring(0, sb.length() - 1); 
}

источник

2013-06-04 21:07:19 jahroy

Не работает, потому что String.replace делает не принимать регулярные выражения. Если вместо этого вы используете replaceAll, он работает. –

Хороший вопрос ... Не знаю, как я забыл (слишком много JavaScript в последнее время). Исправленный. – jahroy

'return mergedStr.substring (0, mergedStr.length() - 1);' может быть лучшим вариантом. – Dukeling

Другой подход:

boolean needComma = false; 
while (blah, blah, blah) { 
    if (needComma) { 
     string += ","; 
    } 
    string += word; 
    needComma = true; 
}

Но есть десяток разных подходов.

источник

2013-06-04 21:08:53

String input = "@[email protected]"; 
     List<String> strSplit = new ArrayList<String>(); 
     String result = ""; 

     Matcher matcher = Pattern.compile("(?<[email protected])\\w+").matcher(input); 
     while (matcher.find()){ 
      strSplit.add(matcher.group()); 

     } 
     for(int j = 0; j< strSplit.size(); j++){ 
      result = result + strSplit.get(j); 
      if(j < strSplit.size() -1){ 
       result = result+","; 
      } 
     } 
     System.out.println("Result : " + result);

источник

2013-06-04 21:17:59 FiendFyre

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

Я знаю, microoptimizations бла, бла, шестой круг ада, бла, бла, но только в том числе здесь для вашего развлечения:

Matcher matcher = Pattern.compile("(?<[email protected])\\w+").matcher(textStr); 
    StringBuilder mergedStr = new StringBuilder(); 
    if (matcher.find()) { 
     mergedStr.append(matcher.group()); 
     while (matcher.find()) { 
     mergedStr.append(',').append(matcher.group()); 
     } 
    } 
    return mergedStr.toString();

Кроме того, я не 100% уверен, что замена квадратичный алгоритм (конкатенация строк) с линейным алгоритмом (StringBuilder) квалифицируется как микрооптимизация в плохом смысле.

источник

2013-06-04 22:58:44 msandiford

лучший способ для создания строки в Java

ответ

Смежные вопросы