이 키 수신기를 올바르게 구조화 했습니까?

Question

사용자가 에을 입력하면 내 GamePanel 클래스의 addShot() 메서드를 호출하기를 원하기 때문에 묻습니다. "이것은 우주선에서 발사 된 미사일을 나타내는 총격 개체를 초기화하지만 아무 것도하지 않습니다. 여기 저기 가시성 문제, 또는 그냥 키 이벤트와 청취자의 관계 잘못된 구조 않았다 나는 단지 관련 코드를 게시하도록하겠습니다 만, 필요한 경우 내가 나머지를 게시 할 수 있습니다 여기에

하면 코드입니다 :?.

public static class GameTest extends JFrame { 

    private static final int WINDOW_WIDTH = 800; 
    private static final int WINDOW_HEIGHT = 500; 
    public GamePanel gamePanel; 


    public GameTest() throws IOException { 
     super("Deep Fried Freedom"); 
     setResizable(false); 
     setDefaultCloseOperation(JFrame.EXIT_ON_CLOSE); 
     setSize(WINDOW_WIDTH, WINDOW_HEIGHT); 
     setLayout(new BorderLayout()); 
     gamePanel = new GamePanel(); 
     add(gamePanel); 
     center(this); 
     setVisible(true); 
     this.addKeyListener(new aKeyListener()); 
     this.setFocusable(true); 

    } // end constructor 

    public void center(JFrame frame) { 
     GraphicsEnvironment ge = GraphicsEnvironment.getLocalGraphicsEnvironment(); 
     Point center = ge.getCenterPoint(); 

     int w = frame.getWidth(); 
     int h = frame.getHeight(); 

     int x = center.x - w/2, y = center.y - h/2; 
     frame.setBounds(x, y, w, h); 
     frame.validate(); 
    }//end of center method 

    public class aKeyListener implements KeyListener { 

     @Override 
     public void keyTyped(KeyEvent e) { 
     }//end empty keyTyped method 

     @Override 
     public void keyPressed(KeyEvent e) { 
      switch (e.getKeyCode()) { 
       case KeyEvent.VK_A: 
        Launcher.lRun = -20; 
        gamePanel.move(gamePanel); 
        break; 
       case KeyEvent.VK_D: 
        Launcher.lRun = 20; 
        gamePanel.move(gamePanel); 
        break; 
       case KeyEvent.VK_ENTER: 
         gamePanel.addShot(); 
        break; 
       default: 
        Launcher.lRun = 0; 
      } 

     }//end keyPressed method 

     @Override 
     public void keyReleased(KeyEvent e) { 
     }//end empty keyReleased method 

    }//end aKeyListener class 

}//end GameTest class 

Answer 1

1. 너의 것이 초점 문제 일 가능성이 높다 - somet 힌트 GamePanel에서 아마도 훔치고 초점을 매우 KeyListeners에 일반적인 문제.
2. 일반 조언 :이 유형의 필요에 대한 KeyListener가 아니라 Key Bindings를 사용하는 것이 좋습니다.

유용한 링크 :

• Swing Resources
• StackOverflow의 질문 here에 대한 내 대답에 내 초 예제 프로그램을 확인하시기 바랍니다.
• Overall Swing Tutorials
• Key Bindings
• Swing Timer
• Concurrency in Swing

---

편집 예를 들어
:

import java.awt.Dimension; 
import java.awt.event.ActionEvent; 
import java.awt.event.KeyEvent; 

import javax.swing.*; 

public class GameTest extends JFrame { 

    private static final int WINDOW_WIDTH = 800; 
    private static final int WINDOW_HEIGHT = 500; 
    public GamePanel gamePanel; 

    public GameTest() { 
     super("Deep Fried Freedom"); 
     setResizable(false); 
     // setDefaultCloseOperation(JFrame.EXIT_ON_CLOSE); 
     setDefaultCloseOperation(JFrame.DISPOSE_ON_CLOSE); 
     // setSize(WINDOW_WIDTH, WINDOW_HEIGHT); // never do this 
     gamePanel = new GamePanel(); 
     setUpKeyBinding(gamePanel); 
     add(gamePanel); 
     pack(); 
     setLocationRelativeTo(null); // to center! 
     setVisible(true); 
    } 

    @Override 
    public Dimension getPreferredSize() { 
     return new Dimension(WINDOW_WIDTH, WINDOW_HEIGHT); 
    } 

    private void setUpKeyBinding(GamePanel gamePanel2) { 

     // only need window to have focus 
     int condition = JComponent.WHEN_IN_FOCUSED_WINDOW; 

     // get the GamePanel's InputMap and ActionMap as these will be used to set 
     // up our Key Bindings 
     InputMap inputMap = gamePanel2.getInputMap(condition); 
     ActionMap actionMap = gamePanel2.getActionMap(); 

     // bind the A key to move back 20 pixels 
     KeyStroke keyStroke = KeyStroke.getKeyStroke(KeyEvent.VK_A, 0); 
     inputMap.put(keyStroke, keyStroke.toString()); 
     actionMap.put(keyStroke.toString(), new MoveAction(gamePanel2, -20)); 

     // bind the D key to move forward 20 pixels 
     keyStroke = KeyStroke.getKeyStroke(KeyEvent.VK_D, 0); 
     inputMap.put(keyStroke, keyStroke.toString()); 
     actionMap.put(keyStroke.toString(), new MoveAction(gamePanel2, 20)); 

     // bind the ENTER key 
     keyStroke = KeyStroke.getKeyStroke(KeyEvent.VK_ENTER, 0); 
     inputMap.put(keyStroke, keyStroke.toString()); 
     actionMap.put(keyStroke.toString(), new EnterAction(gamePanel2)); 

    } 

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

    // our Action classes that will bound to key strokes 
    private class MoveAction extends AbstractAction { 
     private GamePanel gamePanel2; 
     private int distance; 

     public MoveAction(GamePanel gamePanel2, int distance) { 
     this.gamePanel2 = gamePanel2; 
     this.distance = distance; 
     } 

     @Override 
     public void actionPerformed(ActionEvent e) { 
     gamePanel2.moveItem(distance); 
     } 
    } 

    private class EnterAction extends AbstractAction { 
     private GamePanel gamePanel2; 

     public EnterAction(GamePanel gamePanel2) { 
     this.gamePanel2 = gamePanel2; 
     } 

     @Override 
     public void actionPerformed(ActionEvent e) { 
     gamePanel2.addShot(); 
     } 
    } 

} 

// a trivial GamePanel class to just show that everything is working as expected 
class GamePanel extends JPanel { 

    public void moveItem(int i) { 
     // your moveItem method will instead actually move something, a distance of i 
     System.out.println("Move Item Method distance: " + i); 
    } 

    public void addShot() { 
     // your addShot method will actually shoot 
     System.out.println("Add Shot Method"); 
    } 

}