Inhalt von JTextfield an andere Klasse weitergeben

U

unavailable

Mitglied
Huhu Leute,

ich programmiere grade nen Adressbuch, bin schön am Klassen auslagern pi pa po.
Hier erstmal der Kot.

Java: 
public class CMain
{
	public static void main(String[] args)
	{
		SwingUtilities.invokeLater(new Runnable()
		{
			public void run()
			{
				CGraphicalUserInterface GUI = new CGraphicalUserInterface();
			}
		});

	}
}
Java: 
import java.awt.Dimension;
import java.awt.GridBagConstraints;
import java.awt.GridBagLayout;
import java.awt.Insets;
import java.awt.Toolkit;
import java.awt.event.ActionListener;
import javax.swing.JButton;
import javax.swing.JFrame;
import javax.swing.JLabel;
import javax.swing.JPanel;
import javax.swing.JPasswordField;
import javax.swing.JTextField;

public class CGraphicalUserInterface extends JFrame
{
	GridBagConstraints c = new GridBagConstraints();
	JPanel panel = new JPanel(new GridBagLayout());
	Insets insets = new Insets(3, 3, 3, 3);

	CGraphicalUserInterface()
	{
		super("Adressbuch");
		setSize(400, 150);
		centerWindow(this);
		setDefaultCloseOperation(JFrame.EXIT_ON_CLOSE);

		this.add(panel);

		createButton("Anmelden", 1, 2, panel, c);
		createButton("Registration", 0, 2, panel, c);

		createLabel("Benutzername", 0, 0, panel, c);
		createLabel("Passwort", 0, 1, panel, c);

		createTextField(20, 1, 0, panel, c);

		JPasswordField pw = new JPasswordField(20);
		c.gridx = 1;
		c.gridy = 1;
		panel.add(pw, c);

		setVisible(true);
	}

	private void centerWindow(JFrame frame)
	{
		Dimension screenSize = Toolkit.getDefaultToolkit().getScreenSize();

		int x = screenSize.width;
		int y = screenSize.height;

		int posX = (x - frame.getWidth()) / 2;
		int posY = (y - frame.getHeight()) / 2;

		frame.setLocation(posX, posY);
	}

	private JButton createButton(String text, int posX, int posY, JPanel panel,	GridBagConstraints c)
	{
		JButton button = new JButton(text);
		button.addActionListener(new CActionListener(button));
		c.gridx = posX;
		c.gridy = posY;
		c.insets = insets;
		panel.add(button, c);
		return button;
	}

	private JTextField createTextField(int length, int posX, int posY, JPanel panel, GridBagConstraints c)
	{
		JTextField textfield = new JTextField(length);
		textfield.addActionListener(new CActionListener(textfield));
		c.gridx = posX;
		c.gridy = posY;
		c.insets = insets;
		panel.add(textfield, c);
		return textfield;
	}

	private JLabel createLabel(String text, int posX, int posY, JPanel panel, GridBagConstraints c)
	{
		JLabel label = new JLabel(text);
		c.gridx = posX;
		c.gridy = posY;
		c.insets = insets;
		panel.add(label, c);
		return label;
	}
}
Java: 
import java.awt.event.ActionEvent;
import java.awt.event.ActionListener;

import javax.swing.JButton;
import javax.swing.JTextField;

public class CActionListener implements ActionListener
{
	private JButton button;
	private JTextField textfield;

	public CActionListener(JButton button)
	{
		this.button = button;
	}
	
	public CActionListener(JTextField textfield)
	{
		this.textfield = textfield;
	}

	public void actionPerformed(ActionEvent ae)
	{
		if (ae.getActionCommand() == "Anmelden")
		{
			if(isNullOrEmpty(textfield.getText()))
			{
				System.out.println("Jeah!");
			}
		}
		if (ae.getActionCommand() == "Registration")
		{

		}

	}

	private boolean isNullOrEmpty(String s)
	{
		if (s != null && s != "")
		{
			return true;
		}
		else
		{
			return false;
		}
	}
}

Ich übergebe die Buttons & die Textfelder an meinen ActionListener, die Buttons werden beim Klick erkannt & alles funktioniert bis dahin, jedoch kriege ich ne nullpointerexception sobald ich den inhalt abfrage..

seht ihr den fehler?
 
unavailable hat gesagt.:
Java: 
public class CGraphicalUserInterface extends JFrame
Zum Vergrößern anklicken....
Von JFrame ableiten ohne die Klasse zu erweitern ist sinnfrei. Erstelle lieber ein Objekt von JFrame in deiner Klasse und arbeite damit.

unavailable hat gesagt.:
Java: 
public class CActionListener implements ActionListener
{
	public void actionPerformed(ActionEvent ae)
	{
		if (ae.getActionCommand() == "Anmelden")
		{
			if(isNullOrEmpty(textfield.getText()))
			{
				System.out.println("Jeah!");
			}
		}
		if (ae.getActionCommand() == "Registration")
		{

		}
	}
Zum Vergrößern anklicken....

Wo legst du das ActionCommand von Button bzw. Textfield fest?
Strings werden mit equals() nicht mit == verglichen!

unavailable hat gesagt.:
Java: 
	private boolean isNullOrEmpty(String s)
	{
		if (s != null && s != "")
		{
			return true;
		}
		else
		{
			return false;
		}
	}
}
Zum Vergrößern anklicken....

Die Methode lässt sich kürzen 😉

Java: 
private boolean isNullOrEmpty(String s) {
    return (s!= null && s.equals(""));
PS: Hier hast du auch gleich einen String vergleich

unavailable hat gesagt.:
alles funktioniert bis dahin, jedoch kriege ich ne nullpointerexception sobald ich den inhalt abfrage..
Zum Vergrößern anklicken....

Welchen Inhalt fragst du ab? Wie schaut der StackTrace der Exception aus? Normal steht im StackTrace drinnen in welcher Zeile der Fehler passiert. Daher bei Exceptions immer auch den StackTrace posten und den Code bei der entsprechenden Stelle, so kann man leichter helfen.


unavailable hat gesagt.:
Java: 
	public void actionPerformed(ActionEvent ae)
	{
		if (ae.getActionCommand() == "Anmelden")
		{
			if(isNullOrEmpty(textfield.getText()))
			{
				System.out.println("Jeah!");
			}
		}
		if (ae.getActionCommand() == "Registration")
		{

		}

	}
Zum Vergrößern anklicken....
Jeder deiner Button bekommt einen eigenes ActionListener Objekt gesetzt! Somit ist bei jedem ActionListener der Buttons das textfield null => daher auch die NullPointerException sobald du getText() aufrufen willst!
 
Zuletzt bearbeitet:
Joose hat gesagt.:
Von JFrame ableiten ohne die Klasse zu erweitern ist sinnfrei. Erstelle lieber ein Objekt von JFrame in deiner Klasse und arbeite damit.



Wo legst du das ActionCommand von Button bzw. Textfield fest?
Strings werden mit equals() nicht mit == verglichen!
Ups, stimmt.


Die Methode lässt sich kürzen 😉

Java: 
private boolean isNullOrEmpty(String s) {
    return (s!= null && s.equals(""));
PS: Hier hast du auch gleich einen String vergleich
Danke für den Tipp!


Welchen Inhalt fragst du ab? Wie schaut der StackTrace der Exception aus? Normal steht im StackTrace drinnen in welcher Zeile der Fehler passiert. Daher bei Exceptions immer auch den StackTrace posten und den Code bei der entsprechenden Stelle, so kann man leichter helfen.



Jeder deiner Button bekommt einen eigenes ActionListener Objekt gesetzt! Somit ist bei jedem ActionListener der Buttons das textfield null => daher auch die NullPointerException sobald du getText() aufrufen willst!
Zum Vergrößern anklicken....
Mir fällt nur leider überhaupt nicht ein, wie das anders funktionieren soll. Magst du mir vielleicht nur nen kleinen Tip geben?
 
Kann sein das ich es jetzt überlese , aber wo genau setzt du die
Code: 
ActionCommands
auf die du in deinen Anweisungen reagieren möchtest, bzw. nach welchen du die Fallunterscheidungen triffst ?

Java: 
  private JButton createButton(String text, int posX, int posY, JPanel panel, GridBagConstraints c)
    {
        JButton button = new JButton(text);
        button.addActionListener(new CActionListener(button));
        c.gridx = posX;
        c.gridy = posY;
        c.insets = insets;
        panel.add(button, c);
        return button;
    }
Zum Vergrößern anklicken....

Da fehlt noch

Java: 
button.setActionCommand("wtf!?");

//bzw.

button.setActionCommand(text); // in deinem Fall

würde ich mal sagen, du setzt zwar den namen aber keine Command auf die reagiert werden soll.
 
Zuletzt bearbeitet:
Das war gar nicht mein Problem, hat super ohne ActionCommands funktioniert.
ich weiß aber ehrlich gesagt nicht wie ichdas bewerkstelligen soll, dass meine actionlistener klasse weiß, was
in den textfeldern drinne steht..
 
Indem du öffentliche getter und setter für deine Textefelder in deiner Gui anbietest, über diese kannst du , insofern du eine Referenz auf deine Gui hälst zugreifen, den Text aus deinem Textfeld auslesen, und z.B beim Klick auf den Button den Text in ein Label oder ähnliches einfügen.

wenn du ein Beispiel benötigst, kann ich dir das gerne flott schreiben.
 
Zuletzt bearbeitet:
Java: 
public class CGraphicalUserInterface
{
	GridBagConstraints c = new GridBagConstraints();
	JPanel panel = new JPanel(new GridBagLayout());
	Insets insets = new Insets(3, 3, 3, 3);
	JFrame frame = new JFrame();
	
	JTextField textfield;
	JButton button1, button2;
	JPasswordField passwordfield;

	CGraphicalUserInterface()
	{
		frame.setTitle("Adressbuch");
		frame.setSize(400, 150);
		frame.setDefaultCloseOperation(JFrame.EXIT_ON_CLOSE);
		frame.add(panel);

		centerWindow(frame);

		button1 = createButton("Anmelden", 1, 2, panel, c);
		button2 = createButton("Registration", 0, 2, panel, c);

		textfield = createTextField(20, 1, 0, panel, c);

		passwordfield = new JPasswordField(20);
		c.gridx = 1;
		c.gridy = 1;
		panel.add(passwordfield, c);

		createLabel("Benutzername", 0, 0, panel, c);
		createLabel("Passwort", 0, 1, panel, c);
		
		button1.addActionListener(new CActionListener(button1));
		button2.addActionListener(new CActionListener(button2));

		frame.setVisible(true);
	}


	private JButton createButton(String text, int posX, int posY, JPanel panel,
			GridBagConstraints c)
	{
		JButton button = new JButton(text);
		button.setActionCommand(text);
		c.gridx = posX;
		c.gridy = posY;
		c.insets = insets;
		panel.add(button, c);
		return button;
	}

	private JTextField createTextField(int length, int posX, int posY,
			JPanel panel, GridBagConstraints c)
	{
		JTextField textfield = new JTextField(length);
		c.gridx = posX;
		c.gridy = posY;
		c.insets = insets;
		panel.add(textfield, c);
		return textfield;
	}

	public void setTextfieldText(String text)
	{
		textfield.setText(text);
	}
	
	public String getTextfieldText()
	{
		return textfield.getText();
	}
}

Java: 
import java.awt.event.ActionEvent;
import java.awt.event.ActionListener;

import javax.swing.JButton;
import javax.swing.JTextField;


public class CActionListener implements ActionListener
{
	private JTextField textfield;
	private JButton button;
	private CGraphicalUserInterface gui;
	private String text;
	
	public CActionListener(JButton button)
	{
		this.button = button;
	}
	
	public void actionPerformed(ActionEvent e)
	{
		if(e.getActionCommand().equals("Anmelden"))
		{
			text = gui.getTextfieldText();
		}

	}
}

kriege immer noch eine nullpointer :OO langsam raff ich echt gar nichts mehr :/
 
Debugg doch einfach mal und schaue welche variable mit null belegt ist bzw. wird oder bleibt.

was sagt der Stacktrace , an welcher stelle wird die NPE geworfen ?
 
Zuletzt bearbeitet:
Hey

wo übergibst/erzeugst du im Controller für das

private CGraphicalUserInterface gui; in Zeile 12

Im Controller hast du nur eine Variable erstellt und nichts übergeben ^^.

Grüße
 
kaoZ hat gesagt.:
Indem du öffentliche getter und setter für deine Textefelder in deiner Gui anbietest, über diese kannst du , insofern du eine Referenz auf deine Gui hälst zugreifen, den Text aus deinem Textfeld auslesen, und z.B beim Klick auf den Button den Text in ein Label oder ähnliches einfügen.
Zum Vergrößern anklicken....

Dagegen!
Ich würde auf jeden Fall raten, nur eine Referenz auf das JTextField zu übergeben! Oder übergebe dem ActionListener eine Referenz auf zwei JTextFields, eines zum lesen und das andere zum schreiben.
Java: 
public class TextShifter implements ActionListener{
    private JTextField target;
    private JTextField source;
    public TextShifter(JTextField source, JTextField target){
        this.source = source;
        this.target = target;
    }
    public void actionPerformed(ActionEvent ae){
        target.setText(source.getText());
    }
}
Edit: Der String, der an den KOnstruktor eines JButtons übergeben wird, wird aucu automatisch zum actionCommand. Das actionCommand muss nicht extra gesetzt werden.
Und natürlich hat Strußi recht, dass du die NPE vom Zugriff auf eine nicht initialisierte Variable kommt.
 
Zuletzt bearbeitet:
Oder natürlich so, War lediglich ein Vorschlag wie man es lösen könnte, dass es nicht die beste Wahl wäre es so zu realisieren, ist mir durchaus bewusst da so weitere Abhängigkeiten geschaffen werden, welche die Kopplung beeinflussen.
 
Harry Kane hat gesagt.:
Dagegen!
Ich würde auf jeden Fall raten, nur eine Referenz auf das JTextField zu übergeben! Oder übergebe dem ActionListener eine Referenz auf zwei JTextFields, eines zum lesen und das andere zum schreiben.
Java: 
public class TextShifter implements ActionListener{
    private JTextField target;
    private JTextField source;
    public TextShifter(JTextField source, JTextField target){
        this.source = source;
        this.target = target;
    }
    public void actionPerformed(ActionEvent ae){
        target.setText(source.getText());
    }
}
Edit: Der String, der an den KOnstruktor eines JButtons übergeben wird, wird aucu automatisch zum actionCommand. Das actionCommand muss nicht extra gesetzt werden.
Und natürlich hat Strußi recht, dass du die NPE vom Zugriff auf eine nicht initialisierte Variable kommt.
Zum Vergrößern anklicken....
Danke für den Tip, ich versuch mich gleich daran.

VfL_Freak hat gesagt.:
Moin,

[OT]

Ist das ernst gemeint ???:L
Oder meinst Du doch eher "Code" ? :lol:
[/OT]

Gruß
Klaus
Zum Vergrößern anklicken....
[/ironie off]
 

Aktuelle Jobs

Neue Themen

Ähnliche Themen

Eingegebenen Text in JOptionPane in andere Klasse weitergeben
Antworten: 2
Variable an andere Klasse weitergeben - Was mache ich falsch?
Antworten: 2
Instanz einer Klasse an eine andere weitergeben
Antworten: 16
Connection conn in andere Klasse / Objekt weitergeben?
Antworten: 17
variable an andere klassen weitergeben?
Antworten: 3

Zurück
Oben