Event Handling KeyListener funktioniert nach Restart nicht mehr

[Doopy]

Hallo zusammen,

ich arbeite derzeit an einem kleinem 2D-Spiel bzw. Applet.
Man bewegt eine Spielfigur mittels Pfeiltasten bis zu ihrem Tod und kann danach einen neuen Versuch starten.
Nun funktioniert der KeyListener nach dem ersten "Play again"-Button-drücken nicht mehr.
Hier ein Auszug aus der Hauptklasse:

AppMain.class

public void init() {
		setSize(480, 800);
		setFocusable(true);
		
		bg1 = new Background(0, 0, this);
		bg2 = new Background(0, 800, this);
		platformhelper = new PlatformHelper(this);
		platformhelper.createInitialPlatforms();
		spieler = new Spieler(this);
		
		gameKL = new GameKeyListener(spieler);
		addKeyListener(new GameKeyListener(spieler));
		System.out.println("keylisteners_init:"+getKeyListeners());
		
		panel = new AppPanel(this, spieler, platformhelper);
	    panel.setLayout(null);
	    add(panel);
	    this.setContentPane(panel);
	    
	    state = GameState.Running;
	    
	    

	}


[...]


         public void restart() {
		setGameSpeed(-2);
		bg1 = new Background(0, 0, this);
		bg2 = new Background(0, 800, this);
		platformhelper = new PlatformHelper(this);
		platformhelper.createInitialPlatforms();
		state = GameState.Running;
		spieler = new Spieler(this);
		removeKeyListener(gameKL);
		gameKL = new GameKeyListener(spieler);
		addKeyListener(new GameKeyListener(spieler));
		System.out.println("keylisteners_restart:"+getKeyListeners());
		
		panel = new AppPanel(this, spieler, platformhelper);
	    panel.setLayout(null);
	    this.setContentPane(panel);
	    paintComponents(panel.getGraphics());
	}

Wie gesagt, am Anfang klappt es, bis man mit dem Playagain-Button die restart()-Funktion aufruft.
Der Code ist schon etwas hässlich, hab schon ne Menge versucht.


MfG

 

[javampir]

Hallo,
ich versteh nicht ganz: was ist gamekeylistener? eine eigene klasse, die KeyListener implementiert?
Und was passiert mit gameKL aus Zeile 11? das wird ja nirgends geaddet, trotzdem removst du ihn in 38, und dann machst du nochmal das gleiche, du erstellst gameKL und verwendest dann einen neuen GameKeyListener (den du übrigens nicht removt hast). Erklär das bitte noch mal detailierter.
javampir

 

[Doopy]

Achja, ja genau, eine Klasse die KeyListener implementiert. Hier ihr Code:

GameKeyListener.class:

public class GameKeyListener implements KeyListener {

	private Spieler s;
	
	public GameKeyListener(Spieler s) {
		this.s = s;
	}

	@Override
	public void keyPressed(KeyEvent e) {
		System.out.println("key");
		switch (e.getKeyCode()) {
		
		case KeyEvent.VK_LEFT:
			s.moveLeft();
			s.setMovingLeft(true);
		break;

	    case KeyEvent.VK_RIGHT:
	    	s.moveRight();
	    	s.setMovingRight(true);
		break;
		}
	}

	@Override
	public void keyReleased(KeyEvent e) {
         switch (e.getKeyCode()) {
		
		case KeyEvent.VK_LEFT:
			s.stopLeft();
		break;

	    case KeyEvent.VK_RIGHT:
	    	s.stopRight();
		break;
		}
	}

	@Override
	public void keyTyped(KeyEvent e) {
	}

}

Ups, das hab ich voll übersehen Muss natürlich so sein:

gameKL = new GameKeyListener(spieler);
addKeyListener(gameKL);

Geht trotzdem immer noch nicht :/

 

[Doopy]

Keiner eine Idee?

 

[eMmiE]

Guck mal nach, ob du das mit addKeyListener(gameKL) auch beim restart() so drin hast

Was gibt er dir beim SysOut aus?

Gruß eMmiE

 

[Doopy]

Ja, hab ich auch geändert, funzt nicht.

Beim SysOut gibt er mir immer verschiedene KeyListener aus, wie es auch sein sollte.
Allerdings ist mir dabei jetzt eben aufgefallen, dass die restart()-Funktion während dem Plattformgenerieren mehrmals aufgerufen wird, und dadurch wieder alles von vorne generiert wird. Hier der Codeausschnitt aus der GameoverScreen.class:

public void paint(Graphics g) {
		g.drawImage(background, 0, 0, panel);
		System.out.println("DEAD");
		g.setColor(Color.BLUE);
		g.setFont(fontGameover);
		g.drawString("Game Over!", 80, 250);
		g.setFont(fontScore);
		g.drawString("Dein Score: " + String.valueOf(spieler.getDistance()), 140, 400);
		playagain.addActionListener(new ActionListener() {
			public void actionPerformed(ActionEvent e) {
				System.out.println("playagain button");
				m.restart();
			}
		});
		panel.add(playagain);
		

	}

 

[Doopy]

Okay, ich hab endlich den Fehler gefunden.
Es lag ganz einfach daran, dass ein Button den Fokus hatte und deswegen der KeyListener nicht ansprang
Einfach den Button .setFocusable(false) und jetzt gehts
Allerdings frag ich mich immernoch warum die restart()-Funktion mehrmals aufgerufen wird wenn der playagain-Button gedrückt wird...?

 

[Doopy]

Okay das hat sich jetzt auch geklärt.
Die paint()-Methode des Screens wurde permanent aufgerufen um die Grafiken jeweils neu zu zeichnen, da die Buttons aber auch in der selben Funktion waren, wurden sie einfach zigfach übereinander gelegt

Trotzdem Danke euch allen für die Hilfe!