KeyListener wird ignoriert

[JDuke]

Halloo!

Hab folgendes Problem, bei meinem JFrame wird nicht auf den KeyListener reagiert...

     super.addKeyListener(
       new KeyAdapter() {
         public void keyPressed(KeyEvent event)
         {
           if (event.getKeyCode() == KeyEvent.VK_ESCAPE) {
             setVisible(false);
             [...]
           }
         }
       }
     );

    listenerPanel.add(jsVertical);
    listenerPanel.add(jsHorizontal);
    cp = getContentPane();
    cp.setLayout(new BorderLayout());
    cp.add(listenerPanel, BorderLayout.CENTER);
    cp.add(jlbActualAction,BorderLayout.SOUTH);
    setVisible(true);

Hab schon alles probiert. Ohne Erfolg.

Greetz
J

 

[Sky]

Ich habe in Zeile 2 das 'super' durch 'this' ersetzt und schon hat es funktioniert.

 

[JDuke]

Danke erst mal für die schnelle Antwort...
Hier noch mal der Komplette code vielleicht liegt der Fehler ja wo anders

import java.awt.*;
import javax.swing.*;
import java.awt.event.*;

class TestGUI extends JFrame{

    Container cp;
    JPanel listenerPanel;
    JSlider jsVertical;
    JSlider jsHorizontal;
    JLabel jlbActualAction;

    public TestGUI(){
        super("KeyListenerTest");
        setSize(300,300);
        setLocationRelativeTo(null);


        addWindowListener(new WindowAdapter(){
            public void windowClosing(WindowEvent evt){System.exit(1);
        }
    });

    listenerPanel = new JPanel();
    listenerPanel.setLayout(new FlowLayout());
    jsVertical = new JSlider(1,-10,10,0);
    jsHorizontal = new JSlider(0,-10,10,0);
    jsVertical.setValueIsAdjusting(false);
    jsHorizontal.setValueIsAdjusting(false);
    jlbActualAction = new JLabel("Nix");

    this.addKeyListener(new KeyAdapter(){
        public void keyPressed(KeyEvent e){
            int key = e.getKeyCode();
            int w = KeyEvent.VK_W;
            System.out.println("KeyCode:"+key+"\nTaste:"+w);
            if(key==w){           //-- Code für Up
                //jsVertical.setValue(jsVertical.getValue()+1);
                jlbActualAction.setText("Up Pressed...");
            }

            if(e.getKeyCode() == KeyEvent.VK_S){         //-- Code für Down
                //jsVertical.setValue(jsVertical.getValue()-1);
                jlbActualAction.setText("Down Pressed...");
            }

            if(e.getKeyCode() == KeyEvent.VK_A){         //-- Code für Left
                //jsHorizontal.setValue(jsHorizontal.getValue()-1);
                jlbActualAction.setText("Left Pressed...");
            }

            if(e.getKeyCode() == KeyEvent.VK_D){         //-- Code für Right
                //jsHorizontal.setValue(jsHorizontal.getValue()+1);
                jlbActualAction.setText("Right released...");
            }

        }
         public void keyReleased(KeyEvent e){
            if(e.getKeyCode() == KeyEvent.VK_UP){           //-- Code für Up
                //jsVertical.setValue(0);
                jlbActualAction.setText("Up released...");
            }

            if(e.getKeyCode() == KeyEvent.VK_DOWN){         //-- Code für Down
                //jsVertical.setValue(0);
                jlbActualAction.setText("Down released...");
            }

            if(e.getKeyCode() == KeyEvent.VK_LEFT){         //-- Code für Links
                //jsHorizontal.setValue(0);
                jlbActualAction.setText("Left released...");
            }

            if(e.getKeyCode() == KeyEvent.VK_RIGHT){        //-- Code für Rechts
                //jsHorizontal.setValue(0);
                jlbActualAction.setText("Right released...");
            }

         }
    });

    listenerPanel.add(jsVertical);
    listenerPanel.add(jsHorizontal);

    cp = getContentPane();
    cp.setLayout(new BorderLayout());
    cp.add(listenerPanel, BorderLayout.CENTER);
    cp.add(jlbActualAction,BorderLayout.SOUTH);
    setVisible(true);
    }

    public static void main(String[] args){
        new TestGUI();
    }

}

greetz
J

 

[CelikBlek]

mach mal einen repaint direkt nach einer veränderung

 

[JDuke]

is er bereits

 

[DesertFox]

repainten, mehr net, also nach den setText() aufrufen in der Anonymen Klasse einfach nochmal repaint() aufrufen, dann sollte es klappen

 

[CelikBlek]

nach einer repaint muss es funzen

 

[JDuke]

???:L sorry aber es ändert sich nichts... noch immer keine Reaktion auf keyevents

 

[JDuke]

???:L sorry aber es ändert sich nichts... noch immer keine Reaktion auf keyevents.

nach einer repaint muss es funzen

es geht nicht um das repaint bzw da JLabel.setText()... es geht nur darum dass der keylistener erst überhaupt mal reagiert.
könnte genau so ein System.out.println hinschreiben - funzt trotzdem nicht...

 

[bygones]

bei einem setText von einem Label muss kein repaint hin... dat geht von alleine.

Ich geh mal davon aus, dass der Frame nicht den Focus hat, sondern eine seiner Components -> der Frame kann nicht auf Key Ereignisse reagieren

 

[thE_29]

Du musst den Frame so machen, dann sollte es gehen

http://www.java-forum.org/de/viewtopic.php?t=6029&highlight=awteventlistener

 

[Niki]

Beispiel:

JComponent c = (JComponent)getContentPane();
InputMap im = c.getInputMap(JComponent.WHEN_IN_FOCUSED_WINDOW);
ActionMap am = c.getActionMap();

im.put(KeyStroke.getKeystroke([DEIN KEYSTROKE]), "identifier");
am.put("identifier", new AbstractAction(){
    public void actionPerformed(ActionEvent e){
        //tu das was du nach dem keyEvent tun willst
    }
});

Ich habe alles so implementiert, es funktioniert nur dann nicht, wenn der Focus in einer Component drinnen ist, wo du einen ActionListener bzw. einen KeyListener registriert hast

 

[JDuke]

Ok hab den Code so eingefügt wie er ist...
:arrow: Auf das Keyereignis wird nur einmal reagiert...
:?: Muss ich das jetz in einen Thread eipacken und dauernd abfragen?

Is ja ne unschöne lösung oder? Gibts da keine bessere Möglichkeit

Mfg

 

[thE_29]

Wieso nimmst du net den Vorschlag von mir, da musst du weder in einem Thread rumgurken und es ist egal welche Komponente den Focus hat!

 

[JDuke]

weil ich da einige befehle nicht verstehe... habs ja schon probiert...

was meinst du mit windowsstartservices und linuxstartservices bzw. welches import benötige ich dazu?
und was bedeutet "strOSystem" [schild=9 fontcolor=000000 shadowcolor=C0C0C0 shieldshadow=1]Nix kapier![/schild]

Greetz

 

[thE_29]

Aso, na das kannst weglassen!

das alles in der ActionFunkion (//wenn die Taste F5 ist
if (event.paramString().indexOf("F5") != -1)
{
//Windows Funktion
if (strOSystem.compareTo("WINDOWS") == 0)
{
WindowsStartServices();
}
//Linuxfunktionen
else
{
LinuxStartServices();
}
} )

kannst alles weglassen!
Das sind von mir Funktionen

 

[JDuke]

So habs jetz so gemacht... funktioniert schon ma ganz gut!

mit diesem Code:

public void eventDispatched(AWTEvent event){

    int ID = event.getID();

    if (ID == KeyEvent.KEY_PRESSED)
    {
      System.out.println("Kompletter EventString:\n  "+event.paramString());
      System.out.println("Gefiltert: "+event.paramString().substring(31,32)+"\n\n");

      if(event.paramString().substring(31,32).equals("W")){
        jlbActualAction.setText("W is pressed...");
        //TODO Action die beim drücken der "W" Taste ausgeführt wird
      }
      if(event.paramString().substring(31,32).equals("S")){
        jlbActualAction.setText("S is pressed...");
        //TODO Action die beim drücken der "S" Taste ausgeführt wird
      }
      if(event.paramString().substring(31,32).equals("A")){
        jlbActualAction.setText("A is pressed...");
        //TODO Action die beim drücken der "A" Taste ausgeführt wird
      }
      if(event.paramString().substring(31,32).equals("D")){
        jlbActualAction.setText("D is pressed...");
        //TODO Action die beim drücken der "D" Taste ausgeführt wird
      }
      if(event.paramString().substring(31,32).equals("X")){
        jlbActualAction.setText("X is pressed...");
        //TODO Action die beim drücken der "X" Taste ausgeführt wird
      }
    }
   }

Ist wiedermal etwas unschön wie ich den Keywert bekomme...
weiß vielleicht jemand wie ich aus dem ParameterString den KeyCode bekomme.
Greetz

 

[bygones]

kannst doch den event zu einem KeyEvent casten und dann per getKeyCode() das char abfragen

 

[thE_29]

Jop
Nur war ich zu faul, da ich zz nur noch 1 key abfrage