JLabel-Text wird nicht angezeigt

[RoHo]

Hallo,

ich möchte, wenn ich auf ein JPanel klicke, dass in einem JLabel der Name des Panels angezeigt wird (ich hab mehrere Panels) und zwar über:

/**
	 * Erneuert die angezeigten Daten
	 */
	public void updateInfos(){
		System.out.printf("Update Controllpanel:\n");
		
		//Hole dir das aktuelle Feld
		this.markedField = playground.getMarked();
		
		System.out.printf("Marked: %s\n", this.markedField);
		

		
		//Aktualiesiere das mainPanel
		if (markedField != null){
			
			//Aktualiesire die Position
			this.l_pos_value.setText(markedField.toString());
			//TODO hier will er nicht mehr
			
			System.out.printf("in %.5s soll %s und ist %s\n", l_pos_value, markedField.toString(), l_pos_value.getText());
			
		}
		//Feld ist null -> nichts ausgewählt/initialisiert
		else{

		}

		System.out.printf("in %.5s soll %s und ist %s\n", l_pos_value, markedField.toString(), l_pos_value.getText());
		System.out.println("...Ende Update-Controlpanel");
		
	}

das wird über ein MouseListener aufgerufen. Soweit so gut, denn in meinen Kontrollausgaben steht auch

in javax soll (1/1) und ist (1/1)

nur wirds im JLabel nicht angezeigt, da bleibt weiterhin der Standartwert ("N/A") drin.
Der Text steht ja drin, warum wird er nicht angezeigt?
Mit repaint(), validate(), revalidate() hab ich schon wild rumexperimentiert.
Eine andere Methode von mir wird nicht aufgerufenzwischendurch (durch Kontrollausgaben überprüft)

Gruß
Ro

 

[SlaterB]

Fehlermöglichkeiten:
1. l_pos_value ist innerhalb dieser Klasse nicht in der GUI korrekt eingefügt, entweder gar kein add()
oder es wurde eine lokale gleichbenannte Variable eingefügt

oder

2. die ganze Klasse ist in sich korrekt, allerdings wurden an höheren Stellen zwei Objekte dieser Klasse erstellt,
eines ist korrekt in die GUI eingebunden, das andere nicht aber als Listener registriert,
durch System.out.println("test"); im Konstruktor zu erkennen

nur erkennbar an mehr Code, idealerweise (kurzes) komplettes Programm

 

[diggaa1984]

ich gehe mal einfach davon aus, dass die Ausgabe von der unteren Anweisung kommt, und nicht von der, wo kurz davor das label gesetzt wird .. diese Ausgaben sind identisch, aendere sie ein wenig ab und finde damit heraus ob er überhaupt in den if-Block springt

 

[RoHo]

@digga: ausgaben kurz geändert, ist aber in das if gegangen (gab zwei ausgaben und keine null-pointerexception)

@slater zu1: nee, ich hab sogar ein

this

vor dem variablennamen verwendet

ich hab versucht das programm so weit wie möglich zu kürzen, sodass nur noch relevante teile (also die auch aufgerufen werden) drin sind

/**
 * Zeigt (rechts) am Rand Informationen zu dem aktuellen Feld an,
 * bzw. kann das Feld verwalten, das ausgewählt wurde
 * @author Robin
 *
 */
public class ControllPanel extends JPanel {
	//Wozu gehörts?
	private Ground playground = null;
	//Welches ist das aktive Feld?
	private Field markedField = null;
	
	//Das obere Panel mit den STandartinfos/einstellungen
	private JPanel mainPanel = null;
	private JLabel l_header = null;		//Label zur Überrschrift
	private JLabel l_pos_text = null;	//Label zur Positionsanzeige-Ankündigung
	private JLabel l_pos_value = null;	//Label zur Positionsanzeige-Wert
	private JLabel l_type_text = null;	//Label zur Anzige des Auswahlmenüs
	private JComboBox cb_tye = null;	//Combobox um den Typen eines Feldes zu ändern
	
	

	public ControllPanel(Ground g) {
		//Lade das Feld
		this.playground = g;
		//ERstelle das Grundgerüst der Panels
		initPanels();

	}
	
	/**
	 * erzeugt die Panels mit zugehörigen komponenten
	 */
	private void initPanels(){
		this.setLayout(new BorderLayout());
		//init das MainPanel
		initMainPanel();
		this.add(this.mainPanel, BorderLayout.NORTH);

		
	}
	
	/**
	 * Erneuert die angezeigten Daten
	 */
	public void updateInfos(){
		System.out.printf("Update Controllpanel:\n");
		
		//Hole dir das aktuelle Feld
		this.markedField = playground.getMarked();
		
		System.out.printf("Marked: %s\n", this.markedField);
		

		
		//Aktualiesiere das mainPanel
		if (markedField != null){
			
			//Aktualiesire die Position
			this.l_pos_value.setText(markedField.toString());
			//TODO hier will er nicht mehr
			
			System.out.printf("(IF) in %.5s soll %s und ist %s\n", l_pos_value, markedField.toString(), l_pos_value.getText());
			
		}
		//Feld ist null -> nichts ausgewählt/initialisiert
		else{
		}
		
		System.out.printf("(N_IF) in %.5s soll %s und ist %s\n", l_pos_value, markedField.toString(), l_pos_value.getText());
		System.out.println("...Ende Update-Controlpanel");
		
	}
	
	private void initMainPanel (){
		//Grundeinstellungen fürs Panel
		mainPanel = new JPanel();
		mainPanel.setLayout(new BorderLayout());
		
		//Erzeuge die Components, die da drauf müssen
		this.l_header = new JLabel("<html><b>Field</b></html>");
		this.l_pos_text = new JLabel("Col./Row. :");
		this.l_type_text = new JLabel("Type: ");
		this.l_pos_value = new JLabel("N/A");
		this.cb_tye = new JComboBox();
		
		//Mittleres Panel um die Beschriftung zu übernehmen
		JPanel  midMainPanel = new JPanel();
		midMainPanel.setLayout(new GridLayout(2, 0));
		midMainPanel.add(l_pos_text);
		midMainPanel.add(l_pos_value);
		midMainPanel.add(l_type_text);
		midMainPanel.add(cb_tye);
		
		//und alles aufs mainpanel
		mainPanel.add(l_header, BorderLayout.NORTH);
		mainPanel.add(midMainPanel, BorderLayout.CENTER);
		
		//Initioalisere die Combobox
		cb_tye.addItem("Default");
		cb_tye.addItem("Free");
		cb_tye.addItem("Predefining");
		
	
	}

und der Aufruf über

	/**
	 * markiert das Feld 
	 * 
	 */
	@Override
	public void mouseClicked(MouseEvent e) {
		//Setze dieses Feld als marikeirt
		this.playground.markField(this);
		

		
	}

Gruß

 

[SlaterB]

Ground ist die fragliche Klasse, playground ist die Variable? letztere wird nirgendwo in die GUI eingefügt,
dann doch lieber zuviel als zuwenig

 

[RoHo]

doch doch, das läuft dann aber über ein eigenes JFrame,
aber ich hab meinen Fehler gefunden (schäm)
die verbindung zwischen dem Kontrollfeld und dem Ground war nicht gesetzt,
allerdings hat das ändern des markierten Feldes über das Kontrollpanel funktioniert... (hatte ich auskommentiert, als ich den Fehler hier gesucht hatte), seltsam...
naja sry irgendwie war ich da wohl geistig abwesend, danke euch
Gruß Ro