Integer.parseInt

[JavaMeetsBlueJ]

Hallo Forum,

bei nachfolgenden Code wird eine NumberFormatException geworfen, kann mir da jemand auf die Sprünge helfen wieso. Mir ist das ziemlich rätselhaft. Hier die beteiligten Klassen(ja der ist noch nicht sauber)

Klasse Gui:

import java.awt.BorderLayout;
import java.awt.GridLayout;
import java.awt.event.ActionListener;

import javax.swing.JButton;
import javax.swing.JFrame;
import javax.swing.JPanel;
import javax.swing.JTextField;


public class Gui extends JFrame{

	JPanel oberflaeche = new JPanel();                                   //Hauptpanel
	JPanel spielfeld = new JPanel();                                     //kommt auf das Hauptpanel
	JPanel bedienfeld = new JPanel();                                    //kommt auf das Hauptpanel unter das Spielfeld
	//-----------------------------------------------------------------------------------------------------
	
	//-----------------------------------------------------------------------------------------------------
	JTextField edtZahlen[][] = new JTextField[9][9];                     //jeweils ein Textfeld kommt auf jeweils ein Zahlenfeld
	//-----------------------------------------------------------------------------------------------------
	BorderLayout layout = new BorderLayout();
	GridLayout gamelayout = new GridLayout(9, 9);
	//-----------------------------------------------------------------------Layouts----------

	JButton kontrollieren = new JButton("kontrollieren");
	JButton schliessen = new JButton("Schließen");
//---------------------------------------------------------------------------Buttons----------------------
	public Gui()
	{
	  
	  ActionListener al = new ActionHandler(this);
	   
	   setTitle("SudokuReader");
		setSize(800, 800);
		setLocationRelativeTo(null);
		setDefaultCloseOperation(EXIT_ON_CLOSE);
		
		oberflaeche.setLayout(layout);
		bedienfeld.setLayout(new BorderLayout());
		add(oberflaeche);
		oberflaeche.add(bedienfeld,BorderLayout.SOUTH);
		bedienfeld.add(kontrollieren,BorderLayout.WEST);
		bedienfeld.add(schliessen,BorderLayout.EAST);
		spielfeld.setLayout(gamelayout);
		oberflaeche.add(spielfeld,BorderLayout.CENTER);
		
		kontrollieren.addActionListener(al);
		schliessen.addActionListener(al);
		
		setResizable(false);
		
	//-------------------------------------------------------------	
		
		//Erzeugung des kompletten Spielfeldes
		
		for(int zeile = 0; zeile<9;zeile++){
			for(int spalte = 0; spalte<9; spalte++){
				
				
				
				edtZahlen[zeile][spalte] = new JTextField(2);
				spielfeld.add(edtZahlen[zeile][spalte]);
				edtZahlen[zeile][spalte].setText( zeile +" ");
				

			}

		}
		//-----------------------------------------------------------
		
		add(oberflaeche);
		
	
		

		}
	
 
}

Klasse Berechnungen:

public class Berechnungen {
	
	static int tempZahl;
	static int tempElement;
	static boolean isZeile = true;
	static boolean isSpalte = true;
	
	public static boolean pruefeZeilen(Gui fenster)
	{
		for(int zeile = 0; zeile < 9; zeile++){
			for(int element = 0; element < 9; element++)
			{
				for(int spalte = 0; spalte < 9; spalte++)
				{
					try{tempZahl = Integer.parseInt(fenster.edtZahlen[zeile][spalte].getText());
					    tempElement = Integer.parseInt(fenster.edtZahlen[zeile][element].getText());
					    
					    
					    if(spalte != element && tempZahl==tempElement)
					    	{isZeile = false;
					    	
					    	System.out.println("Zeilen wahr: "+isZeile);
					    	break;
					    	}
					   
					    else{
					    	isZeile = true;
					    }
	
					    	
					}catch(Exception e)
					{
						System.out.println(e);     // Wieso wird hier was ausgegeben? Ich füll das Textfeld doch nur mit Zahlen :(
					}
				}
			}
		}
		return isZeile;
		
	}
	
	
	
	
}

Die Klasse Berechnungen soll (bisher) prüfen, ob eine Zahl in einer Reihe eines Sudokus doppelt vorkommt.

Meldung: NumberFormatException: For string input: "X ", wobei X hier eine Zahl ist.
MfG

 

[njans]

Du kannst doch mal mittels System.out.println() die Strings ausgeben lassen und dann siehst du ja, wie der String aussieht.

trim() den Text bevor du ihn parst.

edtZahlen[zeile][spalte].setText( zeile +" ");

oder mach es gleich schlauer:

edtZahlen[zeile][spalte].setText( String.valueOf(zeile));

 

[HimBromBeere]

Schonmal versucht, das folgende auszuführen?

System.out.println(Integer.parseInt("5 "));

Die Exception kommt, weil du eben nicht einfach "5 " in 5 überführen kannst, sondern erst, wie njans schon sagte, die Leerzeichen entfernen musst.

 

[JavaMeetsBlueJ]

Ok ich bau das mal um, ich dachte Java ist so "schlau" und kann das. Könnte ich nicht auch gleich die Zahl, die ich in das Textfeld schreibe in einen String umwandeln, sodass ich keine Leerzeichen hab. (Ist das vielleicht das was njans meint?)

MfG

 

[njans]

edtZahlen[zeile][spalte].setText( String.valueOf(zeile));

Ja das meine ich
Anstatt da über den Operator + im Kontext eines Strings " " etwas zu machen, nehme lieber die Methode, die die Klassse String anbietet.

 

[Kiri]

wenn du da

[JAVA=14]
try{tempZahl = Integer.parseInt(fenster.edtZahlen[zeile][spalte].getText());
tempElement = Integer.parseInt(fenster.edtZahlen[zeile][element].getText());
[/code]

jeweils ein trim() dran hängst, sollte es auch reichen!

also sowas

tempZahl = Integer.parseInt(fenster.edtZahlen[zeile][spalte].getText().trim());

 

[timbeau]

Java ist nicht "schlau" , Java macht genau das was in den APIs drinsteht. Und das ist auch gut so. Denn wen Computer so handeln, wie Menschen denken, dann wirds katastrophal.

Für dich ist .trim() die Lösung aber die Zuweisung nicht vergessen.

 

[JavaMeetsBlueJ]

Lernt man die ganzen die Methoden, die man auf den unzähligen Objekten in Java aufrufen kann, im Laufe sienes Lebens kennen, oder wie kommt man daran? In der Java Api suchen, nach etwas, dessen Existenz einem noch nichtmal bewusst ist, geht ja auch schwer.
MfG

 

[Kiri]

Lernt man die ganzen die Methoden, die man auf den unzähligen Objekten in Java aufrufen kann, im Laufe sienes Lebens kennen, oder wie kommt man daran? In der Java Api suchen, nach etwas, dessen Existenz einem noch nichtmal bewusst ist, geht ja auch schwer.
MfG

Naja, an gewissen Methoden kommt keiner vorbei und da gehört trim() dazu. Du musst einfach logisch an die Sache herangehen. Wo ist das Problem? In deinem Fall das dein String nicht nur Ziffern hatte sondern auch ein Whitespace am Ende. Damit wäre dein Problem lokalisiert - es ist der String. Jetzt schaust du in der API der Klasse String nach einer Methode, die die Whitespaces am Anfang und Ende entfernt. Mit etwas Glück gibt es die - wie in diesem Fall. Guckst du hier! Allgemein solltest du über den Tellerand schauen und dir eine Klasse die du so häufig benutzt auch genauer in der API anschauen, denn String hat viele nützliche Methoden!

 

[JavaMeetsBlueJ]

Ein weiteres Problem, was ich habe, ist, dass trotz des

break;

in der innersten for-Schleife die Schleife nicht abebrochen wird, wenn isZeile false ist. Woran leigt das? Die Methode liefert mit Standardbelegung false zurück, was ja auch richtig ist, aber Ressourcenmäßig ist das schon ziemlich schlecht. Eigentlich wollte ich mich mit sonem Anfängerkram nicht mehr abgeben... . Exception ist jetzt weg Danke

 

[njans]

Das lernt man durch benutzen und lesen kennen.
Nebenbei finde ich es schöner einen String aus einem int mittels String.valueOf() zu machen, als sinnlosen Ballast anzuhängen (value + " ") und dann diesen wieder wegzutrimmen. Das war auch die Aussage meines ersten Posts

 

[Kiri]

Das lernt man durch benutzen und lesen kennen.
Nebenbei finde ich es schöner einen String aus einem int mittels String.valueOf() zu machen, als sinnlosen Ballast anzuhängen (value + " ") und dann diesen wieder wegzutrimmen. Das war auch die Aussage meines ersten Posts

Dass er das macht habe ich glatt übersehen - da hast du natürlich recht! Allerdings finde ich ein trim() bei Benutzereingaben auch nicht verkehrt.

 

[gazzzzt]

In der Java Api suchen, nach etwas, dessen Existenz einem noch nichtmal bewusst ist, geht ja auch schwer.

Was soll daran schwer sein? Erstens bietet die API-Doc eine Methodenübersicht zu jeder Klasse und zweitens gibt in jeder vernünftigen IDE sowas wie Content Assist.

 

[StableElectron]

Ein weiteres Problem, was ich habe, ist, dass trotz des

break;

in der innersten for-Schleife die Schleife nicht abebrochen wird, wenn isZeile false ist. Woran leigt das? Die Methode liefert mit Standardbelegung false zurück, was ja auch richtig ist, aber Ressourcenmäßig ist das schon ziemlich schlecht. Eigentlich wollte ich mich mit sonem Anfängerkram nicht mehr abgeben... . Exception ist jetzt weg Danke

Falls das Problem noch besteht, das [c]break;[/c] in Zeile 34 setzen ?

Hier [c]static boolean isZeile = true;[/c] könnte man das [c]true[/c] weglassen, ist ja per default true ?

 

[Noctarius]

PS: Statt [c]zeile + " "[/c] würde auch noch [c]zeile + ""[/c] funktionieren, ist aber genauso unschön. Nimm die [c]String.valueOf(...)[/c] Methode

 

[JavaMeetsBlueJ]

Falls das Problem noch besteht, das [c]break;[/c] in Zeile 34 setzen ?

Hier [c]static boolean isZeile = true;[/c] könnte man das [c]true[/c] weglassen, ist ja per default true ?

Wird dann nicht, eagl welche Bedingung erfüllt ist , die Schleife abgebrochen, bzw. wenn else "erfüllt ist"?

MfG

 

[njans]

Man kann auch Schleifen labes zuteilen:

mylabel:
for (...)
{
  mylabel2:
  for(...)
  {
  break mylabel; // Springt aus den ganzen Schleifen
  }
}

 

[timbeau]

Ich würde das ganze Mal unterteilen. Erstelle dir eine Methode die eine Zeile liest und in einer geeigneten Datenstruktur speichert. Mit dieser Datenstruktur rufst du dann eine Methode auf, die prüft ob in einer Liste oder in einem Array doppelte Einträge vorkommen. Tipp Sets anwenden.

Dann hast du 2 Schritte, mehr Modularität, kannst prüfen wo es hakt und die Methode mehrfach verwenden. (Spalten, Blöcke etc)

 

[JavaMeetsBlueJ]

Danke für den Tip mit dem Label. Ebenso finde ich das Zwischenspeichern in Arrays auch nicht schlecht, aber das mach ich erst wenn alles läuft, man kann ja mehrere Versionen basteln Später bekommt das Textfeld dann auch noch einen Getter, um nicht gegen das Prinzip der Kapselung zu verstoßen(Textfeld wird dann private).

MfG

 

[timbeau]

Dieser Tip mit dem Label ist "ugly" hoch 10. Dafür lagert man aus und beendet die ausgelagerte Methode mit return und einem definierten Wert.

 

[njans]

Dieser Tip mit dem Label ist "ugly" hoch 10. Dafür lagert man aus und beendet die ausgelagerte Methode mit return und einem definierten Wert.

Joar, bestreitet auch keiner

 

[JavaMeetsBlueJ]

Kann mir vielleicht jemand meinen Denkfehler aufzeigen? Solange ich die Textfelder in Reihe mit zahlen von 0 bis 8 ausstatte, funktioniert alles korrekt, auch fehler werden korrekt ausgegeben. Bei Zahlen ab 9 ist das allerdings nichtmehr so. Woran liegt das? Etiche System.out.println() Ausgaben mit den Werten voon Zeile Spalte und Element, sowie den inhaltlichen Werten der Spalte in Zeile XY und inhaltlichem Wert des betreffenden Elements haben mir nicht weiter geholfen, sondern eher verwirrt. Wenn irgendeine Zahl in dem System größer ist als 8 kann der Rest sien wie er will und ich bekomme von der Methode true.


[EDIT]Man bin ich doof. Die Zahl kommt halt nicht doppelt vor und deshalb wird nichts gemeldet[/EDIT]
MfG

 

[timbeau]

"..ist nicht mehr so" ist keine Fehlermeldung!

BTW: Das mit dem "Zwischenspeichern" ist keine Frage von Lust & Laune sondern eine Frage der Modularität. Und verhindert Spaghetticode bei dem man das Kotzen kriegt wenn man Fehler sucht.

 

[JavaMeetsBlueJ]

Ja das stimmt, es sagt dem Programmierer, dass er etwas falsch gemacht hat

MfG