[KotoFan]

Dzień dobry. Mam pytanie o wydajność zastosowanego rozwiązania.

Problem: string w postaci liczba spacja liczba spacja(...) spacja spacja #num: liczba, z którego należy wydobyć ostatnią liczbę(ilość liczbo spacji na początku dowolna, liczby dowolnej długości). Przykład: "12 18 #num: 338921"
Rozwiązanie:

 String input = "12 18 #num: 338921";

 Pattern p = Pattern.compile("(\d+[ ])+[ ]#Num: ");
 Matcher m = p.matcher(input);

 if(m.find()){
 result = input.substring(m.end()).trim();
 }

Pytanie: czy to jest wydajne rozwiązanie, czy można to zrobić mądrzej/lepiej/szybciej?

#programowanie #java #regexp

[f.....d]

@KotoFan: num jest zawsze na końcu?

[KotoFan]

@fmfd: tak

[Rincewind]

@KotoFan: Jeśli rzeczywiście obchodzi cię tylko ta liczba po #num to można o wiele prościej:

Pattern p = Pattern.compile("#num: (\d+)");
Matcher m = p.matcher(input);
if (m.find()) {
  result = m.group(1);
}

[f.....d]

@KotoFan: to dlaczego tego faktu nie wykorzystałeś?

[Rincewind]

@KotoFan: Aaa, to jest zawsze na końcu? To przecież lastIndexOf(' ') da ci pozycję tuż przed liczbą.

[f.....d]

@KotoFan: plus wiesz że na pewno w tym formacie będzie input? Jeżeli tak to po co Ci preg match w ogóle?

[Rincewind]

@KotoFan: return Integer.parse(input.substring(input.lastIndexOf(' ') + 1));

[KotoFan]

@fmfd: wiem że w tym formacie będzie wejście, zmienia się tylko ilość liczb przed #

@Rincewind: interesuje mnie tez ciąg tych liczn Przed #

[f.....d]

@KotoFan: pattern cachejuesz?

[KotoFan]

@fmfd: nie rozumiem

[Rincewind]

@KotoFan: I tak nie wiesz (tak zakładam), ile będzie liczb przed tym nieszczęsnym "#num:". Regexp niewiele tu pomoże, najprościej będzie zrobić split linii po spacji i pętelka. :)


List numbers = new ArrayList<>();
Integer specialNumber = null;
``````
List tokens = Arrays.asList(line.split("\s+"));
Iterator iter = tokens.iterator();
while (iter.hasNext()) {
  String token = iter.next();
  if (!"#num:".equals(token)) {
    numbers.add(Integer.parseInt(token));
  } else {
    specialNumber = Integer.parseInt(iter.next());
    break;
  }
}

Tu masz działający przykład: http://ideone.com/LZu6oJ

[KotoFan]

@Rincewind: no nie wiem ile będzie liczb przed num, ale to regexp mi tego nie załatwia? Tak jak rozumiem, to załatwia :P

[Rincewind]

@KotoFan: Jeżeli też chcesz je z tego stringa wyciągnąć, to regexpem tego optymalnie i prosto nie zrobisz.

[Rincewind]

@KotoFan: To rozwiązanie które podałem powyżej rozbija ci stringa na listę liczb przed "#num:" i liczbę po tym elemencie. Więc masz wszystko. I – gwarantuję – regexpem tego ani szybciej, ani wygodniej, ani bardziej optymalnie nie #!$%@?. ( ͡° ͜ʖ ͡°)

[KotoFan]

@Rincewind: hehe, tak to jest jak się nie ma specyfikacji przed oczami :P. te liczby przed #num to mnie interesują, ale wystarczy mi jak będę je miał jako stringa "1 123 423 123 " :P

[Rincewind]

@KotoFan: No to line.split("#num:"); :P

[KotoFan]

@Rincewind: dobra nie było sprawy :)