Panel Probleme (Tetris)

[xyss]

Moin,

Ich habe bei meinem aktuellen Projekt ein kleines Problem mit den Panels, es sollte ziemlich offensichtlich sein:

Infos zum Problem:
Wenn man die Pausefunktion aktiviert (per Drücken der P-Taste), wird zusätzlich zum pausierten Spiel das Ganze noch visualisiert, indem das Spielfeld ausgegraut wird. Dies ist realisiert durch ein Panel, welches ich als pausePanel bezeichnet habe. Es ist die ganze Zeit vorhanden, jedoch nicht "sichtbar", da das Spielfeld-Panel dauerhaft repaintet wird. Bei aktivieren der Pause-Funktion wird einfach das pausePanel repainted damit es "drüber" liegt.
So, nun kommt es aber MANCHMAL dazu, dass dieser kleine Kasten, welcher ebenfalls durch ein eigenes Panel dargestellt wird, bei dem pausePanel mit erscheint, und ich bin absolut ratlos, woher das kommt, da diese beiden Panel rein gar nichts gemein haben... außer dass sie nunmal auf dem gleichen Frame liegen.

Genauere Erläuterung der Panels:

		add(pausePanel);
		add(scorePanel);
		add(gamePanel);
		add(ntPanel);
		add(bg);
		pack();

So werden sie zusammen auf das Frame gepackt.
Das pausePanel ist wie gesagt zuständig für das ausgegraute, sobald die Pause aktiviert wurde. Die Größe des Panels ist dementsprechend identisch mit der Spielfeldgröße.

Das scorePanel ist zuständig für die Labels (auch die Steuerung, daher sollte ich den Namen wohl demnächst ändern). Die Größe des score-Panels entspricht der weiß umrandeten Fläche an der rechten Seite (also ohne das Spielfeld)

Das gamePanel enthält das Spielfeld, und wird dementsprechend oft repainted. Die Größe entspricht exakt dem sichtbaren Spielfeld.

Das ntPanel ist das Panel, welches mir scheinbar die Probleme bereitet. Der Name steht für "NextTilePanel", da es sich hierbei um das kleine Kästchen handelt, in welchem angezeigt wird, welches Element als nächstes kommt. Es wird immer gleichzeitig mit dem gamePanel repainted.

Das bg Panel ist das background Panel. Es beinhaltet den schwarzen Hintergrund sowie die weißen Umrandungen etc.

Achja, und wenn es einmal vorgekommen ist, kann ich auch so oft ich will Pause drücken und wieder weg machen, es kommt immer wieder. Umgekehrt ebenso. Anders gesagt, entweder tritt es gar nicht auf, oder bei jedem Mal wo ich Pause drücke (bis ich das Programm halt beende)

Und ich frag mich nun einfach, wieso um alles in der Welt wird beim repaint() des pausePanels, das ntPanel oben links in die Ecke geklatscht...


So... falls es doch ein Code-basierter Fehler ist, und kein Denkfehler, werde ich noch Code posten..

lg

 

[Bug Fisher]

Hi xyss,

schick, wie du das gebaut hast; mein Tetris von vor paar Jahren war nicht so stylisch

So, nun kommt es aber MANCHMAL dazu...

'Manchmal' ist meistens ein Hinweis auf fehlende Synchronisation. Startest du eigene Threads (oder highleveligere Konstrukte, die Threads erzeugen) ?
Kannst du sicher sein, dass das repaint() immer und einzig vom EDT durchrannt wird ?

Auch möglich, dass du die EventQueue des EDT "bombardierst", dann kann es ebenfalls zu solchen Artefakten kommen.

Du nutzt vermutlich ein null-Layout ?? (setLayout(null))
Eine LayeredPane wäre auch eine Möglichkeit.

Rufe auf alle Komponenten, bei denen es Sinn macht, ein setOpaque(true).

 

[xyss]

Ich habe mir gedacht, dass ich bei Tetris keine extra Threads benötige, da im Endeffekt immer nur ein Element gleichzeitig in Bewegung ist.. Denkfehler?
Von daher -> nein ich starte keine eigenen Threads..

Ich hab mir auch schon gedacht, dass es ein Synchronisationsproblem ist, jedoch bin ich nunmal Ahnungslos, wie ich es löse

Das pausePanel benutzt ein BoxLayout, die anderen sind Null-Layouts (abgesehen vom ScorePanel benutze ich ja auch keine anderen Componenten, sondern "male" nur rum, dann brauch ich doch kein Layout, oder nicht?)

Das mit setOpaque werde ich mal versuchen

Etwas was mir auch aufgefallen ist:
Ich habe es jetzt so eingestellt, dass es ganz am Anfang pausiert ist, also, das gesamte Spiel wird initialisiert, aber bevor auch nur ein "tick" durchlaufen wird, wird auf Pause gesetzt. Und in dem Zeitpunkt, obwohl das ganze Frame ja bereits aufgebaut wurde etc, ist das ntPanel (NextTilePanel) nicht sichtbar.

Edit: Scheint doch kein Sync Problem zu sein.. ich konnte den Fehler "manuell" reproduzieren, in dem ich mehrmals das pausePanel und das ntPanel repainted habe..
Hier mal Schritt für Schritt, wie sich das Aussehen verändert, wenn ich die beiden Abwechselnd repainte, nachdem das ganze Frame fertig aufgebaut ist:

 

[Bug Fisher]

Zeig doch mal den ganzen Code um das

add(pausePanel);
add(scorePanel);
add(gamePanel);
add(ntPanel);
add(bg);
pack();

herum, wenn es dir nichts ausmacht.

 

[xyss]

Ich habe es vorerst so gelöst, dass ich bei der paint Methode vom pausePanel zuerst clearRect benutze. Dadurch sieht man während der Pause das Spielfeld nicht mehr, stattdessen ist das pausePanel komplett grau. Aber immerhin sieht man dadurch den Fehler auch nicht mehr.

backgroundPanel:

	public BackgroundPane(){
		setBounds(0,0,WIDTH,HEIGHT);
		setPreferredSize(new Dimension(WIDTH,HEIGHT));
		
	}

	public void paint(Graphics g){
		g.clearRect(0, 0, WIDTH, HEIGHT);
		
		/*Hintergrund*/
		g.setColor(Color.BLACK); 
		g.fillRect(0, 0, WIDTH, HEIGHT);
		
		
		//Spielfeldumrandung
		g.setColor(Color.WHITE);
		g.fillRect(GamePanel.OFFSET_X-5, GamePanel.OFFSET_Y-5, GamePanel.WIDTH+10, GamePanel.HEIGHT+10);
		g.setColor(new Color(0,102,255,170));
		g.fillRect(GamePanel.OFFSET_X-5, GamePanel.OFFSET_Y-5, GamePanel.WIDTH+10, GamePanel.HEIGHT+10);
		
		//Linke Box
		g.setColor(Color.WHITE);
		g.drawRect(GamePanel.OFFSET_X - 30, GamePanel.OFFSET_Y- 30, GamePanel.WIDTH + 60, GamePanel.HEIGHT + 60);
		
		
		int boxMidX = (GamePanel.WIDTH+60)/2;
		int boxMidY = (GamePanel.HEIGHT+60)/2;
		
		//Rechte Box
		g.drawRect(BOXCOOR_X, BOXCOOR_Y, RIGHT_BOX_WIDTH, GamePanel.HEIGHT + 60);
	
		
	}

GamePanel:

public GamePanel(Tetris tetris){
		this.tetris = tetris;
		setBounds(OFFSET_X,OFFSET_Y,WIDTH,HEIGHT);
		setPreferredSize(new Dimension(WIDTH,HEIGHT));
		setLayout(null);	
	}

	public void paint(Graphics g){
		g.clearRect(0, 0, WIDTH, HEIGHT);
		
		g.setColor(new Color(0,102,255,100)); //Blauer Hintergrund
		g.fillRect(0, 0, WIDTH, HEIGHT);
		
		g.setColor(new Color(0,0,0,100)); //SpielfeldRaster
		for (int i = 0; i <= Tetris.MAX_X; i++)
			g.drawLine(i*Tile.TILESIZE, 0, i*Tile.TILESIZE, HEIGHT);
		for (int i = 0; i <= Tetris.MAX_Y; i++)
			g.drawLine(0, i*Tile.TILESIZE, WIDTH, i*Tile.TILESIZE);
		
		
		tetris.getElement().paint(g); //Painte das aktuelle Element (das, welches gerade am runterfallen ist)
		
		Tile[][] t = tetris.getTiles(); //Sobald ein Element am Boden bzw an einem anderen Element ankommt, ist es ja nicht mehr steuerbar, sondern fest (bis auf Zeilenlöschungen), daher wird es dann in einen Tile[][] reingeschrieben. Ein "Tile" ist ein einzelnes Kästchen im Spielfeld. Ein Element (Spielstein) ist also aus 4 Tiles aufgebaut. Hier werden also die restlichen Tiles gemalt.

		for (int i = 0; i < t.length; i++){
			for (int j = 0; j < t[i].length ; j++){
				if (t[i][j] != null)
					t[i][j].paint(g);
			}
		}
		
	}

scorePanel

	public ScorePanel(Tetris tetris){
		score = 0;
		level = 1;
		lines = 0;
		this.tetris = tetris;
		
		//setBounds(BackgroundPane.BOXCOOR_X+1+10, BackgroundPane.BOXCOOR_Y + BackgroundPane.NEXTBOX_SIZE+ 30 + 20, WIDTH-10 , HEIGHT );
		//setPreferredSize(new Dimension(WIDTH,HEIGHT));
		setBounds(BackgroundPane.BOXCOOR_X+1, BackgroundPane.BOXCOOR_Y + 1, WIDTH, HEIGHT);
		setPreferredSize(new Dimension(WIDTH,HEIGHT));
		setBackground(Color.BLACK);
		setLayout(null);
		initLabels();
	}

pausePanel:

public PausePanel(){
		setBounds(OFFSET_X,OFFSET_Y,WIDTH,HEIGHT);
		setPreferredSize(new Dimension(WIDTH,HEIGHT));	
		setBackground(new Color(0,0,0,200));
		setLayout(new BoxLayout(this, BoxLayout.Y_AXIS));
		this.state = STATE_PAUSED; //Es gibt auch noch STATE_GAMEOVER, das sorgt einfach dafür, dass je nach State ein anderer Text im Label steht, also entweder "Pause" oder "GameOver"
		
		lblMessage = new JLabel("Pause (P)");
		lblMessage.setFont(FONT);
		lblMessage.setForeground(COLOR);
		lblMessage.setAlignmentX(CENTER_ALIGNMENT);
		lblMessage.setBounds(10,87,100,40);
		
		add(Box.createRigidArea(new Dimension(10,HEIGHT/2-20)));
		add(lblMessage);
	}

	public void paint(Graphics g){
		g.clearRect(0, 0, WIDTH, HEIGHT); //Mit clearRect wird es "untransparent", wodurch der Fehler nicht mehr zu sehen ist, Notlösung.
		super.paint(g);
	}

nextTilePanel:

	public NextTilePanel(Tetris tetris){
		this.tetris = tetris;
		setBounds(BackgroundPane.BOXCOOR_X + 150, BackgroundPane.BOXCOOR_Y + 30, NEXTBOX_SIZE, NEXTBOX_SIZE );
		setPreferredSize(new Dimension(NEXTBOX_SIZE, NEXTBOX_SIZE));
	}
	
	public void paint(Graphics g){
		g.clearRect(0, 0, NEXTBOX_SIZE, NEXTBOX_SIZE); //cleare Alles
                g.setColor(Color.BLACK); 
		g.fillRect(1, 1, NEXTBOX_SIZE, NEXTBOX_SIZE); //schwarz füllen
		g.setColor(Color.WHITE);
		g.drawRect(0, 0, NEXTBOX_SIZE-1, NEXTBOX_SIZE-1); //Weiße Umrandung
		tetris.getNextPaint().paint(g); //Painte das nächste Element
	}

 

[Bug Fisher]

In Swing-Komponenten UNBEDINGT die

protected void paintComponent(Graphics)

nicht

public void paint(Graphics)

überschreiben.

Die Semantik ist exakt die gleiche, du brauchst also nur den Namen und die Sichtbarkeit zu ändern.

Und gewöhne dir besser an, die Annotation @Override zu verwenden, dann hilft dir auch der Compiler, falls du dich aus Versehen beim Methodennamen verschreibst.

 

[xyss]

Wenn die Semantik exakt die gleiche ist, ist es dann nicht irrelevant, welche der beiden man benutzt?

 

[Bug Fisher]

Sorry, ich meinte mit Semantik das, was innerhalb der Methode steht. Vielleicht hätte ich besser Inhalt schreiben sollen.

Es ist einfach so: Swing (als Framework) ruft paint(Graphics g) auf, welches seinerseits paintBorder(g), paintComponent(g), paintChildren(g) auf den Plan ruft. (Ich hoffe die Reihenfolge ist richtig, das kann man nachlesen).
Wenn du jetzt paint() überschreibst, ohne dort super.paint() aufzurufen, machst du das ganze System kaputt und dadurch kann es eben zu solchen Effekten wie bei dir führen, da (relative) Layout-Informationen nicht korrekt weitergegeben werden.

 

[xyss]

Ah, alles klar. Wieder mal etwas dazugelernt

Vielen Dank!