Inhalt von JTextfield an andere Klasse weitergeben

[unavailable]

Huhu Leute,

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

Spoiler: Main

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

	}
}

Spoiler: GUI

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;
	}
}

Spoiler: ActionListener

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?

 

[njans]

if (ae.getActionCommand() == "Anmelden")

Strings vergleicht man mit equals().

 

[Joose]

public class CGraphicalUserInterface extends JFrame

Von JFrame ableiten ohne die Klasse zu erweitern ist sinnfrei. Erstelle lieber ein Objekt von JFrame in deiner Klasse und arbeite damit.

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")
		{

		}
	}

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

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

Die Methode lässt sich kürzen

private boolean isNullOrEmpty(String s) {
    return (s!= null && s.equals(""));

PS: Hier hast du auch gleich einen String vergleich

alles funktioniert bis dahin, jedoch kriege ich ne nullpointerexception sobald ich den inhalt abfrage..

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.

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

		}

	}

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!

 

[unavailable]

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

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!

Mir fällt nur leider überhaupt nicht ein, wie das anders funktionieren soll. Magst du mir vielleicht nur nen kleinen Tip geben?

 

[kaoZ]

Kann sein das ich es jetzt überlese , aber wo genau setzt du die

ActionCommands

auf die du in deinen Anweisungen reagieren möchtest, bzw. nach welchen du die Fallunterscheidungen triffst ?

  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;
    }

Da fehlt noch

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.

 

[unavailable]

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..

 

[kaoZ]

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.

 

[unavailable]

Spoiler: die gui

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();
	}
}

Spoiler: der controller

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 :/

 

[kaoZ]

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 ?

 

[strußi]

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

 

[Harry Kane]

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.

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.

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.

 

[kaoZ]

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.

 

[VfL_Freak]

Moin,

[OT]

Hier erstmal der Kot.

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

Gruß
Klaus

 

[unavailable]

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.

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.

Danke für den Tip, ich versuch mich gleich daran.

Moin,

[OT]

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

Gruß
Klaus

[/ironie off]