Statusleiste mit JLabel aktualisieren

[Optimum]

Hallo Community,

habe ein Problem mit einem GUI. Ich will unten eine Statusleiste einfügen, die ein Label enthält, das mir den Status über die aktuell durchgeführten Berechnungen gibt. Leider wird die Textaänderung nicht angezeigt. Habe dazu mal ein Minimalbeispiel gebastelt:

import javax.swing.*;

public class StatusbarTest extends JFrame {
	
	private static final long serialVersionUID = 1L;
	
	public StatusbarTest() {
		setDefaultCloseOperation(javax.swing.WindowConstants.EXIT_ON_CLOSE);
		status = new JLabel(" ");
		ok = new JButton("OK");
		ok.addActionListener(new java.awt.event.ActionListener() {
			public void actionPerformed(java.awt.event.ActionEvent evt) {
				okDo(evt);
			}
		});
		ok.setEnabled(true);
		
		getContentPane().add(ok, java.awt.BorderLayout.CENTER);
		getContentPane().add(status, java.awt.BorderLayout.SOUTH);
		pack();
	}
	
	private void okDo(java.awt.event.ActionEvent evt) {
		
		status.setText("Working...");
		doSomething();
		status.setText("Ready!");
		
	}
	
	private void doSomething() {
		try {
			Thread.sleep(3000);
		} catch (InterruptedException e) {
			e.printStackTrace();
		}
	}
	
	public static void main(String Args[]) {
		java.awt.EventQueue.invokeLater(new Runnable() {
			public void run() {
				new StatusbarTest().setVisible(true);
			}
		});
	}

	private JLabel status;
	private JButton ok;
}

Wenn man nun "OK" klickt, dann soll eigentilch "Working..." dastehen, nach Abschluss dann "Ready!" Die Berechnungen werden durch einen Wait-Thread simuliert. Ich habe schon einige Foren dazu durchsucht, aber immer nur gefunden, dass das GUI erst nach Abschluss der Threads neu gezeichnet wird. Wie kann ich eine Neuzeichnung erzwingen?

repaint()

und

validate()

, wie in einigen Threads vorgeschlagen, funktionieren nicht.

 

[Gucky]

Du musst die Berechnung in einen externen Thread auslagern. Dann müsste es hinhauen. Ich weiß zwar nicht, warum das hier nicht funktioniert aber dann müsste es hinhauen. Aber es kann sein, dass du

java.awt.EventQueue.invokeLater(new Runnable() {
     public void run() {
           setzeText();
      }
});

setzen musst, wenn du den Text aus dem anderen Thread ändern willst.

 

[Optimum]

Also ich habe da schon verschiedene Möglichkeiten probiert, die einzelnen Anweisungen auf verschiedene Weise in Threads zu packen, aber das lief dann entweder darauf hinaus, dass zuerst das GUI fertig berechnet wurde und anschließend erst die eigentliche Berechnung gemacht wurde oder wenn ich die mit [c]invokeLater()[/c] verwendet habe, dass dann die GUI-Änderungen in der Queue erst nach der eigentlichen Berechnung eingereiht wurden und somit auch nicht angezeigt wurden.

Allerdings ergibt sich mit Threads das Problem, dass ich lokale Variablen habe, die in die Methoden übergeben werden und die müsste ich dann alle erst finalisieren und das scheint mir auch nicht die eleganteste Lösung zu sein.

 

[Gucky]

Es ginge mit einer privaten Klasse, die den Thread enthält. Oder wrd die Berechnung und das Setzen des Textes schon in einem anderen Thread durchgeführt? Debug doch mal und guck dir die Belegungen der Variablen an. Vielleicht hilft das schon.

 

[Flown]

Du kannst es lösen in dem du deine "Berechnungen" auslagerst und in einem eigenen Thread laufen lässt, da sonst die GUI blockiert und somit auch der SwingWorker.

Habs jetzt auf die schnelle geschrieben und glaube das es funktioniert.

Spoiler: Code

import javax.swing.*;

public class StatusbarTest extends JFrame {

	private static final long serialVersionUID = 1L;
	private JLabel status;
	private JButton ok;

	public StatusbarTest() {
		setDefaultCloseOperation(javax.swing.WindowConstants.EXIT_ON_CLOSE);
		status = new JLabel(" ");
		ok = new JButton("OK");
		ok.addActionListener(new java.awt.event.ActionListener() {
			public void actionPerformed(java.awt.event.ActionEvent evt) {
				okDo(evt);
			}
		});
		ok.setEnabled(true);

		getContentPane().add(ok, java.awt.BorderLayout.CENTER);
		getContentPane().add(status, java.awt.BorderLayout.SOUTH);
		pack();
	}

	private void okDo(java.awt.event.ActionEvent evt) {
		new Thread(new Calculations(this)).start();
	}
	
	public void setStatus(String statusText) {
		status.setText(statusText);
	}

	public static void main(String Args[]) {
		java.awt.EventQueue.invokeLater(new Runnable() {
			public void run() {
				new StatusbarTest().setVisible(true);
			}
		});
	}
	
	
	class Calculations implements Runnable {
		
		private StatusbarTest frame;
		
		public Calculations(StatusbarTest frame) {
			this.frame = frame;
		}
		@Override
		public void run() {
			doSomething();
		}
		
		private void doSomething() {
			try {
				frame.setStatus("Working...");
				Thread.sleep(3000);
				frame.setStatus("Finished");
			} catch (InterruptedException e) {
				e.printStackTrace();
			}
		}
	}
}

 

[Optimum]

Vielen Dank, die Lösung sieht sehr praktikabel aus, vor allem bekommt das ganze dadurch auch Struktur, indem man den Controller vom View trennt, gefällt mir sehr gut. Werde ich gleich mal in mein GUI einbauen und ausprobieren.