Java KeyAdapter - PullRequest
       27

Java KeyAdapter

0 голосов
/ 27 октября 2009

Я немного незнаком с тем, как работает Java KeyAdapter, и получаю неожиданные результаты со следующим кодом, использующим KeyAdapter. Проблема возникает, когда клавиша нажата, в то время как другая клавиша уже нажата, независимо от того, была ли вызвана isKeyPressed().

Примечание: я знаю, что это много кода, и я прошу прощения. Я старался изо всех сил, чтобы изолировать его, и я думаю, что он находится в основном вокруг комментариев в методе keyHandler ниже (как keyHandler() переводит нажатые клавиши в keysHeld). Надеюсь, подробные комментарии полезны.

keyHandler:

ArrayList keysHeld = new ArrayList<KeyEvent>();

private void keyHandler()
{
    KeyAdapter keyListnr = new KeyAdapter()
    {
        public void keyPressed(KeyEvent e)
        { 
            int keyCode = e.getKeyCode();

            int index = 0;
            boolean found = false;
            while(!found && index<keysHeld.size()) //While not already found, and end of ArrayList not reached
            {
                System.out.print("errorCheck: keysHeld: "+keysHeld+", "+(Object)keyCode+" "); //PRINT
                if(keysHeld.get(index) == (Object)keyCode)
                {
                    System.out.println("found"); //PRINT
                    found = true; //This key is already recognized as held
                }
                else
                {
                    System.out.println("not found"); //PRINT
                    //This key is not recognized as held
                }
            }
            if(!found) //If key must be added to keysHeld
            {
                keysHeld.add(keyCode); //Add to list of held keys
            }
        System.out.println(keysHeld.toString()); //PRINT ArrayList of all held keys
    } //end of keyPressed


        public void keyReleased(KeyEvent e) //similar in concept to keyPressed
        {
         int keyCode = e.getKeyCode();

         int index = 0;
         boolean found = false;
         while(!found && index < keysHeld.size())
         {
          if(keysHeld.get(index) == (Object)keyCode)
          {
           keysHeld.remove(index); //remove key from keysHeld
           found = true;
          }
          else
          {
           index++;
          }
         }
         System.out.println(keysHeld.toString()); //PRINT ArrayList of all held keys
        } //end of keyReleased
    };
    addKeyListener( keyListnr );
}

isKeyHeld:

public boolean isKeyHeld(int e)
{
 int keyCode = e;
 Object key = (Object)keyCode;

 if(!keysHeld.isEmpty())
 {
  int index = 0;
  while(index<keysHeld.size())
  {
   if(keysHeld.get(index) == key)
   {
    return true;
   }
   index++;
  }
 }
 return false;
}

Вывод на консоль: (удерживается стрелка влево [37], а затем нажимается стрелка вправо [39])

[37]
errorCheck: keysHeld: [37], 39 not found
errorCheck: keysHeld: [37], 39 not found
errorCheck: keysHeld: [37], 39 not found
errorCheck: keysHeld: [37], 39 not found
...

Ответы [ 2 ]

1 голос
/ 28 октября 2009

Пара баллов:

  • Вы не заполняете свой массив keysHeld экземплярами KeyEvent, но объектами autoBoxed Integer, полученными из int keyCodes.
  • Вам нужно увеличить переменную index, если вы хотите выйти из цикла while в keyPressed
  • Вы не должны использовать == для сравнения двух Objects в вашем while цикле

Вы можете проверить что-то вроде следующего:

    if(keysHeld.get(index++).equals(new Integer(keyCode))
0 голосов
/ 02 февраля 2013

При обработке нескольких клавиш лучше всего использовать метод keyReleased(KeyEvent): он упрощает обработку нескольких комбинаций клавиш во время отпускания ключа.

Что я заметил, так это то, что, находясь внутри keyPressed(), я получал только один ключевой символ. На keyReleased я смог захватить несколько символов (например, CTRL - V ).

...