Interpreter-Fehler NumberFormatException Textfeld

[daZza]

Moinsen,

Ich muss für die Uni einen simplen BankAccount programmieren. Soweit funktioniert auch alles, nur das Problem ist, dass ich einen Interpreter Fehler kriege, wenn ich das Pin Feld leer lasse. Trägt man eine falsche, zu kurze, zu lange, ... PIN ein, so gibt er ganz normal die Meldung aus, dass die PIN falsch ist und korrigiert werden muss. Ich verstehe einfach nicht, warum er das nicht auch macht, wenn das PIN Feld leer ist, denn ist habe in der gleichen Bedingung stehen hilfe1.isEmpty(), wobei hilfe1 der Inhalt des PIN Feldes ist.

Ich hoffe ihr könnt mir helfen diesen kleinen Fehler zu beheben.

Fehlercode:

Exception in thread "AWT-EventQueue-0" java.lang.NumberFormatException: For input string: ""
	at java.lang.NumberFormatException.forInputString(Unknown Source)
	at java.lang.Integer.parseInt(Unknown Source)
	at java.lang.Integer.parseInt(Unknown Source)
	at EinAus.actionPerformed(EinAus.java:176)
	at javax.swing.AbstractButton.fireActionPerformed(Unknown Source)
	at javax.swing.AbstractButton$Handler.actionPerformed(Unknown Source)
	at javax.swing.DefaultButtonModel.fireActionPerformed(Unknown Source)
	at javax.swing.DefaultButtonModel.setPressed(Unknown Source)
	at javax.swing.plaf.basic.BasicButtonListener.mouseReleased(Unknown Source)
	at java.awt.Component.processMouseEvent(Unknown Source)
	at javax.swing.JComponent.processMouseEvent(Unknown Source)
	at java.awt.Component.processEvent(Unknown Source)
	at java.awt.Container.processEvent(Unknown Source)
	at java.awt.Component.dispatchEventImpl(Unknown Source)
	at java.awt.Container.dispatchEventImpl(Unknown Source)
	at java.awt.Component.dispatchEvent(Unknown Source)
	at java.awt.LightweightDispatcher.retargetMouseEvent(Unknown Source)
	at java.awt.LightweightDispatcher.processMouseEvent(Unknown Source)
	at java.awt.LightweightDispatcher.dispatchEvent(Unknown Source)
	at java.awt.Container.dispatchEventImpl(Unknown Source)
	at java.awt.Window.dispatchEventImpl(Unknown Source)
	at java.awt.Component.dispatchEvent(Unknown Source)
	at java.awt.EventQueue.dispatchEventImpl(Unknown Source)
	at java.awt.EventQueue.access$000(Unknown Source)
	at java.awt.EventQueue$3.run(Unknown Source)
	at java.awt.EventQueue$3.run(Unknown Source)
	at java.security.AccessController.doPrivileged(Native Method)
	at java.security.ProtectionDomain$1.doIntersectionPrivilege(Unknown Source)
	at java.security.ProtectionDomain$1.doIntersectionPrivilege(Unknown Source)
	at java.awt.EventQueue$4.run(Unknown Source)
	at java.awt.EventQueue$4.run(Unknown Source)
	at java.security.AccessController.doPrivileged(Native Method)
	at java.security.ProtectionDomain$1.doIntersectionPrivilege(Unknown Source)
	at java.awt.EventQueue.dispatchEvent(Unknown Source)
	at java.awt.EventDispatchThread.pumpOneEventForFilters(Unknown Source)
	at java.awt.EventDispatchThread.pumpEventsForFilter(Unknown Source)
	at java.awt.EventDispatchThread.pumpEventsForHierarchy(Unknown Source)
	at java.awt.EventDispatchThread.pumpEvents(Unknown Source)
	at java.awt.EventDispatchThread.pumpEvents(Unknown Source)
	at java.awt.EventDispatchThread.run(Unknown Source)

Java Code:

import java.awt.*;
import java.awt.event.*;
import javax.swing.*; 

public class EinAus extends JFrame implements ActionListener 
{  
     
     private JTextField tfBetrag, tfPIN, tfSystem, tfKontostand;
     private int Einzahlung, Auszahlung, Kontostand = 0, Limit = 3000, Geheimzahl = 1234, Dispo = -5000, Auszahlungen = 0;
     private JTextArea tfTest;
     private JButton btAbheben = new JButton();
     private JButton btBestätigen = new JButton();
     private JButton btEinzahlen = new JButton();
               
     public EinAus()   
     {    
          super("Bank Account");
          this.getContentPane().setBackground(Color.lightGray);
          this.getContentPane().setLayout(null);
          //Einzahlen-Button
          JButton btEinzahlen = new JButton("Einzahlen");
          btEinzahlen.setBounds (20, 120, 120, 30);
          btEinzahlen.setBackground(Color.green);
          btEinzahlen.addActionListener(this);
          this.getContentPane().add(btEinzahlen);
          //Auszahlen-Button
          JButton btAbheben = new JButton("Abheben");
          btAbheben.setBounds (20, 160, 120, 30);
          btAbheben.setBackground(Color.green);
          btAbheben.addActionListener(this);
          this.getContentPane().add(btAbheben);
          //Bestätigen-Button
          JButton btBestätigen = new JButton("Bestätigen");
          btBestätigen.setBounds (20, 200, 120, 30);
          btBestätigen.setBackground(Color.green);
          btBestätigen.addActionListener(this);
          this.getContentPane().add(btBestätigen);
          //Eingabe-Label
          JLabel lbBetrag = new JLabel ("Betrag:");
          lbBetrag.setBounds (110, 40, 90, 30);
          this.getContentPane().add (lbBetrag);
          //Kontostand-Label
          JLabel lbKontostand = new JLabel ("Kontostand");
          lbKontostand.setBounds (440, 35, 90, 30);
          this.getContentPane().add (lbKontostand);
          //Eingabe-Textfeld
          tfBetrag = new JTextField (40);
          tfBetrag.setText ("");
          tfBetrag.setBounds (160, 40, 200, 30);
          tfBetrag.setBackground(Color.white);
          this.getContentPane().add (tfBetrag);
          //Kontostand-Textfeld
          tfKontostand = new JTextField (40);
          tfKontostand.setText ("");
          tfKontostand.setBounds (390, 60, 160, 30);
          tfKontostand.setBackground(Color.white);
          this.getContentPane().add (tfKontostand);
          //Ausgabe-Label
          JLabel lbPIN = new JLabel ("PIN:");
          lbPIN.setBounds (130, 80, 90, 30);
          this.getContentPane().add (lbPIN);
          //Ausgabe-Textfeld
          tfPIN = new JTextField (40);
          tfPIN.setText ("");
          tfPIN.setBounds (160, 80, 200, 30);
          tfPIN.setBackground(Color.white);
          this.getContentPane().add (tfPIN);
          //System-Textfeld
          //tfSystem = new JTextField (40);
          //tfSystem.setText ("");
          //tfSystem.setBounds (375, 120, 200, 200);
          //tfSystem.setBackground(Color.white);
          //this.getContentPane().add (tfSystem);
          //Test-Textarea
          tfTest = new JTextArea(1,1);
          tfTest.setText ("");
          tfTest.setBounds (375, 120, 200, 200);
          tfTest.setBackground(Color.white);
          tfTest.setWrapStyleWord(true);
          tfTest.setLineWrap(true);
          this.getContentPane().add (tfTest);
          this.setVisible(true);
     }


     public void actionPerformed(ActionEvent event)   
     {    
          String cmd = event.getActionCommand();
          if (cmd.equals("Einzahlen"))
          {
               int hilfe = Integer.parseInt(tfPIN.getText());
               String hilfe1 = tfBetrag.getText();
               int hilfe3 = Integer.parseInt(tfBetrag.getText());

               if (!hilfe1.isEmpty() &&  hilfe == Geheimzahl && hilfe3 != 0)
               {
                Einzahlung = Integer.parseInt(tfBetrag.getText());
                tfBetrag.setText("");
                Kontostand = Kontostand + Einzahlung;
                tfTest.setText(tfTest.getText() + "\n" +  "Einzahlung erfolgreich. Kontostand aktualisiert.");
                String hilfe2 = String.valueOf(Kontostand);
                tfKontostand.setText(hilfe2);
               }

               else
               {
                tfTest.setText(tfTest.getText() + "\n" +  "Bitte überprüfen sie ihre Eingaben");
               }

          }
          else if (cmd.equals("Abheben"))
          {

               int hilfe = Integer.parseInt(tfPIN.getText());
               String hilfe1 = tfBetrag.getText();
               int hilfe3 = Integer.parseInt(tfBetrag.getText());
               
               
               if (!hilfe1.isEmpty() &&  hilfe == Geheimzahl)
               {
                
            	if (hilfe3 > Limit)
            	{
            	 tfTest.setText(tfTest.getText() + "\n" +  "Betrag überschreitet Limit. Bitte kleineren Betrag wählen");
            	}
            	else if (hilfe3 <= Limit && Auszahlungen < 10000 && Kontostand > -5000 && hilfe3 != 0)
            	{
            	Auszahlung = Integer.parseInt(tfBetrag.getText());
                tfBetrag.setText("");
                int hilfe4 = Kontostand - Auszahlung;
                if (hilfe4 < -5000)
                {
                	tfTest.setText(tfTest.getText() + "\n" +  "Dispo wird mit dieser Auszahlung überzogen. Bitte kleineren Betrag wählen.");
                }
                else
                {
                Kontostand = Kontostand - Auszahlung;
                tfTest.setText(tfTest.getText() + "\n" +  "Auszahlung erfolgreich. Kontostand aktualisiert.");
                String hilfe2 = String.valueOf(Kontostand);
                tfKontostand.setText(hilfe2);
                Auszahlungen = Auszahlungen + Auszahlung;
                }
               
            	}
            	
            	else if (hilfe3 == 0)
            	{
            	tfTest.setText(tfTest.getText() + "\n" +  "Bitte überprüfen sie ihre Eingaben");
            	}
            	
            	else if (Kontostand < -5000)
            	{
            		 tfTest.setText(tfTest.getText() + "\n" +  "Dispo überzogen. Bitte Konto ausgleichen.");	
            	}
            	
            	else if (Auszahlungen >= 10000)
            	{
            		tfTest.setText(tfTest.getText() + "\n" +  "Globales Limit für den heutigen Tag erreicht. Bitte kommen sie morgen wieder.");	
            	}
            	
               }
               
               else
               {
                tfTest.setText(tfTest.getText() + "\n" +  "Bitte überprüfen sie ihre Eingaben");
               }

          }
          else if (cmd.equals("Bestätigen"))
          {
               int hilfe = Integer.parseInt(tfPIN.getText());
               String hilfe1 = tfPIN.getText();
               String hilfe2 = String.valueOf(Kontostand);
               
               if (hilfe1.isEmpty() || hilfe != Geheimzahl)
               {
                 tfTest.setText(tfTest.getText() + "\n" + "PIN falsch. Bitte korrigieren.");
               }
               else
               {
                tfTest.setText(tfTest.getText() + "\n" + "PIN Authentifizierung erfolgreich");
                tfKontostand.setText(hilfe2);
               }
          }
     }
     
      public static void main(String[] args)
     {
          EinAus wnd = new EinAus();
          wnd.setSize(600,400);
          wnd.setVisible(true);
     }
}

 

[tfa]

Wenn du der parseInt-Methode einen Leerstring übergibst, kommt eben eine NumberFormatException. Ganz einfach.
Du musst eben diese Situation verhindern, indem du zuerst prüfst, ob der String "" ist und dann erst einen int daraus parst.

 

[SlaterB]

was soll an einer NumberFormatException für "" ein 'seltsamer Interpreretfehler' sein? Titel geändert

du solltest bei Fehlern besser deinen Code kürzen/ straffen, arbeite testweise mit nur EINEM Textfeld, schaue dir dessen Inhalt an und werte ihn strukturiert aus,
wenn du zig Textfelder und Variablen hilfe1 - hilfe4 hast, mal String, mal int, dann siehst du wahrscheinlich selber nicht mehr durch..

bisher schon wichtig: Strings nie mit == vergleichen, nur mit equals,
Variablen nicht groß schreiben, naja und hilfe1 - hilfe4 ist natürlich auch nicht hilfreich

edit:
deine ständigen Zeilen
[c]tfTest.setText(tfTest.getText() + "\n" + [Nachricht]);[/c]
ersetze lieber durch viel kürzeres
[c]log([Nachricht]);[/c]
mit entsprechender Hilfsmethode, die den Rest macht, und zudem noch später bei Bedarf komfortabel zu ändern ist statt zig Codezeilen im Programm

 

[stikio]

Du versuchst n String zu ner Number zu konvertieren, aber der String ist keine zahl

	at java.lang.Integer.parseInt(Unknown Source)
	at java.lang.Integer.parseInt(Unknown Source)

da liegt der fehler.

*edit: zu spät ^^

 

[Michael...]

Immer diese GUIs ohne LayoutManager
Du versuchst mehrfach den Inhalt des Textfeldes ungeprüft in einen int zu parsen. Daher fliegt wenn das Feld leer ist - und übrigens auch wenn man buchstaben oder sonstige Zeichen eingibt - diese Exception.
Man könnte grundsätzlich auch diese Exception abfangen und falls diese Auftritt ein "... ungültig" ausgeben.

 

[daZza]

Alles klar, Danke schonmal.
Habe halt erst angefangen zu coden und bin noch nicht so auf dem Optimierungstrip Hauptsache es funktioniert.
Kann mir denn jemand konkret sagen wie man das abfangen kann? Hab jetzt nochmal was versucht, allerdings ohne Erfolg

 

[Michael...]

Kann mir denn jemand konkret sagen wie man das abfangen kann? Hab jetzt nochmal was versucht, allerdings ohne Erfolg

Als aller erstes könnt man ja mal prüfen ob überhaupt etwas im Textfeld steht. Deine Prüfung setzt wohl ein bisschen zu spät an ;-)

 

[daZza]

Gut. Habe das jetzt mit if (tfPIN.getText().isEmpty()) am Anfang abgefangen. Jetzt bin ich schonmal einen Schritt weiter, da ein leeres Feld keinen Fehler mehr verursacht. Allerdings macht er das immer noch, wenn irgendwo im Pin Leerzeichen drin sind, oder Buchstaben benutzt werden. Kann man das Eingabealphabet irgendwie auf 0-9 begrenzen?

 

[VfL_Freak]

Moin,

wie wäre es mit einem JFormattedTextField" zur Eingabe ?
JFormattedTextField

Gruß
Klaus

 

[daZza]

Dann kriege ich folgenden Fehler

EinAus.java:200:67: error: cannot find symbol
JFormattedTextField dateField = new JFormattedTextField (new MaskFormatter("####"));
^
symbol: class MaskFormatter
location: class EinAus
1 error


edit:

Hat sich erledigt. Ich habe den Fehler jetzt einfach mit catch (java.lang.NumberFormatException e) abgefangen. Danke an alle

 

[VfL_Freak]

Moin,

Dann kriege ich folgenden Fehler
EinAus.java:200:67: error: cannot find symbol
JFormattedTextField dateField = new JFormattedTextField (new MaskFormatter("####"));
^
symbol: class MaskFormatter
location: class EinAus
1 error

ich hoffe, Du hast javax.swing.JJFormattedTextField importiert ???:L


EDIT: oh Mann - ich Looser, ich ...... ;(
of course nur ein 'J' ....

Hat sich erledigt. Ich habe den Fehler jetzt einfach mit catch (java.lang.NumberFormatException e) abgefangen

Wie jetzt? Ohne die Ursache behoben zu haben ???:L

Gruß
Klaus

 

[daZza]

Jop ohne die "Ursache" behoben zu haben Er gibt dann halt aus, dass nur Zahlen als PinCode in Frage kommen. Habe

import javax.swing.*;

importiert. Dazu sollte ja auch javax.swing.JJFormattedTextField gehören?! Ich habe auch versucht das direkt zu importieren, allerdings dann mit einem weiteren Fehler.

 

[VfL_Freak]

Moin,

Jop ohne die "Ursache" behoben zu haben Er gibt dann halt aus, dass nur Zahlen als PinCode in Frage kommen.

Sicher nicht die glücklichste Lösung :noe:

Habe
import javax.swing.*;
importiert. Dazu sollte ja auch javax.swing.JJFormattedTextField gehören?!
Ich habe auch versucht das direkt zu importieren, allerdings dann mit einem weiteren Fehler

Also ich versuche immer derartige Klassen direkt zu importieren !
Und welcher Fehler war das ???:L
Verrätst Du das ? :autsch:

Gruß
Klaus

 

[daZza]

Dieser hier:

EinAus.java:9:19: error: cannot find symbol
import javax.swing.JJFormattedTextField;
^
symbol: class JJFormattedTextField
location: package javax.swing
1 error

 

[SlaterB]

du meine Güte, das doppelte J ist nicht zu erkennen?
und beim vorherigen Fehler ging es doch ziemlich deutlich um MaskFormatter statt JFormattedTextField,
solche Hilfsklassen liegen eben nicht unbedingt direkt in javax.swing

 

[VfL_Freak]

Moin,

Dieser hier:

EinAus.java:9:19: error: cannot find symbol
import javax.swing.JJFormattedTextField;
^
symbol: class JJFormattedTextField
location: package javax.swing

Nun heißt die Klasse ja auch JFormattedTextField - ein J reicht völlig

Gruß
Klaus

 

[daZza]

Habs copy pasted Ich versuchs gleich nochmal
edit: jetzt geht es probier ich das alles gleich nochmal ;-)

edit2: Zurück beim "Urpsprungsfehler" mit dem MaskFormatter. Muss ich dafür auch noch zusätzliche Dinge importieren? Ich habe im Moment folgende:

import java.awt.*;
import java.awt.event.*;
import javax.swing.*;
import javax.swing.JFormattedTextField;

 

[SlaterB]

du musst MasterFormatter importieren,
reichen all deine bisherigen Java-Kenntnisse zusammen aus, dies zu schaffen?

 

[VfL_Freak]

Moin,

Habs copy pasted

und genau aus diesem Grund sollte man sowas eigentlich auch nie machen ... oder zumindest mal ansatzweise draufschauen :lol:

Aber ok - Asche auf mein Haupt (siehe mein EDIT in meinem Beitrag von 14:02 h)

Gruß
Klaus

 

[Michael...]

edit2: Zurück beim "Urpsprungsfehler" mit dem MaskFormatter. Muss ich dafür auch noch zusätzliche Dinge importieren?

Ja, einfach mal in der API Doku nachschauen in welchem Package die Klasse steckt:
Java Platform SE 6

 

[daZza]

Habe nun also import javax.swing.text.MaskFormatter; gemacht, nun sagt er aber folgendes:

EinAus.java:93:68: error: unreported exception ParseException; must be caught or declared to be thrown
JFormattedTextField dateField = new JFormattedTextField (new MaskFormatter("####"));
^
1 error


Muss ich das jetzt einfach mit try/catch abfangen oder wie?

 

[VfL_Freak]

Moin,

zum Beispiel !

must be caught or declared to be thrown

Irgendwie ist die Meldung doch selbsterklärend, oder ?

Gruß
Klaus

 

[daZza]

Eigentlich schon Man muss das catchen oder declaren um es auszuwerfen... Die frage ist nur wie ich das mache. Mit try/catch hatte ich noch keinen Erfolg

 

[VfL_Freak]

Moin,

Die frage ist nur wie ich das mache. Mit try/catch hatte ich noch keinen Erfolg

und das bedeutet ???:L ???:L ???:L
Lass Dir doch nicht alle relevanten Details einzeln aus der Nase ziehen ..... ueh:

Gruß
Klaus

 

[daZza]

Hab mir jetzt erstmal einen Workaround gebastelt und werde mich da später nochmal dransetzen. Gebe aktuell einfach nur den aktuellen Vorgang aus und nicht noch zusätzliche alle davor. Damit wird das scrollen unnötig.

Vielen Dank an alle die geholfen haben