Frage zu Threads

[mrbela]

Hey Leute!

Ich bin gerade dabei Tetris zu programmieren. Jetzt stoße ich aber auf eine Designfrage bzgl Threads:

Ich benutze das MVC:

- Als Model benutze ich ein Array, das das Spielfeld darstellt.
- Im View habe ich ein JPanel eingebettet, das mit Hilfe der paint-Methode basierend auf dem Array des Models das Spielfeld malt. Klappt alles soweit.
- Der Controller initialisiert den View und das Model, erbt von Thread. Hier habe ich die run-Methode eingebettet, die entsprechend immer das Model bearbeitet, damit das Spiel ist wie tetris (Figur drehen, senken etc.)

Dann habe ich noch eine initiale Klasse, die ich Tetris genannt habe. Die sieht so aus:

package init;

import javax.swing.UIManager;
import javax.swing.UnsupportedLookAndFeelException;

import controller.TetrisControler;
import view.View2;

public class Tetris {
	
	Thread aktSpiel;
	
	public Tetris() throws InterruptedException {
						
		aktSpiel = new TetrisControler(this);
		aktSpiel.start();
	}
	
	public void resetGame() {
		System.out.println("------->"+ aktSpiel.isAlive());
		
		aktSpiel = new TetrisControler(this);
		aktSpiel.start();
	}
	
	public static void main(String[] args) throws InterruptedException, ClassNotFoundException, InstantiationException, IllegalAccessException, UnsupportedLookAndFeelException {
		UIManager.setLookAndFeel(
	            UIManager.getSystemLookAndFeelClassName());
		new Tetris();
	}
}

Nun zu meiner Frage: Rufe ich vom View aus die resetGame() von Tetris auf, so wird ein ganz neues Fenster geöffnet, da ja mein Controller-Thread ein neues Fenster intialisiert.

Wie muss ich das ganze denn umdesignen, dass es Sinn macht?

Ich dachte etwa daran, dass ich in der Klasse Tetris den View initialiseire (der läuft dann ja im main-Thread, nicht im Controller-Thread) und einen eigenen Controller-Thread. Wenn ich das tue, dann "hört" das JFrame (z.B. bei repaint()) aber nicht mehr auf mein Controller, also wenn der sich zB ändert oder? Da ja der View im Main-Thread und nicht mehr im Controller-Thread läuft.

Ich bin jetzt am überlegen, wei ich das umdesigne... Bisher habe ich das reset() im Controller gemacht. DAs ist glaube ich aber kein schöner Stil, da der Thread immer weiterläuft.

Ich hoffe, ihr habt verstanden, um was es mir geht.

Vielen Dank für die Hilfe!!

 

[Marco13]

Ein bißchen wenig Code und etwas zu schwammig und/oder unpräzise, um was konkretes sagen zu können...

- Als Model benutze ich ein Array, das das Spielfeld darstellt.

Ein roher Array ist kein Modell. Das Modell im MVC-Pattern ist üblicherweise etwas, wo man Listener/Observer dran hängen kann, die bei Änderungen benachrichtigt werden. (Ein solcher Listener ist dann eben die View). Ob MVC das Paradepattern für ein Spiel wie Tetris ist, darüber könnte man philosophieren, aber darum mag es hier mal nicht gehen...

Nun zu meiner Frage: Rufe ich vom View aus die resetGame() von Tetris auf, so wird ein ganz neues Fenster geöffnet, da ja mein Controller-Thread ein neues Fenster intialisiert.

Wie muss ich das ganze denn umdesignen, dass es Sinn macht?

Von der View aus nicht Tetris#resetGame aufrufen :joke: Nein, im ernst: Wenn man da ein neues Objekt erstellt, ist das ja kein "reset" in diesem Sinne. Man würde ja meinen, dass dieser Reset z.B. bewirken sollte, dass im Modell alle Steine gelöscht werden oder so... demnach könnte man eine Aufrufkette haben, über den Controller ans Modell, wo dann irgendwas wie "removeAllStones" oder so aufgerufen wird... Wenn dort aber auch noch ein Thread angehalten werden soll, muss man sich genau überlegen, wie und wo man das macht (allgemein, die Funktionen die man öfter braucht: Start, Stop, Reset)

Ich dachte etwa daran, dass ich in der Klasse Tetris den View initialiseire (der läuft dann ja im main-Thread, nicht im Controller-Thread)

Die View (bzw. alles, was mit Swing zu tun hat) läuft im Event-Dispatch-Thread, und NICHT im main-Thread. Auch das sollte man beachten.

und einen eigenen Controller-Thread. Wenn ich das tue, dann "hört" das JFrame (z.B. bei repaint()) aber nicht mehr auf mein Controller, also wenn der sich zB ändert oder?

Das "hören" könnte durch die oben angedeuteten Listener abgedeckt sein ... (vielleicht heißen die genau deswegen auch so :reflect: 😉 )

 

[mrbela]

Erstmal vielen Dank für Deine Hilfe!

Das mit den Threads, das ist sone Sache..!!

Welcher Thread zeichnet denn zB im JPanel (mit Hilfe der paint-Methode). Das müsste auch der Event-Dispatch-Thread sein, oder?

Noch eine frage: Wenn das Spiel vorbei ist, dann öffnet sich ein JDialog mit ner Highscore (per JDBC). Sollte der Thread, der das Spielfeld malt, dann per stop() gestoppt werden, oder einfach nichts mehr zeichnen? Was wäre denn der schönere Programmierstil?

Vielen Dank!

 

[Marco13]

Ja, Zeichnen macht der EDT. Deswegen irritiert mich auch die Frage nach dem "Thread, der das Spielfeld malt" - den solltest du demnach tunlichst NICHT stoppen 😉 Oder meintest du einen anderen Thread, der mit repaint() ein Neuzeichnen auslöst?