Keylistener hört auf zu arbeiten

[Titanpharao]

Guten morgen,

ich habe ein JFrame, darauf liegen zwei panels. Dem fenster habe ich einen keylistener zugewiesen, der auch soweit funktioniert. Nur sobald ich einen Button auf dem zweiten Panel anklicke, hört dieser auf zu arbeiten...warum?
Habe schon setFocusable gemacht, damit es überhaupt läuft.
Mfg

 

[The_S]

Weil nur die Componente, die auch wirklich den Fokus hat, KeyEvents "absetzen" kann.

 

[Titanpharao]

Versteh ich jetzt nicht so ganz...soweit bin ich bisjetzt auszüge...

public class Window extends JFrame implements KeyListener{

	public Window(Gebietverwaltung gv,Spieler spieler,Buttonloader bl){
		this.setLayout(null);
		this.setResizable(false);
		this.addWindowListener(new WindowAdapter(){
			public void windowClosing(WindowEvent e){
				System.exit(0);
			}});
		this.setLocationRelativeTo(null);		
		this.setVisible(true);
		this.addKeyListener(this);
		this.setFocusable(true);

		menu=new Menu(this.bl,spieler,this);
		this.add(menu);


class Menu extends JPanel{	
	public Menu(Buttonloader bl,final Spieler spieler,final Window fenster){
		this.setSize(640,32);
		this.setLayout(null);
		this.setLocation(0, 480);		
		anzeige.setSize(640, 32);
		anzeige.setLocation(0, 0);
		hp=new JProgressBar();
		...weitere Button etc...

Wenn ich jetzt in diesem neuen Panel einen Button klicke, geht mein Keylistener für das Fenster, in dem sich das Panel befindet nicht mehr...

 

[The_S]

Dann poste doch mal auch wirklich relevanten Code. Und nicht sowas, was überhaupt nichts mit deinem Problem zu tun hat.

Außerdem:

- Wenn du Klassen so benennst, wie Standardklassen aus dem JDK, kann es leicht zu dummen Verwechslungen kommen, wenn man nicht aufpasst (Window)
- Null-Layout nur dann verwenden, wenn du wirklich weißt, was du da tust

 

[Titanpharao]

Ok, habe sie mal zu Engine umbenannt. Ja ich weis wie das NullLayout fuktioniert und brauch dies auch. Es macht sich nun mal einfach dumm, bei einem spiel alles in Layoutmanager zu packen, wenn der eine Button 2 bzw der andere 3Pixel weit entfernt sein soll.

Hier mal der große Code, sry für ganzen überflüssigen rest.

package testworld_IMoY;

import java.awt.event.KeyEvent;
import java.awt.event.KeyListener;
import java.awt.event.WindowAdapter;
import java.awt.event.WindowEvent;
import java.io.FileInputStream;
import java.io.FileNotFoundException;

import javax.swing.*;

import javazoom.jl.decoder.JavaLayerException;
import javazoom.jl.player.Player;

public class Engine extends JFrame implements KeyListener{
	boolean sound=true;
	boolean run=true;
	Gebietverwaltung gv;
	Buttonloader bl;
	JPanel world=new JPanel();
	Menu menu;
	JLabel player;
	Spieler spieler;
	Music bgm;

	public Engine(Gebietverwaltung gv,Spieler spieler,Buttonloader bl){
		this.gv=gv;
		this.bl=bl;
		this.spieler=spieler;
		this.setTitle("IMoY Test");
		this.setSize(640+4,512+32);
		this.setLayout(null);
		this.setResizable(false);
		this.addWindowListener(new WindowAdapter(){
			public void windowClosing(WindowEvent e){
				System.exit(0);
			}});
		this.setLocationRelativeTo(null);		
		this.setVisible(true);
		this.addKeyListener(this);
		this.setFocusable(true);
		world.setSize(640,480);
		world.setLocation(0, 0);
		world.setLayout(null);
		this.add(world);
		menu=new Menu(this.bl,spieler,this);
		this.add(menu);	
	    player=new JLabel(new ImageIcon("testtexture/player.gif")); 
	    player.setSize(32, 32); 
	    player.setLocation(spieler.xposy, spieler.yposy);
		setWorld(gv.aktivgebiet);		
		//addRucksack();
		if(spieler.sound){
		bgm=new Music(getBGM(gv.aktivgebiet));
		bgm.execute();		
		}
	}
		

	
	public void setWorld(int worldnr){//zum begehen, der neuen Welt
		sound(worldnr,gv.aktivgebiet);
		world.removeAll();
		gv.aktivgebiet=worldnr;
		world.add(player);
		world.add(gv.gebiete.get(worldnr).hintergrund);
		this.repaint();
	}
	   public void keyReleased(KeyEvent ke){ 
	   } 
	   public void keyTyped(KeyEvent ke){ 
	   } 
	   public void keyPressed(KeyEvent ke){ 
	   int key = ke.getKeyCode();	   
	   System.out.println("dekey");
	   if(run){		    
		   switch (key) {
		   	case KeyEvent.VK_RIGHT:laufen(0);break;
		   	case KeyEvent.VK_DOWN:laufen(1);break;
		   	case KeyEvent.VK_LEFT:laufen(2);break;
		   	case KeyEvent.VK_UP:laufen(3);break;
		   	default:break;
		   }
	   }	   
	   } 

package testworld_IMoY;

import javax.swing.*;

import java.awt.Color;
import java.awt.Insets;
import java.awt.event.ActionEvent;
import java.awt.event.ActionListener;
class Menu extends JPanel{	
	JLabel anzeige;
	JProgressBar hp,mp;
	Spieler spieler;
	Engine fenster;
	public Menu(Buttonloader bl,final Spieler spieler,final Engine fenster){
		this.fenster=fenster;
		this.spieler=spieler;
		this.setSize(640,32);
		this.setLayout(null);
		this.setLocation(0, 480);		
		switch (spieler.typ) {
		case 0:	anzeige=new JLabel(new ImageIcon("testtexture/menu0.gif"));break;
		case 1: anzeige=new JLabel(new ImageIcon("testtexture/menu1.gif"));break;
		case 2:	break;//TODO^^
		default:break;
		}
		anzeige.setSize(640, 32);
		anzeige.setLocation(0, 0);
		hp=new JProgressBar();
		hp.setValue(spieler.getPercentHP());		
		hp.setSize(75,12);
		hp.setLocation(32, 4);
		hp.setBackground(Color.BLACK);
		hp.setForeground(Color.RED);
		bl.blist.get(0).addActionListener(new ActionListener() {
			public void actionPerformed(ActionEvent evt) {
				spieler.useHeiltrank();
				hp.setValue(spieler.getPercentHP());					
				}
		});
		mp=new JProgressBar();
		mp.setValue(spieler.getPercentMP());
		mp.setSize(75,12);
		mp.setLocation(32, 18);
		mp.setBackground(Color.BLACK);
		mp.setForeground(Color.BLUE);
		
		JLabel sound=new JLabel("M:");
		sound.setSize(16, 26);
		sound.setLocation(573, 4);
		this.add(sound);
		JCheckBox mute=new JCheckBox();
		if(spieler.sound)mute.setSelected(true);
		mute.setLocation(587, 12);
		mute.setSize(13, 13);
	    ActionListener actionListener = new ActionListener() {
	        public void actionPerformed(ActionEvent actionEvent) {
	          AbstractButton abstractButton = (AbstractButton) actionEvent.getSource();
	          boolean selected = abstractButton.getModel().isSelected();
	          if(selected){
	          spieler.sound=true;
	          fenster.startsoundfrom();
	          }
	          else{
	          spieler.sound=false;  
	          fenster.bgm.p.close();
	          }
	          System.out.println(selected);
	        }
	      };
		mute.addActionListener(actionListener);
		mute.setMargin(new Insets(-2,-2,-2,-2));//Läst die render verschwinden
		this.add(mute);
		this.add(bl.blist.get(0));
		this.add(bl.blist.get(1));
		this.add(bl.blist.get(2));
		this.add(bl.blist.get(3));
		this.add(bl.blist.get(4));
		this.add(bl.blist.get(5));
		this.add(bl.blist.get(6));
		this.add(hp);
		this.add(mp);		
		this.add(anzeige);		
	}
}

So, also das Menu ist das zweite Panel...was komischerweise dann keine eingaben mehr zuläst. Hatte sowas fast genauso schonmal geschrieben, nur das es dort nur ein Panel gab und es funktionierte
:bahnhof:

 

[Titanpharao]

Glaube ich habe ein oder das Problem.
Wenn ich zum beispiel, einen Button unten anklicke, bleibt der "aktiv" also die Leertaste würde dann zum beispiel denn button betätigen, wie man es aus windows kennt. Erst wenn ich wieder ins oberer Feld reinklicke, dann gehts wieder...

 

[Titanpharao]

Habe jetzt mal an denn Button denn selben KeyListener dran gehangen.

bl.blist.get(0).addKeyListener(fenster);

bzw 

bl.blist.get(0).setFocusable(false);

Dann funktionierts für denn Button. Toll...aber es soll fürs ganze Panel gehen...will doch nicht bei jeden Element das dranhängen.

 

[The_S]

Ja, das ist genau das, wo ich gesagt habe. Der KeyListener reagiert nur, wenn wirklich die Componente, die er hinzugefügt wurde, den Fokus hat. Entweder allen Componenten den KeyListener zuweisen, eine andere Lösungen suchen, oder den AWTEventListener verwenden.

Auch für dein Vorhaben gibt es sehr mächtige und dafür geeignete LayoutManager!

 

[Titanpharao]

nadann....muss ichs halt überall dazumachen....danke.
bzw warum schwer, wenns auch einfach geht^^

 

[The_S]

bzw warum schwer, wenns auch einfach geht^^

bezogen auf?