Threads Swing

[chnooorzi]

Hallo

Ich habe ein kleines oder auch grosses Problem.
Für die Schule muss ich ein Programm entwickeln, welches auf einem Gridlayout
Roboter (Threads) durch ein GridLayout wandern lassen muss.
Die Roboter werden durch einen Punkt dargestellt.
Wenn ich das Ganze in der Konsole ausgebe funktioniert es einwandfrei.
Sobald ich das Ganze jedoch in einem JFrame anzeigen möchte, funktioniert es
manchmal, meistens aber nicht.

Ich vermute, dass ich irgendwo etwas mit SwingUtilities einbauen muss, weiss aber nicht
wo... Oder habe ich irgendwo anders einen Fehler?
Das Programm gibt automatisch für jeden Thread jeweils den Output auch in der Console aus
(zum Vergleichen).

Im Anhang habe ich eine etwas abgespeckte Version gezippt (ohne Writer und FileReader).

Besten Dank für die Hilfe

Gruss chnooorzi

 

[Gucky]

Swing hat etwas eingebaut, dass es sich nicht aus einem anderen Threads verändern lässt (klappt nicht immer).
Deshalb musst du die SwingUtilities immer dann einbauen, wenn du schreibend auf einen SwingComponent zugreifen willst.

SwingUtilities.invokeLater(new Runnable () {//Aus einem Thread heraus darf die GUI nicht verändert werden. Daher muss invokeLater aufgerufen werden, um die Befehle an den Thread der GUI weiterzugeben
	public void run(){
		doStuff();
	}
});

Damit müsste es eigentlich funktionieren. Zumindest hatte ich mal das Problem, dass zwar keine Fehlermeldung ausgegeben wurde, aber einfach nichts passiert ist.

 

[chnooorzi]

Hallo

Danke für die rasche Antwort.
MIr ist leider noch ein wenig unklar wo ich das einbauen muss...

Ich habe die folgenden Klassen:
- RobotGameViewer
- RobotGameFrame extends JFrame
- GameField
- RobotRunnable: extends Runnable
- Field: extends JPanel
- CircleComponent: extends JComponent

Zu Beginn erstelle ich ein JFrame mit
JFrame frame = new RobotGameFrame(new Gamefield());

Das Gamefield ist ein Array welches aus mehreren Feldern (JPanel) besteht
Im Frame wird dieses Array dann in ein GridLayout geladen, in dem die einzelnen
Felder, welche ja JPanel sind, der Reihe nach eingelesen werden.

Der Roboter ändert eigentlich nur den Status des Feldes von locked (add CircleComponent) auf unlocked (removeAll()) und unbekehrt.

Das heisst ich muss in der createPanel(); Methode bereits SwingUtilities einbauen und zudem
auch noch in der Methode in der die Felder den Status ändern. Habe ich das richtig Verstanden?

Danke

Gruss chnooorzi

 

[sirkarpfen]

Im Prinzip musst du invokeLater() nur dort verwenden wo du die RoboterThreads erstellst. Das hier is zB. relativer Unsinn, weil das Erstellen eines Frames nicht wirklich was mit einem seperaten Thread zu tun hat, der auf die GUI zugreift:

SwingUtilities.invokeLater(new Runnable() {
    public void run() {
        final JFrame frame = new RobotGameFrame(gamefield);
        frame.setTitle("Robot Game");
        frame.setDefaultCloseOperation(JFrame.EXIT_ON_CLOSE);
        frame.pack();
        frame.setVisible(true);
    }});

Hier wäre die einzig sinnvolle Verwendung von invokeLater():

if(!gamefield.getSingleField(column, row).isLocked()) {
    Runnable robotRunnable = new RobotRunnable(gamefield, column, row, new  Color(red,green,blue), DEFAULT_DELAY);
    SwingUtilities.invokeLater(robotRunnable);
    robots.add(new Thread(robotRunnable));
}

Nochmal zur Erklärung, invokeLater() dient nur dazu anderen Threads zu "erlauben" die GUI zu verändern .

 

[Gucky]

Nochmal zur Erklärung, invokeLater() dient nur dazu anderen Threads zu "erlauben" die GUI zu verändern .

Das ist, finde ich, ein bisschen irreführend, da die Befehle nicht mehr im Thread ausgeführt werden, der sie ohne invokeLater aufgerufen hätte, sondern im GUI Thread. Sehr rechenintensive Dinge blockieren dann trotzdem das GUI.

Ich tue das invokeLater immer dort hin, wo auch das GUI verändert wird.

 

[chnooorzi]

Hallo

Danke für die Antworten.

Im Prinzip musst du invokeLater() nur dort verwenden wo du die RoboterThreads erstellst. Das hier is zB. relativer Unsinn, weil das Erstellen eines Frames nicht wirklich was mit einem seperaten Thread zu tun hat, der auf die GUI zugreift:

SwingUtilities.invokeLater(new Runnable() {
    public void run() {
        final JFrame frame = new RobotGameFrame(gamefield);
        frame.setTitle("Robot Game");
        frame.setDefaultCloseOperation(JFrame.EXIT_ON_CLOSE);
        frame.pack();
        frame.setVisible(true);
    }});

Habe ich so von Oracle übernommen... Hatte das bisher auch nie so gemacht
Wenn ich das so ergänze wie Du es mir geraten hast, funktioniert es nicht. Das Frame bleibt
leer und es wird nur noch ein Roboter bewegt...
Auch wenn ich das oben erwähnte nicht in SwingUtilities packe.

@Gucky Ich habe das invokeLater in der Field-Classe in der Methode changeState hinzugefügt...
DOch leider funktioniert es auch so nicht

Ich habe jedoch bemerkt, dass es unter Windows ohne Probleme läuft. Unter Mac OSX Mavericks habe ich das Problem ab 6 Threads...

 

[Gucky]

ich weiß nicht, was changeState macht. Eu müsstest das invokeLater dort machen, wo direkt auf der Swing Componente etwas gemacht wird oder in übergeordneten Methoden.

Versuch mal die andere Möglichkeit. Eigentlich hat es auch mit meiner zu funktionieren aber die Andere ist natürlich leichter.

 

[chnooorzi]

Hallo

Die Methode changeLockedState gehört zur Klasse Field (extends JPanel).
Und macht das folgende:

	public void changeLockedState(final Color color) {
		locked = !locked;

			SwingUtilities.invokeLater(new Runnable() {
				public void run() {
					
					if(locked) add(new CircleComponent(getWidth() / 2, color));
					else removeAll();
					
					revalidate();
					repaint();
				}});		
	}

Sie fügt also zum JPanel einen Circle hinzu oder entfernt ihn.
Wenn ich das Ganze wie oben mache, funktioniert es immer noch einwandfrei unter Windows.
Jedoch unter OSX habe ich immer noch das selbe Problem -.-

Gruss chnooorzi

 

[sirkarpfen]

Das ist, finde ich, ein bisschen irreführend, da die Befehle nicht mehr im Thread ausgeführt werden, der sie ohne invokeLater aufgerufen hätte, sondern im GUI Thread. Sehr rechenintensive Dinge blockieren dann trotzdem das GUI.

Ich tue das invokeLater immer dort hin, wo auch das GUI verändert wird.

Jo du hast recht, das war wirklich schlecht formuliert. Manchmal versucht man etwas verständlich zu machen und dann kommts doch irreführender raus als man will .

Wenn ich das Ganze wie oben mache, funktioniert es immer noch einwandfrei unter Windows.
Jedoch unter OSX habe ich immer noch das selbe Problem -.-

Gruss chnooorzi

Was genau funktioniert denn nicht? Gibts Fehlermeldungen oder tuts nicht das was es soll?

 

[Gucky]

@sirkarpfen
Das Problem kenne ich gut.

@TO
Poste mal bitte, wie sirkarpfen schon angedeutet hat, eine vollständige Fehlerbeschreibung. Wird das Ganze überhaupt aufgerufen? Unterstützt das Mac OS X vielleicht kein derartiges Multithreading? Wird vielleicht eine Exception geworfen?

 

[chnooorzi]

Hallo

Es macht nicht das was es soll...
Eigentlich sollten zu jederzeit 10 Punkte (robots) von feld zu feld springen.
Unter Windows funktioniert das auch. Unter Mac sehe ich zum Teil nicht 10
Punkte und zum Teil sehe ich ein paar Punkte doppelt (diese haben eine
unterschiedliche Farbe).

Die einzelnen Felder sehe ich (die haben einen Border), jedoch werden diese nicht
sortiert. Wenn ich das Ganze in der Konsole ausgebe, funktioniert es.

Gruss chnooorzi

 

[chnooorzi]

Hallo

Funktioniert jetzt...
Meine Änderungen waren:
- SwingUtilites in der changeState.Methode
- Gamefield extends JPanel
- Und noch ein gamefield.revalidate und gamefield.repaint in der move-methode.

Danke für eure Hilfe!!!