[Kamishimi]

Cześć, mam String "C2 D83". Potrzebuję zwrócić tylko liczby na których będę później pracował. Napisałem coś takiego. Co o tym myślicie ?

String test = "C2 D83";
List<String> list = new ArrayList<String>(Arrays.asList(test.split(" ")));

StringBuilder sb = new StringBuilder();

for (String str : list) {
String number = str.substring(1, str.length());
String addNum = number + " ";
sb.append(addNum);
}

String numbers = sb.toString();
List<String> secondList = new ArrayList<String>(Arrays.asList(numbers.split(" ")));

int firstNumber = Integer.parseInt(secondList.get(0));
int secondNumber = Integer.parseInt(secondList.get(1));

#java #naukaprogramowania

[GeDox]

@Kamishimi: użyj regexa

[elvenpath]

@GeDox:

Some people, when confronted with a problem, think "I know, I'll use regular expressions." Now they have two problems.

[Kamishimi]

@GeDox: Jeszcze nie uczyłem się o tych regexach ale sprawdzę. Dzięki za wskazówkę.

[GeDox]

Now they have two problems

@elvenpath: jeśli nie umie się ich używać, to tak. Ja zawsze używam do walidacji i jakoś żaden hakier nie potrafi tego obejść :)

[elvenpath]

@GeDox: bardziej chodzi o utrzymanie tego i czytelność, nie mówię żeby całkiem nie używać, ale do tak banalnej rzeczy jak kolega wyżej to chyba bym sobie darował. Właśnie to po części jest problemem, że czasem nawet do prostych rzeczy są zaprzęgane regexy :)

[wpiot]

@Kamishimi: StringBuilder warto znać, ale jak masz krótkie ciągi znaków to nie warto używać.
Spróbuj zrobić to samo ze stream, można wtedy tą spację " " elegancko ogarnąć, bo teraz nawet po ostatnim elemencie dodajesz

[draxgar]

@Kamishimi:

List<Integer> result = Arrays.stream(testString.split(" ")).map(str -> str.replaceAll("\\D", "")).map(Integer::valueOf).collect(Collectors.toList());

[Kamishimi]

@draxgar: ciekawe rozwiązanie, Dzięki za pomoc