KeyListener funktioniert nicht

[hyperion]

Hallo,

hatte heute mal wieder ein wenig Lust zu programmieren und da wollte ich mich mal damit versuchen das Spiel Snake nachzubauen. Habe mir schon gedacht, dass ich das nicht alles runterprogrammieren kann und ein paar mal hängen bleiben werde, aber das ist ja nicht weiter schlimm.

Die Schlange soll der benutzer mit den Pfeiltasten steuern können. Leider passiert noch gar nichts wenn der Benutzer eine Pfeiltaste drückt.

public class SnakeGame 
{
	...	
	SnakeGame()
	{
		//Begin of GUI
                ...
		labyrinth = new Labyrinth();
                ...
		//End of GUI
	}
	
	public static void main(String[] input)
	{
		new SnakeGame();
	}
}

class Labyrinth extends JLabel
{
	Snake schlange;
	
	Labyrinth()
	{
		...
		
		schlange = new Snake(pos, this);
		[b]addKeyListener((KeyListener)schlange);[/b]
	}
	
	public void setBackground(Color farbe)
	{
		super.setBackground(farbe);
	}

	public void paint(Graphics g)
	{
		...
	}	
}

public class Snake [b]implements KeyListener[/b]
{
	Labyrinth test;
        ...
	
	Snake(int[][] pos, Labyrinth test)
	{
		this.pos = pos;
		this.test = test;
	}	
	
	public void moveUp()
	{
		
	}
	
	public void moveRight()
	{
		[b]test.setBackground(Color.cyan);[/b]
	}
	
	public void moveDown()
	{
		
	}
	
	public void moveLeft()
	{
		
	}
	
	public void keyReleased(KeyEvent e)
	  {
		  
	  }
	  [b]public void keyPressed(KeyEvent e)
	  {
		 if (e.getKeyCode() == KeyEvent.VK_RIGHT)
		 {
			 moveRight();
		 }
	  }[/b]

	  public void keyTyped(KeyEvent e)
	  {
		  
	  }
}

Eigentlich sollte doch der Hintergrund des JLabels cyan werden, er bleibt aber weiß. Ich habe das Gefühl es liegt an addKeyListener((KeyListener)schlange);, da ich ehrlichgesagt überhaupt nicht wusste wo ich diesen KeyListener drauf tun sollte. Bei einem ActionListener ist es ja einfach. Einfach auf den auslösenden Button, aber bei einem KeyListener kann ich die Pfeiltaste adden Wenn Ihr noch Code braucht wo jetzt "..." steht sagt bitte bescheid.

mfg hyperion

 

[cover1012]

ohne mir deinen Code genau angesehen zu haben und ohne Verständnis, warum du ein JLabel als Zeichenfläche benutzt:
Du kannst zwar einem JLabel einen Keylistener hinzufügen, nur braucht das Label ja den Focus damit der Listener anspricht. Ich vermute, genau hier liegt das Problem.

 

[AmunRa]

Du musst ihm auch noch den Auftrag geben das Fenster neuzu zeichnen mit repaint() z.B.


Den KeyListener addest du am Besten dem Fenster auf dem du zeichnen möchtest und nicht auf nem Label

 

[hyperion]

Hallo,

ah ok, ich habe irgendow mal gelesen, dass ein JLabel zum Zeichnen verwendet werden soll. Aber worauf soll ich denn dann zeichnen? Meinst du mit Fenster ein JWindow?

mfg hyperion

 

[AmunRa]

kannst ein JWindow verwenden ein JFrame oder auch ein JPanel

wobei du für letzteres auch wieder ein Frame oder Window brauchst um es anzuzeigen

 

[MiDniGG]

Hm? Wieso nicht einfach ein Canvas? Oder sitz ich grad aufm Schlauch???

 

[Verjigorm]

In/auf JFrame/JWindow zeichnet man nur, wenn man sich das genau überlegt hat.

Stell dir mal vor, du hast eine komplexe Zeichnung auf deinem JFrame und willst noch einen Button oder sonstwas hinzufügen, dann kriegste Schwierigkeiten.
Deshalb ist Zeichnen auf einem JPanel/JLabel imho sinnvoller.

 

[hyperion]

Hallo,

leider hat sich das Problem imemrnoch nicht erledigt und die Meinungen scheinen hier doch ziemlich auseinander zu gehen.
Ich bin jetzt bei einem JLabel geblieben, weil ich das für meine Bedürfnisse passend finde und zumindest einer gesagt hat, dass das nicht verkehrt sei.
Wie setzte ich denn den Fokus auf das JLabel? Habe es shcon mit exzessiven Kilcks auf das JLabel probiert aber die Pfeiltasten zeigen leider trotzdem keine Wirkung.

Und daran könnte es nicht liegen:
addKeyListener((KeyListener)schlange);

Das schaut für mich irgendwie komisch aus und ich habe das nur gemacht, weil es nur mit schlange nicht ging

mfg hyperion

 

[hyperion]

Hallo,

also es lag am Fokus. Mit der Mehtode requestFocusInWindow() lässt sich der Fokus zuweisen

mfg hyperion