Hallo zusammen,
ich habe für eine Aufgabe folgendes Programm erstellt:
	
	
	
	
	
		
	
Zu Beginn ist der Operator "Addition" markiert. Wenn man auf den Button "Berechnen" klickt, dann wird beim ersten Mal eine Addition durchgeführt.
Wird nun in dem Kombinationsfeld eine andere Operation ausgewählt, wird diese automatisch durchgeführt, es muss nicht mehr extra auf Berechnen geklickt werden.
Werden nun ungültige Zeichen eingegeben, dann wird dies über eine NumberFormatException abgefangen.
Wird z.B. bei der ersten Operation "Addition" auf Berechnen geklickt, erscheint die Fehlermeldung einmal. (was auch so gewollt ist)
Wählt man nun einen anderen Operator aus, dann erscheint die Fehlermeldung zweimal.
Könnt ihr mir bitte helfen, warum dies so ist und wie ich das abändern kann?
Vielen Dank!
Viele Grüße!
			
			ich habe für eine Aufgabe folgendes Programm erstellt:
		Java:
	
	import java.awt.BorderLayout;
import java.awt.GridLayout;
import java.awt.event.ActionEvent;
import java.awt.event.ActionListener;
import java.awt.event.ItemEvent;
import java.awt.event.ItemListener;
import java.text.DecimalFormat;
import javax.swing.JButton;
import javax.swing.JComboBox;
import javax.swing.JFrame;
import javax.swing.JLabel;
import javax.swing.JOptionPane;
import javax.swing.JPanel;
import javax.swing.JTextField;
import javax.swing.border.TitledBorder;
public class TaschenrechnerGUIV2 extends JFrame{
	//automatisch über Eclipse eingefügt
	private static final long serialVersionUID = 3006212012028893840L;
	
	//die Komponente
	//zwei Eingabefelder
	private JTextField eingabe1, eingabe2;
	//die vier Rechenarten in einem Array für das Kombinationsfeld
    private	String [] operatoren = {"Addition", "Subtraktion", "Multiplikation", "Division"};
	
    //das Kombinationsfeld für die Rechenoperationen
    private JComboBox auswahlOperatoren;
	
	
	//zwei Schaltflächen
	private JButton schaltflaecheBerechnen, schaltflaecheBeenden;
	//ein Label für die Ausgabe
	private JLabel ausgabe;
	
	//die innere Klasse für die Ereignisverarbeitung
	class MeinListener implements ActionListener, ItemListener {
		
		@Override
		public void actionPerformed(ActionEvent e) {
			//wurde auf Beenden geklickt?
			//dann das Programm beenden
			if (e.getActionCommand().equals("ende")) 
				System.exit(0);
			//wurde auf Berechnen geklickt?
			//dann eine Methode für die Berechnung aufrufen
			//und das Ergebnis anzeigen
			if (e.getActionCommand().equals("rechnen"))  
				ausgabe.setText(berechnen());
		}
		@Override
		public void itemStateChanged(ItemEvent e) {
			// TODO Auto-generated method stub
			Object ausloeser = e.getSource();
			
			if (ausloeser instanceof JComboBox)
			ausgabe.setText(berechnen());
		}
	}
	
	//der Konstruktor
	public TaschenrechnerGUIV2(String titel) {
		
		super(titel);
		
		//insgesamt 4 Panels
		JPanel panelEinAus, panelBerechnung, panelButtons, gross;
		
		//die Panels über die Methoden erstellen
		panelEinAus = panelEinAusErzeugen();
		panelBerechnung = panelBerechnungErzeugen();
		panelButtons = panelButtonErzeugen();
		
		//das Border-Layout benutzen
		//es ist Standard und muss nicht gesetzt werden
		
		//die Panels hinzufügen
		//die beiden größeren Panel fassen wir noch einmal zusammen
		gross = new JPanel();
		gross.add(panelEinAus);
		gross.add(panelBerechnung);
		//die beiden kommen in die Mitte
		add(gross,BorderLayout.CENTER);
		//das Panel mit den Buttons nach rechts
		add(panelButtons, BorderLayout.EAST);
		//die Standard-Aktion setzen
		setDefaultCloseOperation(JFrame.EXIT_ON_CLOSE);
		
		//packen und anzeigen
		pack();
		setVisible(true);
		
		//Größenänderungen sind nicht zugelassen
		//damit das mühsam erstellte Layout nicht durcheinander kommt
		setResizable(false);
	}
	//die Methode erzeugt das Panel für die Ein- und Ausgabe
	//und liefert es zurück
	private JPanel panelEinAusErzeugen() {
		JPanel tempPanel = new JPanel();
		//es enthält die Eingabefelder mit beschreibendem Text und die Ausgabe
		//die Länge der Felder ist auf 10 Zeichen beschränkt
		eingabe1 = new JTextField(10);
		eingabe2 = new JTextField(10);
		ausgabe = new JLabel("");
		
		//das Panel bekommt ein GridLaoyut mit 2 Spalten und etwas Abstand
		tempPanel.setLayout(new GridLayout(0,2,10,10));
		//ein beschreibendes Label für die erste Eingabe
		tempPanel.add(new JLabel("Zahl 1:"));
		//das erste Eingabefeld
		tempPanel.add(eingabe1);
		
		//und jetzt das zweite Eingabefeld
		tempPanel.add(new JLabel("Zahl 2: "));
		tempPanel.add(eingabe2);
		
		//und nun das Ausgabefeld für das Ergebnis
		tempPanel.add(new JLabel("Ergebnis: "));
		tempPanel.add(ausgabe);
		
		//einen Rahmen um das Panel ziehen
		tempPanel.setBorder(new TitledBorder("Ein- und Ausgabe"));
		
		//das Panel zurückliefern
		return tempPanel;
	}
	
	//die Methode erzeugt das Panel für die Auswahl
	//der Rechenoperation und liefert es zurück
	private JPanel panelBerechnungErzeugen() {
		JPanel tempPanel = new JPanel();
		//es enthält das Kombinationsfeld mit den Rechenoperationen
		
		auswahlOperatoren = new JComboBox(operatoren);
		
		tempPanel.add(auswahlOperatoren);
		
		
		
		//das Panel bekommt ein GridLaoyut mit 1 Spalte
		tempPanel.setLayout(new GridLayout(0,1));
	
		
		//und einen Rahmen
		tempPanel.setBorder(new TitledBorder("Operation: "));
		//das Panel zurückliefern
		return tempPanel;
	}
	
	//die Methode erzeugt das Panel für die Schaltflächen
	//und liefert es zurück
	private JPanel panelButtonErzeugen() {
		JPanel tempPanel = new JPanel();
		schaltflaecheBeenden = new JButton(" Beenden ");
		//das Aktion-Command setzen
		schaltflaecheBeenden.setActionCommand("ende");
		schaltflaecheBerechnen = new JButton("Berechnen");
		schaltflaecheBerechnen.setActionCommand("rechnen");
		
		//Zwischenpanel für die Schaltflächen
		//ebenfalls ein GridLayout
		tempPanel.setLayout(new GridLayout(0,1));
		tempPanel.add(schaltflaecheBerechnen);
		//ein leeres Label einfügen
		tempPanel.add(new JLabel());
		tempPanel.add(schaltflaecheBeenden);
		
		//die Schaltflächen mit dem Listener verbinden
		MeinListener listener = new MeinListener();
		schaltflaecheBeenden.addActionListener(listener);
		schaltflaecheBerechnen.addActionListener(listener);
		auswahlOperatoren.addItemListener(listener);
		
		//das Panel zurückliefern
		return tempPanel;
	}
	
	//die Methode berechnet das Ergebnis und liefert es als 
	//Zeichenkette zurück
	private String berechnen() {
		//ergebnis muss initialisiert werden
		double zahl1, zahl2, ergebnis = 0;
		boolean fehlerFlag = false;
	
		//jetzt mit Fehlerbehandlung und Ausgabe der Meldung durch eine eigene Methode
		try {
			zahl1 = Double.parseDouble(eingabe1.getText());
		}
		catch (Exception NumberFormatException) {
			fehlermeldung(eingabe1);
			return ("Nicht definiert");
		}
		
		try {
			zahl2 = Double.parseDouble(eingabe2.getText());
		}
		catch (Exception NumberFormatException) {
			fehlermeldung(eingabe2);
			//das Ergebnis ist nicht definiert
			return ("Nicht definiert");
		}
		//welche Operation soll durchgeführt werden?
		//Addition = Index 0
		if (auswahlOperatoren.getSelectedIndex() == 0)
			ergebnis = zahl1 + zahl2;
		
		if (auswahlOperatoren.getSelectedIndex() == 1)
			ergebnis = zahl1 - zahl2;
		
		if (auswahlOperatoren.getSelectedIndex() == 2)
			ergebnis = zahl1 * zahl2;
		//bei der Division überprüfen wir den zweiten Wert auf 0
		
		if (auswahlOperatoren.getSelectedIndex() == 3) {
			if (zahl2 != 0)
				ergebnis = zahl1 / zahl2;
			else 
				fehlerFlag = true;
		}
		
		//wenn es keine Probleme gegeben hat, liefern wir das Ergebnis zurück
		if (fehlerFlag == false) {
			//die Formatvorgabe über die Klasse DecimalFormat
			DecimalFormat formatFolge = new DecimalFormat("0.##");
			//das Format anwenden, das Ergebnis ist eine Zeichenkette
			return (formatFolge.format(ergebnis));
		}
		else
			return ("Nicht definiert");
	}
	
	private void fehlermeldung(JTextField eingabefeld) {
		JOptionPane.showMessageDialog(this, "Ihre Eingabe ist nicht gültig","Eingabefehler", JOptionPane.ERROR_MESSAGE);
		eingabefeld.requestFocus();
	}
	
}
	Zu Beginn ist der Operator "Addition" markiert. Wenn man auf den Button "Berechnen" klickt, dann wird beim ersten Mal eine Addition durchgeführt.
Wird nun in dem Kombinationsfeld eine andere Operation ausgewählt, wird diese automatisch durchgeführt, es muss nicht mehr extra auf Berechnen geklickt werden.
Werden nun ungültige Zeichen eingegeben, dann wird dies über eine NumberFormatException abgefangen.
Wird z.B. bei der ersten Operation "Addition" auf Berechnen geklickt, erscheint die Fehlermeldung einmal. (was auch so gewollt ist)
Wählt man nun einen anderen Operator aus, dann erscheint die Fehlermeldung zweimal.
Könnt ihr mir bitte helfen, warum dies so ist und wie ich das abändern kann?
Vielen Dank!
Viele Grüße!