Keylistener problem

[grayson]

hallo, ich breche mir gerade an einem keylistener einen ab.

und zwar habe ich eine gui, in der ich chatten kann etc.
ausserdem wird eine zweite gui (von jframe abgeleitet) geöffnet, wenn in meinem spiel ein kampf gemeldet wird.

die erste (chatgui) bleibt aber offen, so das man auch im spiel weiter mit der lobby chatten kann.

das ganze sieht so aus :

klasse chatgui extends jframe ----> der chat
klasse gameclient implements remote blah blub ---> der spieleclient (über rmi)
klasse gameUIContainer extends jframe -----> die spielUI (das grundgerüst buttons , scrollpane etc)
unterklasse von gameclient gameui extends gameUIContainer ----> hier wird die spielkarte in den container eingehängt und die eigendlichen actions etc. implementiert

der konstruktor von gameUI ruft natürlich als erstes super(); auf, damit der gameUIContainer aufgebaut wird.

nun möchte ich auf die spielkarte (extends jcomponent) einen keylistener einhängen oder zumindest doch auf die gameUI, damit ich per tastatur auf der karte scrollen kann und diverse andere sachen.

da aber meine chatUI noch offen ist, bekomme ich zb. requestFocus() immer false, wesshalb natürlich der als anonyme klasse im konstruktor von gameUI eingehängte keylistener nie anspringt.

so, wie bekomm ich nun auf meine spielkarte (liegt im mapScrollPane (ein jscrollpane)) einen keylistener hin, ohne das ich die chatUI zu machen muss?

 

[Sky]

Kannst Du das nochmal verständlicher formulieren bzw. mal'n bißchen Code zeigen... ich komm' mit deiner Beschreibung überhaupt nicht zurecht und verstehe das Problem nicht.

 

[grayson]

also ganz schlicht ausgedrückt : ich hab 2 jframes offen und will auf dem 2ten nen keylistener einhängen.

dummerweise wird der 2te aus dem ersten heraus geöffnet und lässt sich nicht mit requestFocus() fokussieren, womit der keylistener nicht reagiert

 

[grayson]

der code :
also das hier ist das fenster, wo der listener nicht geht

package client;

import gui.GameClientContainer;
import .....

/**
 * Der spielclient - hier geht das eigendliche game ab
 */
public class GameClient extends UnicastRemoteObject implements GameClientMethods {

  
    GameUI mUI = new GameUI();

    /**
     * Constructor for GameClient.
     * 
     * @param pl
     *            Planet
     * @param mapID
     *            int
     * @param attacker
     *            User
     * @throws RemoteException
     */
    public GameClient(Planet pl, int mapID, User attacker) throws RemoteException {
        super();
        BasicConfigurator.configure();
        mUser = attacker;
        mPlanet = pl;
        mHexMapID = mapID;
        mUI.setTitle("Strategic Command Console, Commander " + mUser.getName());
        mUI.setSize(new Dimension(800, 700));
        mUI.setCenter();
        mUI.setVisible(true);

    }

 ....methoden......


/*HIER GEHTS LOS; DAS IST DER FRAME AUF DEM DER KEYLISTENER SITZEN SOLL */  
  class GameUI extends GameClientContainer {
        ArrayList mFieldstoMove = new ArrayList();

        Coord mSelected = null;

        public int mSpaceclickcount = 0;

        Unit mUnitToDoAction = null;

        // cursor move codes

        final int CURSOR_UP = 38;

        final int CURSOR_DOWN = 40;

        final int CURSOR_LEFT = 37;

        final int CURSOR_RIGHT = 39;

        private HexMap mHexMap = null;

        public GameUI() {
            super();
       
            
            //das hier geht nicht
            requestFocus();
          addKeyListener(new KeyAdapter() {
                public void keyPressed(KeyEvent e) {
                    moveCursor(e.getKeyCode());
              }
           });
        }
      
     

        /**
         * bewegt den cursor über die map
         * 
         * @param direction
         *            int
         */
        public void moveCursor(int direction) {

            Coord start = mHexMap.getSelection();
            Coord step = null;
            switch (direction) {
            case (CURSOR_UP):
                step = new Coord(start.x, start.y - 1);
                if (step.y < 0 || step.y > mHexMap.getModel().h) {
                } else {

                    mHexMap.setSelection(step);

                    JLabel tmp = new JLabel();
                    tmp.setLocation(mHexMap.getFieldPoint(step));
                    if (this.getMainPanel().getViewport().getViewRect().contains(tmp.getLocation()) == false) {
                        this.getMainPanel().getVerticalScrollBar().setValue((int) mHexMap.getFieldPoint(step).getY());
                    }
                }
                break;

            case (CURSOR_DOWN):
                step = new Coord(start.x, start.y + 1);
                if (step.y < 0 || step.y > mHexMap.getModel().h - 1) {
                } else {
                    mHexMap.setSelection(step);
                    JLabel tmp = new JLabel();
                    tmp.setLocation(mHexMap.getFieldPoint(step));
                    if (this.getMainPanel().getViewport().getViewRect().contains(tmp.getLocation()) == false) {
                        this.getMainPanel().getVerticalScrollBar().setValue((int) mHexMap.getFieldPoint(step).getY());
                    }
                }
                break;

            case (CURSOR_RIGHT):
                step = new Coord(start.x + 1, start.y);
                if (step.x < 0 || step.x > mHexMap.getModel().w - 1) {
                } else {
                    mHexMap.setSelection(step);
                    JLabel tmp = new JLabel();
                    tmp.setLocation(mHexMap.getFieldPoint(step));
                    if (this.getMainPanel().getViewport().getViewRect().contains(tmp.getLocation()) == false) {
                        this.getMainPanel().getHorizontalScrollBar().setValue((int) mHexMap.getFieldPoint(step).getX());
                    }
                }
                break;

            case (CURSOR_LEFT):
                step = new Coord(start.x - 1, start.y);
                if (step.x < 0 || step.x > mHexMap.getModel().w) {
                } else {
                    mHexMap.setSelection(step);
                    JLabel tmp = new JLabel();
                    tmp.setLocation(mHexMap.getFieldPoint(step));
                    if (this.getMainPanel().getViewport().getViewRect().contains(tmp.getLocation()) == false) {
                        this.getMainPanel().getHorizontalScrollBar().setValue((int) mHexMap.getFieldPoint(step).getX());
                    }
                }
                break;

            }

        }

     

     
}

naja und hier wird das obige fenster aus einem anderen geöffnet :

....
 try {
            GameClient game = new GameClient(pl, mapID, mUser);
            HexMap map = new MapFileHandler().LoadMap(mServer.getMapForID(mapID).getDoc());

            game.setMap(map);
            game.actAsServer(mServer);
            game.setMMyUnits(mServer.getUnitsForPlayer(mUser.getID()));

        } catch (RemoteException rex) {
            log.error("could not start gameclient! " + rex);
            return;
        }

.....

 

[Sky]

Es könnte daran liegen, dass Du in die "requestFocus();" aufrufst, bevor der Frame sichtbar ist.
Ruf mal im Konstruktor von 'GameClient' nach dem "mUI.setVisible(true);" folgendes auf:

mUI.requestFocus();

 

[TRunKX]

Ich weiss das es dumm klingt aber der Key Listener muss auch implementiert werden



..... implements KeyListener sollte da oben noch hin oder hat das beerbte Objekt sowas schon???

Stehe da gerade etwas auf dem schlauch ... Und der Focus wechselt normal autom ins neue Fenster!

 

[Sky]

..... implements KeyListener sollte da oben noch hin oder hat das beerbte Objekt sowas schon???

Prinzipiell schon vorhanden:
siehe

addKeyListener(new KeyAdapter() {
  public void keyPressed(KeyEvent e) { 
    moveCursor(e.getKeyCode()); 
  } 
});

Es wird eine neue Instanz der Klasse KeyAdapter erzeugt (naja, eigentlich ist es eine anonyme Klasse, die von KeyAdapter erbt), bei der die Methode keyPressed überschrieben ist. Die Klasse KeyAdapter implementiert in diesem Fall das notwendige Interface.

 

[grayson]

aye, habs mittlerweile auch hin bekommen.

die anonyme klasse wird nun nicht mehr im konstruktor der gameUI implementiert sondern in der übergeordneten klasse gameclient ----> find ich zwar doof, weil ich den code dort halten wollte, wo er etwas bewirken soll aber ok.

sieht dann so aus :

public GameClient(Planet pl, int mapID, User attacker) throws RemoteException {
        super();
        BasicConfigurator.configure();
        mUser = attacker;
        mPlanet = pl;
        mHexMapID = mapID;
        mUI.setTitle("Strategic Command Console, Commander " + mUser.getName());
        mUI.setSize(new Dimension(800, 700));
        mUI.setCenter();
        mUI.setVisible(true);
       // so geht es ---> fragt mich nicht warum
       mUI.requestFocus();
  
// sieht aus, als wenn standardmäßig schon einer da wäre, der mit den cursortasten brechend langsam die karte scrollt
//----> aber ich hab den nicht eingehängt, scheint son standard swing teil zu sein, das per default auf nem scrollpane
//liegt...?!?  naja, jedenfalls nehm ich ihn hier erstmal weg sicher ist sicher ;)
  for(int i=0;i<mUI.getKeyListeners().length;i++){
        KeyListener l = mUI.getKeyListeners()[i];
        mUI.removeKeyListener(l);
    }
        
  // so und nun geht es :D
        mUI.addKeyListener(new KeyAdapter() {
            public void keyPressed(KeyEvent e) {
       ....

Zeilenumbrüche eingefügt / Stefan1200