Swing JTextfield.getText() == empty?

[OasisCritter]

Hallo,

hab folgendes Problem beim auslesen des Textes aus einem JTextField:

           // variable
	private JTextField Lang110 =null;
           ...

//Getter
 public JTextField getLang110TextField() {
		 if(Lang110 == null) {
			 Lang110 = new JTextField();
			 Lang110.setBounds(new Rectangle(0,ComponentHeight, ComponentWidth, ComponentHeight));
			 Lang110.setFont(new Font("italic", Font.BOLD, 16));
		 }
		 return Lang110;
	 }

 public JButton getAddButton() {
		 if(add==null) {
			 add= new JButton("Hinzufügen");
			 add.addActionListener(new ActionListener() {
                                            @Override
				public void actionPerformed(ActionEvent e) {
                                                      ...
					System.out.println(getLang110TextField().getText());
				         ...
}
}}

@Override
	public void setFrameContents() {
                      Container contentPane = getContentPane();
		contentPane.setLayout(null);
                 ...     
                       contentPane.add(getLang110TextField());
                 ...
}

Ihr müsst euch vorstellen das dass Frame richtig implementiert ist. Also wenn ich auf den Button klicke wird der Inhalt des JTextField auf der Console ausgegeben. Das Problem ist, dass es immer leer ist egal was im Feld steht.
Das komische ist, in einer anderen Klasse funktioniert es genau so. Suche schon ne Zeit lang nach dem Fehler, viell. hab ich was übersehn. Kann mir jmd helfen?

MfG

 

[adr1an]

Ich vermute dein fehler liegt in der Methode

getLang110TextField()

Im dritten Codeschnipsel rufst du die Methode nochmals auf, welche dir ein neues Textfeld macht.
Wenn du es jedesmal neu initialisierst, ist der Inhalt immer null.
Probier mal einen String vor dem Return auszugeben.
Gruss adr1an

 

[SlaterB]

nana, da steht doch
if(Lang110 == null) {
in getLang110TextField(),initialisiert wird also nur einmal

was ist überhaupt das Problem? dass das TextFeld immer leer ist? dazu kann man am Code wenig sehen,

die Überschrift deutet auf einen ==-Vergleich hin, Strings immer mit equals vergleichen!

 

[jemandzehage]

Hallo,
also der Code sieht gut aus. Ich hab den jetzt mal bei mir eingebaut und bei modifizierungen vorgenommen und so läuft er auf jeden Fall. Alle Codeschnipsel die du gezeigt hast scheinen alle Fehlerfrei zu sein.

Hier mal der Code so wie ich ihn getestet habe:

import java.awt.BorderLayout;
import java.awt.Container;
import java.awt.Dimension;
import java.awt.Font;
import java.awt.event.ActionEvent;
import java.awt.event.ActionListener;

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


public class AusgabeTest extends JFrame{
	private static final long serialVersionUID = -5323202415726307786L;
	private JTextField lang110 = null;
	private JButton add = null;
	
	private int componentWidth = 100;
	private int componentHeight = 25;
	
	public AusgabeTest() {
		setVisible(true);
		setDefaultCloseOperation(EXIT_ON_CLOSE);
		
		setFrameContents();
		
		pack();
		repaint();
		requestFocus();
	}

	public JTextField getLang110TextField() {
        if(lang110 == null) {
            lang110 = new JTextField();
			lang110.setPreferredSize(new Dimension(componentWidth, componentHeight));
            lang110.setFont(new Font("italic", Font.BOLD, 16));
        }
        return lang110;
    }
	
	public JButton getAddButton() {
       if(add == null) {
			add = new JButton("Hinzufügen");
			add.addActionListener(new ActionListener() {
				@Override
				public void actionPerformed(ActionEvent e) {
					System.out.println(getLang110TextField().getText());
				}
			});
			add.setPreferredSize(new Dimension(componentWidth, componentHeight));
            
		}
       
       return add;
	}
	
	public void setFrameContents() {
        Container contentPane = getContentPane();
        contentPane.setLayout(new BorderLayout());   
        contentPane.add(getLang110TextField(), BorderLayout.NORTH);
        contentPane.add(getAddButton(), BorderLayout.SOUTH);
	}
	
	public static void main(String[] args) {
		new AusgabeTest();
	}
}

Es kann auch sein, dass der Fehler irgendwo in den Teilen ist, die du ausgelassen hast.

 

[OasisCritter]

Wenn ich im ActionListener direkt sage:

  System.out.println(Lang110.getText());

dann bekomm ich eine NullPointer, also ist die variable immernoch, aber wie kann das sein, wenn sie doch auf dem Frame angezeigt ist?

...

 

[jemandzehage]

Könntest du mir das Problem noch ein bisschen detaillierter Erklären? Was genau funktioniert denn nicht und funktioniert der Code den ich geschrieben habe bei dir?? Und was passiert wenn du meinen Code bei deinem Programm anwendest?

 

[OasisCritter]

Ja, dein code funktioniert.
Nagut ich fang ganz vorn an. Die Oberklasse meines Frames ist so definiert:

package vok.ui;

import java.awt.HeadlessException;

import javax.swing.JFrame;

import vok.control.controller;

public abstract class AddFrame extends JFrame {

	/**
	 * 
	 */
	private static final long serialVersionUID = 1L;
	private controller control =null;
	private AddFrame Window = this;

	
	/*Setters & Getters =============================================*/

	public controller getControl() {
		return control;
	}

	public void setControl(controller control) {
		this.control = control;
	}

	public AddFrame getWindow() {
		return Window;
	}

	public void setWindow(AddFrame window) {
		Window = window;
	}

	/**
	 * creates a new instance of this class
	 * @param control
	 * @throws HeadlessException
	 */
	public AddFrame(controller control) throws HeadlessException {
		this.control=control;
		initialize();
		setFrameContents();
	}

	/**
	 * 
	 */
	public abstract void initialize();

	/**
	 * 
	 */
	public abstract void setFrameContents();
	

}

Mein Frame erbt also von dieser Klasse und implementiert

initialize()

und

setFrameContents()

.
Mein Problem ist das dass JTextField immer leer ist, egal was ich reinschreibe und zu dem Zeitpunkt andem der ActionListerner des AddButtons versucht darafu zuzugreifen = null ist.
Kann es daran liegen, dass setFrameContents in der Oberklasse implementiert sind, weil in einer anderen Klasse die ein Panel darstellt funktioniert es einwandfrei, aber die letztgenannte Klasse implementiert eben nicht diese Oberklasse.

Anders kann ich es mir nicht erklären, bzw selbst wenn dem so ist könnt ich es mir nicht erklären.

Ach ja es geht um folgendes Programm:
http://www.java-forum.org/codeschnipsel-u-projekte/100323-vokabeltrainer.html

Beim Panel zum hinzufügen von Vokabeln kann ich sehr gut den Inhalt der JTextField auslesen. Jetzt arbeite ich an einem Frame zu erweiterten bearbeiten und hinzufügen von Vokabeln.

MfG

 

[jemandzehage]

Ähm, also das könnte nur daran liegen, dass du im Konstruktor der Klasse die von AddFrame erbt nicht super(controller); aufrufst.

(schreibst du deine Klassen eigentlich immer Wahlweise mal groß und mal klein?? Weil der übersicht halber würde ich Klassen immer Groß anfangen und Objekte immer klein)

 

[OasisCritter]

Doch Java hat eine starke statische Typprüfung, ohne den Aufruf super(controller); im Konstruktor des erben, lässt sich der Code nicht compilieren. Meine Klassen werden immer Groß am Anfang geschrieben.

...

 

[jemandzehage]

super(controller) muss in der ersten Zeile des Konstruktors stehen.

(hab mich nur gewundert wegen Zeile 15 in deinem letzen Code den du geschrieben hast. da ist controller auch klein)

 

[Gast2]

im Konstruktor des erben, lässt sich der Code nicht compilieren. Meine Klassen werden immer Groß am Anfang geschrieben.

...

Nicht nur deine Klassen leider!!!--> Variablen werden klein geschrieben...
Mach doch einfach ein KSKB dann sieht man am besten was du falsch machst

 

[OasisCritter]

Naja es funktioniert jetzt. Es lag an der Oberklasse, also "AddFrame". Wieso und Weshalb kann ich mir nicht erklären, muss ein schwerer Fehler von Sun sein. Jetzt benutze ich einfach JFrame als Oberklasse.


...

 

[Gast2]

muss ein schwerer Fehler von Sun sein. Jetzt benutze ich einfach JFrame

Das bezweifel ich mal ganz stark, weil du schreibst Klassen klein, dann mal wieder groß. Das lässt darauf schließen, dass deine Erfahrung nicht so groß ist, dass so eine These aufstellen kannst, vor allem ohne Beweis.