Maximale Zahl in einem String

[novulu]

Hallo!

Ich will in einem String die maximale Zahl finden. Der String sieht z.B. so aus

"1, -4, 13, -15"

Ergebnis wäre dann 13.

Meine Idee war, den String erst in ein Array zu zerlegen und dann die einzelnen Einträge zu vergleichen. Allerdings scheint bei der Zerlegung was schiefzulaufen.

public int findMaxNumber(String str){

int result;

String[] stringarray = str.split(", ");          //string to stringarray
int[] intarray = new int[stringarray.length];

for(int i=0; i<stringarray.length; i++)       //stringarray to intarray
intarray[i] = Integer.parseInt(stringarray[i]);

if(intarray.length == 0) return Integer.MIN_VALUE;

else {
  result = intarray[0];
  for(int i=0; i<intarray.length; i++){
    if(intarray[i] > result) result = intarray[i];
  }
}
return result;
}

Gibt

java.lang.NumberFormatException: For input string: ""
	at java.lang.NumberFormatException.forInputString(NumberFormatException.java:48)
	at java.lang.Integer.parseInt(Integer.java:470)
	at java.lang.Integer.parseInt(Integer.java:499)
	at Wrapper.findMaxNumber(Wrapper.java:19)
	at Evaluator.main(Evaluator.java:50)
	at EvaluatorExecutor.main(EvaluatorExecutor.java:7)

aus. Allerdings weiß ich nicht, wo da ein leerer String vorkommt der ihn raushaut.
Dachte durch die Trennung an ", " und nicht nur an "," schneidet er die Leerzeichen mit raus.

Hoffe mir kann jemand helfen.

Liebe Grüße,
novulu

 

[nrg]

bei mir funktionierts. scheints wohl nicht den String zu übergeben, den du oben gepostet hast...

 

[Xeonkryptos]

Ich wüsste auch nicht, woher der Fehler kommen könnte, aber schau dir das mal an Anderer Beitrag

Hier wurde das Thema schon einmal angesprochen. SF forever!

Edit: Und noch ein Beitrag

 

[novulu]

bei mir funktionierts. scheints wohl nicht den String zu übergeben, den du oben gepostet hast...

Das oben war auch nur ein Beispiel, aber so ist er zumindest aufgebaut, egal mit welchen Zahlen. :x
Aber okay, wenn's bei dir so funktioniert werd ich nochmal rumprobieren!

Ich wüsste auch nicht, woher der Fehler kommen könnte, aber schau dir das mal an Anderer Beitrag

Hier wurde das Thema schon einmal angesprochen. SF forever!

Edit: Und noch ein Beitrag

Da geht es um Ziffern in einer Zahl, sowas in der Art hab ich schon gelöst.
Aber jetzt geht es halt um Strings mit verschiedenen Zahlen.

 

[Xeonkryptos]

dann splitte es mal so:

String[] stringarray = str.split(",").trim();

Dabei gehe ich davon aus, dass du keinen leeren String bekommst. Du splittest ausschließlich nach dem Komma und entfernst daraufhin gleich die Leerzeichen (trim()). Damit sollte das Prob gelöst sein.

 

[novulu]

int result = Integer.MIN_VALUE;
if(str.length() == 0) return result;

String[] stringarray = str.split(", ");
int[] intarray = new int[stringarray.length];
for(int i=0; i<stringarray.length; i++)
{
if(stringarray[i] != "")
intarray[i] = Integer.parseInt(stringarray[i]);
else;
}

if(stringarray.length == 0) return result;
else {
  result = intarray[0];
  for(int i=0; i<intarray.length; i++){
  if(intarray[i] > result) result = intarray[i];
  }
}
return result;

Habs jetzt auf dem Weg gelöst, trim() durften wir nicht verwenden.
Funktioniert, danke für die Hilfe!

 

[ARadauer]

Generell empfehl ich dir java Code sauber einzurücken

if(stringarray.length == 0) return result;
else {

Bei sowas muss sogar ein profi zweimal hinschaun, um zu verstehen was da passiert

 

[faetzminator]

Und vor allem ist es... unnötig. Genau so wie ich den leeren String nicht überprüfen würde. Warum sollte da ein leerer String drin sein?

 

[xehpuk]

Vermutlich kommt der String so an, weil er selbst über eine Schleife erstellt wird:

"1, -4, 13, -15, "

Probier es mal mit [c]str.split(", ", 0)[/c].