Problem mit NumberFormatException

[jacka]

Hi, habe hier folgenden code geschrieben:
von Zeile 199-211 muss irgendwas nicht passen, weil wenn ich das gui starte und im Preisfeld den Preis eingebe dann kommt immer eine NumberFormatException und die Fehlermeldung: bitte als int eingeben obwohl ich alles richtig eingebe kann mir jmd helfen?

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

import javax.swing.JButton;
import javax.swing.JFrame;
import javax.swing.JLabel;
import javax.swing.JOptionPane;
import javax.swing.JPanel;
import javax.swing.JScrollPane;
import javax.swing.JTextArea;
import javax.swing.JTextField;

public class BibliothekGui extends JFrame {
	private JLabel titelLabel = new JLabel("Titel: ");
	private JTextField titelFeld = new JTextField();
	private JLabel isbnLabel = new JLabel("ISBN: ");
	private JTextField isbnFeld = new JTextField();
	private JLabel preisLabel = new JLabel("Preis: ");
	private JTextField preisFeld = new JTextField();
	private JButton hinzufuegenButton = new JButton("Hinzufuegen");
	private JButton entfernenButton = new JButton("Entfernen");
	private JTextArea ausgabe = new JTextArea();
	final JScrollPane scroll = new JScrollPane(ausgabe);
	private JPanel eingabeBereich = new JPanel();

	private BibliothekSet bibliothek;

	public BibliothekGui() throws Exception {
		bibliothek = new BibliothekSet("Bibliothek");

		setLayout(new GridLayout(2, 1));
		setTitle("Bibliothek");
		setSize(350, 350);
		setDefaultCloseOperation(EXIT_ON_CLOSE);

		eingabeBereich.setLayout(new GridLayout(4, 2));
		eingabeBereich.add(titelLabel);
		eingabeBereich.add(titelFeld);
		eingabeBereich.add(isbnLabel);
		eingabeBereich.add(isbnFeld);
		eingabeBereich.add(preisLabel);
		eingabeBereich.add(preisFeld);
		eingabeBereich.add(hinzufuegenButton);
		eingabeBereich.add(entfernenButton);
		add(eingabeBereich);

		hinzufuegenButton.addActionListener(new ActionListener() {

			@Override
			public void actionPerformed(ActionEvent e) {
				String inputTitel = titelFeld.getText();
				titelFeld.setText("");

				if (inputTitel == null) {
					try {
						throw new Exception("Titel darf nicht null sein");
					} catch (Exception ec) {
						ausgabe.setText(ec.getMessage());
					}
				}
				if (inputTitel.isEmpty()) {
					try {
						throw new Exception("Titel darf nicht leer sein");
					} catch (Exception ec) {
						ausgabe.setText(ec.getMessage());
					}
				}
				String inputIsbn = isbnFeld.getText();
				isbnFeld.setText("");

				if (inputIsbn == null) {
					try {
						throw new Exception("ISBN darf nicht null sein");
					} catch (Exception ec) {
						ausgabe.setText(ec.getMessage());
					}
				}
				if (inputIsbn.isEmpty()) {
					try {
						throw new Exception("ISBN darf nicht leer sein");
					} catch (Exception ec) {
						ausgabe.setText(ec.getMessage());
					}
				}
				String inputPreis = preisFeld.getText();
				preisFeld.setText("");

				if (inputPreis == null) {
					try {
						throw new Exception("Preis darf nicht null sein");
					} catch (Exception ec) {
						ausgabe.setText(ec.getMessage());
					}
				}
				if (inputPreis.isEmpty()) {
					try {
						throw new Exception("Preis darf nicht leer sein");
					} catch (Exception ec) {
						ausgabe.setText(ec.getMessage());
					}
				}

				int i = Integer.parseInt(inputPreis);

				if (i <= 0) {
					try {
						throw new Exception(
								"Preis darf nicht kleiner oder gleich 0 sein");
					} catch (Exception ec) {
						ausgabe.setText(ec.getMessage());
					}
				}

				;
				try {
					Buch b = new Buch(titelFeld.getText(), isbnFeld.getText(),
							Integer.parseInt(preisFeld.getText()));
					bibliothek.hinzufuegen(b);
					ausgabe.setText("" + b);
				} catch (NumberFormatException e1) {
					JOptionPane
							.showMessageDialog(BibliothekGui.this,
									"Fehler beim Umwandeln des Preises. Bitte als int eingeben.");
				} catch (Exception e1) {
					JOptionPane.showMessageDialog(BibliothekGui.this, e1
							.getMessage());
				}

			}
		});

		entfernenButton.addActionListener(new ActionListener() {

			@Override
			public void actionPerformed(ActionEvent e) {
				String inputTitel = titelFeld.getText();
				titelFeld.setText("");

				if (inputTitel == null) {
					try {
						throw new Exception("Titel darf nicht null sein");
					} catch (Exception ec) {
						ausgabe.setText(ec.getMessage());
					}
				}
				if (inputTitel.isEmpty()) {
					try {
						throw new Exception("Titel darf nicht leer sein");
					} catch (Exception ec) {
						ausgabe.setText(ec.getMessage());
					}
				}
				String inputIsbn = isbnFeld.getText();
				isbnFeld.setText("");

				if (inputIsbn == null) {
					try {
						throw new Exception("ISBN darf nicht null sein");
					} catch (Exception ec) {
						ausgabe.setText(ec.getMessage());
					}
				}
				if (inputIsbn.isEmpty()) {
					try {
						throw new Exception("ISBN darf nicht leer sein");
					} catch (Exception ec) {
						ausgabe.setText(ec.getMessage());
					}
				}
				String inputPreis = preisFeld.getText();
				preisFeld.setText("");

				if (inputPreis == null) {
					try {
						throw new Exception("Preis darf nicht null sein");
					} catch (Exception ec) {
						ausgabe.setText(ec.getMessage());
					}
				}
				if (inputPreis.isEmpty()) {
					try {
						throw new Exception("Preis darf nicht leer sein");
					} catch (Exception ec) {
						ausgabe.setText(ec.getMessage());
					}
				}

				int i = Integer.parseInt(inputPreis);

				if (i <= 0) {
					try {
						throw new Exception(
								"Preis darf nicht kleiner oder gleich 0 sein");
					} catch (Exception ec) {
						ausgabe.setText(ec.getMessage());
					}
				}
				try {
					Buch b = new Buch(titelFeld.getText(), isbnFeld.getText(),
							Integer.parseInt(preisFeld.getText()));
					bibliothek.entfernen(b);
					ausgabe.setText("" + bibliothek);
				} catch (NumberFormatException e1) {
					JOptionPane
							.showMessageDialog(BibliothekGui.this,
									"Fehler beim Umwandeln des Preises. Bitte als double eingeben.");
				} catch (Exception e1) {
					JOptionPane.showMessageDialog(BibliothekGui.this, e1
							.getMessage());
				}
				// Buch b = new Buch(titelFeld.getText(), isbnFeld.getText(),
				// Double.parseDouble(preisFeld.getText()));
				// bibliothek.entfernen(b);
				//
				// ausgabe.setText("" + bibliothek);

			}
		});

		add(scroll);

	}

	public static void main(String[] args) {
		try {
			BibliothekGui frame = new BibliothekGui();
			frame.setVisible(true);
		} catch (Exception e) {
			System.out
					.println("Fehler beim Starten des Programmes, breche ab.");
		}
	}
}

 

[Fu3L]

Ohne die Klasse Buch kann mans schlecht Testen.

Gibst du auch wirklich nur Ganzzahlen ein und nicht 19.99 oder so?

Edit: Mein Nachposter hat Recht^^

 

[TR]

[JAVA=172] preisFeld.setText("");[/code]
[JAVA=201] Integer.parseInt(preisFeld.getText()));[/code]

das funktioniert nicht.
einfach nur

i

übergeben!

 

[jacka]

aja und ja gebe ich ich hatte es ja vorher mit double und da gings nicht und dann wollt ichs mit int probieren und dann kommt auch die Meldung

hier die klasse buch

import java.io.Serializable;

public class Buch implements Serializable {
	private String titel;
	private String isbn;
	private boolean ausgeliehen;

	public Buch(String titel, String isbn, boolean ausgeliehen)
			throws Exception {
		if (titel == null) {
			throw new Exception("Name darf nicht null sein.");
		} else if (titel.isEmpty()) {
			throw new Exception("Name darf nicht leer sein.");
		}

		if (isbn == null) {
			throw new Exception("ISBN darf nicht null sein.");
		}
		if (isbn.isEmpty()) {
			throw new Exception("ISBN darf nciht leer sein");
		}

		this.titel = titel;
		this.isbn = isbn;
		this.ausgeliehen = ausgeliehen;
	}

	public String getTitel() {
		return titel;
	}

	public String getISBN() {
		return isbn;
	}

	public boolean isAusgeliehen() {
		return ausgeliehen;
	}

	public String toString() {
		return "Buch [titel=" + titel + ", isbn=" + isbn + ", ausgeliehen="
				+ ausgeliehen + "]";
	}

	public boolean equals(Object obj) {

		if (obj == null) {
			return false;
		}

		if (this == obj) {
			return true;
		}

		if (obj.getClass() != getClass()) {
			return false;
		}

		Buch b = (Buch) obj;

		return isbn.equals(b.isbn);
	}

	public int hashCode() {
		return isbn.hashCode();
	}

}

 

[SlaterB]

Buch ist nicht so dringend, den entscheidenden Code
> Integer.parseInt(preisFeld.getText())
kann man sicherlich extrahieren,

danach zu fragen aber nicht zu verraten, was eingetippt wird, ist allerdings wirklich ein starkes Stück

---------

übertroffen noch durch:

                if (inputIsbn == null) {
                    try {
                        throw new Exception("ISBN darf nicht null sein");
                    } catch (Exception ec) {
                        ausgabe.setText(ec.getMessage());
                    }
                }

wie wärs mit

                if (inputIsbn == null) {
                    ausgabe.setText("ISBN darf nicht null sein");
                }

?

der Abbruch der Methode findet bisher in beiden Varianten nicht statt

 

[jgh]

nebenbei bemerkt: es gibt keinen Konstruktor Buch, der als Parameter [c](String titel, String isbn, int preis)[/c] erwartet...zumal die Klasse Buch noch nicht mal einen Preis als Instanzvariable besitzt!

 

[AFlieger]

Ich hab noch was gefunden:lol:

} catch (NumberFormatException e1) {
                    
JOptionPane.showMessageDialog(BibliothekGui.this,
"Fehler beim Umwandeln des Preises. Bitte als double eingeben.");

würd einfach Double.parseDouble vorschlagen.

entbindet natürlich nicht davon, sicherzustellen, dass korrekte Zahlenwerte vorhanden sind, wenn man die NFE vermeiden möchte.