Swing Gui Update Problem (EDT)

[hdi]

Hey, ich habe zwei Methoden in meinem JPanel, sie löschen jeweils Componenten und adden neue. Hier sind sie:

/**
	 * Setzt den Quellcode, der angezeigt wird.
	 * 
	 * @param quellcode
	 *            der neue Quellcode
	 */
	public void setInhalt(final String quellcode) {
		textArea.setText(quellcode);
		if (quellcode == null) {
			this.clear();
		} else {
			final JComponent th1s = this;
			EventQueue.invokeLater(new Runnable() {
				@Override
				public void run() {
					th1s.removeAll();
					setLayout(new BorderLayout());
					th1s.add(new JScrollPane(textArea), BorderLayout.CENTER);
					th1s.validate();
				}
			});
		}
	}

	/**
	 * Löscht den Quellcode und zeigt die Drag & Drop Information an
	 */
	public void clear() {

		final JComponent th1s = this;
		EventQueue.invokeLater(new Runnable() {
			@Override
			public void run() {
				th1s.removeAll();
				setLayout(new FlowLayout(FlowLayout.CENTER));
				th1s.add(info);
				th1s.validate();
			}
		});
	}

...die setInhalt() Methode funktioniert wunderbar. Die GUI macht immer schön ein Update. die clear() Methode allerdings kriegt das nicht hin. Hier ist es so, dass sich das Panel nicht verändert, erst wenn ich es resize...

Beide Methoden sind doch gleich aufgebaut, ich rufe schön brav ein validate() auf, laut API sollte es das dann auch gewesen sein. Und beide werden vom EDT verarbeitet.
Habt ihr irgendeine Idee wo man hier noch nach einem Fehler/Unterschied suchen könnte?

Wie kann ich zB herausfinden welcher Thread gerade diese Methoden aufruft? Eigentlich müsste es bei beiden der gleiche sein, aber wer weiss...

Bin um jede Hilfe froh, ich kämpf hier seit einer STunde, die eine Methode klappt immer perfekt, die andere klappt nie.

danke

 

[Schandro]

versuchs mal mit revalidate() statt validate()

ansonsten:
Du kannst auch innerhalb einer inneren Klasse das this auf die äußere Klasse beziehen lassen:

KlassennameDerÄußerenKlasse.this
statt
this

 

[hdi]

danke für den Tipp mit this.

revalidate() hatte ich auch schon versucht, ohne Ergebnis.

Ich würde nochmal auf meine Frage zurückkommen, wie ich prüfen kann welcher Thread die Methoden betritt. Vllt ist es wirklich ein verschiedener bei beiden Methoden. Ansonsten ist der code der Methoden ja nämlich eigentlich der gleiche. Wieso also sollte die eine immer gehen, die andere niemals :bahnhof:

 

[SlaterB]

Thread.getCurrentThread()

bau doch mal ein lauffähiges Beispiel zusammen

 

[hdi]

Also ich fände es nett wenn ihr das mit mir zusammen rekonstruiert. Ich geh das ganze jetzt mit euch Step by Step durch, vllt versteht ihr dann wieso es hakt, weil ich verstehe es nicht.

Also das ganze wird ausgelöst durch Klick auf einen Button mit folgendem ActionListener:

class ClearAction implements ActionListener {

		@Override
		public void actionPerformed(ActionEvent e) {
			new Thread() {
				@Override
				public void run() {
					parent.loescheWorkspace();
				}
			}.start();
		}
	}

Der EDT kommt also rein, startet schnell einen neuen Thread und geht wieder raus, ist also wieder ready.

die Methode ist nur ein kleiner Wrapper indem nur diese hier aufgerufen wird:

@Override
	public void loescheModelDaten() {
		model.setQuellcode(null);
		model.update(); 
	}

model.update() macht ein setChanged und notifyObservers auf den Observern. Der einzige Observer ist mein Haupt-Frame, d.h. seine update-Methode wird aufgerufen. Die update-Methode in meinem Frame sieht so aus:

@Override
	public void update(Observable obs, Object obj) {
		ModelImpl model = (ModelImpl) obs;
		editor.clear();
	}

letztendlich die clear-Methode:

	public void clear() {
		EventQueue.invokeLater(new Runnable() {
			@Override
			public void run() {
				EditorPanel.this.removeAll();
				EditorPanel.this.setLayout(new FlowLayout(FlowLayout.CENTER));
				EditorPanel.this.add(info);
				EditorPanel.this.validate();
			}
		});
	}

So, und hier ist genau der Moment, wo er nicht das tut, was im Code steht.
Er macht zwar ein repaint nach removeAll(), also der Editor wird leer, aber er zeigt
nicht das "info" an, was ja geaddet wurde.
Erst, wenn ich das Editor-Fenster resize.

Warum macht der EDT hier nur halbe Arbeit? Es ist immer das selbe, also scheinbar deterministisches Verhalten. Aber warum ?

Bin echt superfroh über Hilfe, dieses EDT-Thema ist für mich noch immer ein Rätsel, weil es mich jedesmal auf's neue überrascht, wenn ich dachte es verstanden zu haben

 

[SlaterB]

warum es nötig ist weiß ich grad auch nicht und hab bisschen probiert, aber nichts wirklich inhaltlich interessantes gefunden,
jedenfalls brauchst du einen repaint()-Aufruf

public class TestGUI extends JFrame {


	public TestGUI() {

		final EditorPanel editor = new EditorPanel();
		editor.setInhalt("text");
		editor.setBackground(Color.YELLOW);
		add(editor);

		Model model = new Model(editor);
		Parent parent = new Parent(model);
		ClearAction clearAction = new ClearAction(parent);

		JButton clearButton = new JButton("Clear");
		clearButton.addActionListener(clearAction);

		add(clearButton, BorderLayout.SOUTH);

		JButton setButton = new JButton("Set");
		setButton.addActionListener(new ActionListener() {

			@Override
			public void actionPerformed(ActionEvent e) {
				editor.setInhalt("neuer Inhalt " + Math.random());
			}

		});

		add(setButton, BorderLayout.NORTH);

		setSize(400, 300);
		setDefaultCloseOperation(JFrame.EXIT_ON_CLOSE);
		setVisible(true);

	}

	public static void main(String[] args) {
		new TestGUI();
	}

}

class ClearAction implements ActionListener {
	Parent parent;

	public ClearAction(Parent parent) {
		this.parent = parent;
	}

	@Override
	public void actionPerformed(ActionEvent e) {
		new Thread() {
			@Override
			public void run() {
				parent.loescheWorkspace();
			}
		}.start();
	}
}

class Parent {
	Model model;

	public Parent(Model model) {
		this.model = model;
	}

	public void loescheModelDaten() {
		model.setQuellcode(null);
		model.update();
	}

	public void loescheWorkspace() {
		loescheModelDaten();
	}

}

class Model {

	EditorPanel editor;

	public Model(EditorPanel editor) {
		this.editor = editor;
	}

	public void update(Observable obs, Object obj) {
		// ModelImpl model = (ModelImpl) obs;
		editor.clear();
	}

	public void setQuellcode(Object object) {

	}

	public void update() {
		update(null, null);
	}
}

class EditorPanel extends JPanel {
	JPanel info;
	JTextArea textArea = new JTextArea();

	public EditorPanel() {
		setLayout(new BorderLayout());

		info = new JPanel();
		info.add(new JLabel("ich bin info"));
		info.add(new JLabel("ich bin info 2"));
		
	}

	public void clear() {
		EventQueue.invokeLater(new Runnable() {
			@Override
			public void run() {
				EditorPanel.this.removeAll();
				EditorPanel.this.setLayout(new FlowLayout(FlowLayout.CENTER));
				EditorPanel.this.add(info);
				EditorPanel.this.validate();
				EditorPanel.this.repaint();
			}
		});

	}

	public void setInhalt(final String quellcode) {
		textArea.setText(quellcode);
        if (quellcode == null) {
            this.clear();
        } else {
            final JComponent th1s = this;
            EventQueue.invokeLater(new Runnable() {
                @Override
                public void run() {
                    th1s.removeAll();
                    setLayout(new BorderLayout());
                    th1s.add(new JScrollPane(textArea), BorderLayout.CENTER);
                    th1s.validate();
                    th1s.repaint();
                }
            });
        }

	}

}

 

[hdi]

Tausend Dank!