Hilfe bei GUI Verständnisprobleme

[c_sidi90]

Hallo ich fange gerade an mit dem GUI erstellen. Jedoch versteh ich nicht wieso mein Code nicht funktioniert, hab ihn schon mehrfach umgeschrieben. Es sollen die beiden Werte aus den jTextfeldern addiert werden und im 3. Textfeld ausgegeben werden. Berechnung startet via jButton Hier der Code

import java.awt.event.ActionEvent;
import javax.swing.AbstractAction;
import javax.swing.GroupLayout;
import javax.swing.JButton;
import javax.swing.JComponent;
import javax.swing.JPanel;
import javax.swing.JTextField;
import javax.swing.LayoutStyle;

import javax.swing.WindowConstants;
import javax.swing.SwingUtilities;
import javax.xml.bind.ParseConversionEvent;


public class NewJFrame extends javax.swing.JFrame {
	private JTextField jTextField1;
	private JButton jButton1;
	private AbstractAction abstractAction1;
	private JPanel jPanel1;
	private JTextField jTextField3;
	private JTextField jTextField2;


	public int zahl1=0; //Variable für das Speichern des Inhalts von jTextField1
	public int zahl2=0; //Variable für das Speichern des Inhalts von jTextField2
	public int ergebnis = zahl1 + zahl2;

	public static void main(String[] args) {
		

		SwingUtilities.invokeLater(new Runnable() {
			public void run() {
				NewJFrame inst = new NewJFrame();
				inst.setLocationRelativeTo(null);
				inst.setVisible(true);
			}
		});
	}
	
	public NewJFrame() {
		super();
		initGUI();
	}
	
	private void initGUI() {
		try {
			GroupLayout thisLayout = new GroupLayout((JComponent)getContentPane());
			getContentPane().setLayout(thisLayout);
			setDefaultCloseOperation(WindowConstants.DISPOSE_ON_CLOSE);
			{
				jPanel1 = new JPanel();
				GroupLayout jPanel1Layout = new GroupLayout((JComponent)jPanel1);
				jPanel1.setLayout(jPanel1Layout);
				{
					
					jTextField3 = new JTextField();
					GroupLayout jTextField3Layout = new GroupLayout((JComponent)jTextField3);
					jTextField3.setLayout(jTextField3Layout);
					jTextField3.setText("Ergebnis");  //Ergebnisausgabefeld
					jTextField3Layout.setVerticalGroup(jTextField3Layout.createParallelGroup());
					jTextField3Layout.setHorizontalGroup(jTextField3Layout.createParallelGroup());
					
				}
				{
					jTextField1 = new JTextField();
					GroupLayout jTextField1Layout = new GroupLayout((JComponent)jTextField1);
					jTextField1.setLayout(jTextField1Layout);
					jTextField1.setText("Zahl eingeben");
					jTextField1Layout.setVerticalGroup(jTextField1Layout.createParallelGroup());
					jTextField1Layout.setHorizontalGroup(jTextField1Layout.createParallelGroup());
					zahl1 = Integer.parseInt(jTextField3.getText()); //Wandelt den Inhalt in einen Integer um (berechnung)
				}
				{
					jTextField2 = new JTextField();
					GroupLayout jTextField2Layout = new GroupLayout((JComponent)jTextField2);
					jTextField2.setLayout(jTextField2Layout);
					jTextField2.setText("Zahl eingeben");
					jTextField2Layout.setVerticalGroup(jTextField2Layout.createParallelGroup());
					jTextField2Layout.setHorizontalGroup(jTextField2Layout.createParallelGroup());
					zahl2 = Integer.parseInt(jTextField3.getText()); //Wandelt den Inhalt in einen Integer um (berechnung)
					
					
				
				}
				{
					jButton1 = new JButton();
					GroupLayout jButton1Layout = new GroupLayout((JComponent)jButton1);
					jButton1.setLayout(jButton1Layout);
					jButton1.setText("Berechne");
					jButton1.setAction(getAbstractAction1());
					jButton1Layout.setHorizontalGroup(jButton1Layout.createParallelGroup());
					jButton1Layout.setVerticalGroup(jButton1Layout.createParallelGroup());
				}
				jPanel1Layout.setHorizontalGroup(jPanel1Layout.createSequentialGroup()
					.addGroup(jPanel1Layout.createParallelGroup()
					    .addGroup(jPanel1Layout.createSequentialGroup()
					        .addComponent(jTextField3, GroupLayout.PREFERRED_SIZE, 228, GroupLayout.PREFERRED_SIZE))
					    .addGroup(GroupLayout.Alignment.LEADING, jPanel1Layout.createSequentialGroup()
					        .addComponent(jTextField1, GroupLayout.PREFERRED_SIZE, GroupLayout.PREFERRED_SIZE, GroupLayout.PREFERRED_SIZE)
					        .addGap(30)
					        .addComponent(jTextField2, GroupLayout.PREFERRED_SIZE, GroupLayout.PREFERRED_SIZE, GroupLayout.PREFERRED_SIZE)
					        .addGap(25)
					        .addComponent(jButton1, GroupLayout.PREFERRED_SIZE, GroupLayout.PREFERRED_SIZE, GroupLayout.PREFERRED_SIZE)))
					.addContainerGap(92, Short.MAX_VALUE));
				jPanel1Layout.setVerticalGroup(jPanel1Layout.createSequentialGroup()
					.addGap(8)
					.addGroup(jPanel1Layout.createParallelGroup(GroupLayout.Alignment.BASELINE)
					    .addComponent(jTextField1, GroupLayout.Alignment.BASELINE, GroupLayout.PREFERRED_SIZE, GroupLayout.PREFERRED_SIZE, GroupLayout.PREFERRED_SIZE)
					    .addComponent(jTextField2, GroupLayout.Alignment.BASELINE, GroupLayout.PREFERRED_SIZE, GroupLayout.PREFERRED_SIZE, GroupLayout.PREFERRED_SIZE)
					    .addComponent(jButton1, GroupLayout.Alignment.BASELINE, GroupLayout.PREFERRED_SIZE, GroupLayout.PREFERRED_SIZE, GroupLayout.PREFERRED_SIZE))
					.addGap(18)
					.addComponent(jTextField3, GroupLayout.PREFERRED_SIZE, 21, GroupLayout.PREFERRED_SIZE)
					.addContainerGap(86, Short.MAX_VALUE));
			}
			thisLayout.setVerticalGroup(thisLayout.createSequentialGroup()
				.addContainerGap(31, 31)
				.addComponent(jPanel1, GroupLayout.PREFERRED_SIZE, 154, GroupLayout.PREFERRED_SIZE)
				.addContainerGap(81, Short.MAX_VALUE));
			thisLayout.setHorizontalGroup(thisLayout.createSequentialGroup()
				.addContainerGap(22, 22)
				.addComponent(jPanel1, GroupLayout.PREFERRED_SIZE, 320, GroupLayout.PREFERRED_SIZE)
				.addContainerGap(50, Short.MAX_VALUE));
			pack();
			setSize(400, 300);
		} catch (Exception e) {
		    //add your error handling code here
			e.printStackTrace();
		}
	}
	
	
	
	private AbstractAction getAbstractAction1() {
		if(abstractAction1 == null) {
			abstractAction1 = new AbstractAction("abstractAction1", null) {
				public void actionPerformed(ActionEvent evt) {
					
					jTextField3.setText(Integer.toString(ergebnis));
				}
			};
		}
		return abstractAction1;
	}

}

Danke !

 

[SlaterB]

main-Methode muss static sein

immer Fehlermeldung/ beobachtetes Fehlverhalten dazuposten

 

[nrg]

public static void main(String[] args) {

edit: zu langsam 🙁. allerdings ist der Titel in Bezug auf die Problematik auch nicht schlecht 😉

 

[c_sidi90]

ups mein Fehler, danke! Funktioniert allerdings immernoch nicht richtig.

 

[SlaterB]

->

immer Fehlermeldung/ beobachtetes Fehlverhalten dazuposten

<-

 

[nrg]

bisschen genauere Fehlerbeschreibungen wären wünschenswert..

 

[SlaterB]

sorry nrg, immer zu langsam 😉

 

[Marco13]

Ungetestet, nur aus der Kristallkugel: Schreib' mal in die actionPerformed

zahl1 = Integer.parseInt(jTextField2.getText());
zahl2 = Integer.parseInt(jTextField3.getText());
ergebnis = zahl1+zahl2;
jTextField3.setText(Integer.toString(ergebnis));

 

[c_sidi90]

Hab den Code nochmal aktualisiert, vlt schaut ihr es euch selbst in einer Programmierumgebung an, die Zahlen werden nicht addiert und richtig ausgegeben. Habe ich es denn überhaupt richtig deklariert mit den Variablen un den Typumwandlungen von String zu Int und am Ende bei der Ausgabe wieder mit toString ?

 

[Marco13]

Da hat die Kugel ja zumindest ansatzweise funktioniert 🙂

 

[c_sidi90]

Das sah auf den ersten Blick logisch aus Marco, allerdings wäre ich nicht darauf gekommen da ich dachte die Funktionen nicht erst in der Eventmethode einzubauen. Funktioniert leider trotzdem noch nicht, bei der Kristallkugellösung kommt

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 NewJFrame$2.actionPerformed(NewJFrame.java:138)
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.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)

 

[Tomate_Salat]

du liest die neuen Werte nie ein und selbst wenn, wird das Ergebnis schon bei der Erstellung der Klasse berechnet und dann nie wieder. Du brauchst entweder Listener, die die Eingabe in den TextFeldern überwachen und das Ergebnis neu berechnen, oder einen Button, der alles neu berechnet, wenn du draufklickst. D.h. du musst auch [c]ergebnis[/c] immer neu berechnen.

(und vllt. solltest du auch noch nicht auf einen GUI-Editor zurückgreifen. Arbeite erstmal ein Tutorial bzgl. GUI-Programmierung durch, damit du verstehen kannst, was du machen sollst)

 

[c_sidi90]

Ich hatte es zuvor so programmiert, dass die Umwandlungen in einer seperaten Methode geschehen, dann kam als Ergebnis aber immer 0 raus, da er die addition der Methode nicht berücksichtigt hat. Sehr fragwürdig das Ganze lach

btw

Mittlerweile ist die Berechnung in der Event methode des Buttons

private AbstractAction getAbstractAction1() {
		if(abstractAction1 == null) {
			abstractAction1 = new AbstractAction("abstractAction1", null) {
				public void actionPerformed(ActionEvent evt) {
					zahl1 = Integer.parseInt(jTextField2.getText());
					zahl2 = Integer.parseInt(jTextField3.getText());
					ergebnis = zahl1+zahl2;
					jTextField3.setText(Integer.toString(ergebnis));
				}
			};
		}
		return abstractAction1;
	}

 

[c_sidi90]

Habs nicht überprüft marco hat bei der Wertzuweisung der jTextfelder mit den Zahlvariablen die nummern vertauscht.

So ist es richtig

public void actionPerformed(ActionEvent evt) {
					zahl1 = Integer.parseInt(jTextField2.getText());
					zahl2 = Integer.parseInt(jTextField1.getText());
					ergebnis = zahl1+zahl2;
					jTextField3.setText(Integer.toString(ergebnis));

funktioniert jetzt alles, eine Runde Danke an alle!