キー入力を１回だけ行う方法

テトリスを作っています。

コード
public class KeyInput implements KeyListener{

	boolean keyUp;			//上移動キー
	boolean keyDown;		//下移動キー
	boolean keyLeft;		//左移動キー
	boolean keyRight;		//右移動キー

	/*コンストラクタ*/
	public KeyInput(){
		keyUp 		= false;
		keyDown		= false;
		keyLeft 	= false;
		keyRight 	= false;
	}

	/*キーが押されたときに呼ばれるメソッド*/
	@Override
	public void keyPressed(KeyEvent e){

		/**キー取得*/
		int keycode = e.getKeyCode();

		if(keycode == KeyEvent.VK_UP){
			keyUp = true;
		}
		if(keycode == KeyEvent.VK_DOWN){
			keyDown = true;
		}
		if(keycode == KeyEvent.VK_LEFT){
			keyLeft = true;
		}
		if(keycode == KeyEvent.VK_RIGHT){
			keyRight = true;
		}
	}

	/*キーを離されたときに呼ばれるメソッド*/
	@Override
	public void keyReleased(KeyEvent e){

		/**キー取得*/
		int keycode = e.getKeyCode();

		if(keycode == KeyEvent.VK_UP){
			keyUp = false;
		}
		if(keycode == KeyEvent.VK_DOWN){
			keyDown = false;
		}
		if(keycode == KeyEvent.VK_LEFT){
			keyLeft = false;
		}
		if(keycode == KeyEvent.VK_RIGHT){
			keyRight = false;
		}
	}

	@Override
	public void keyTyped(KeyEvent e){
	}

}

上下左右キーを押したらtrue、離すとfalseになるようなっています。

また別のクラスで下のようなプログラムを書いています。

コード
  public void run(){
		while(true){
			repaint();

			try{
				Thread.sleep(20);
			}
			catch(InterruptedException e){
			}
		}
	}


	/**描画処理*/
	@Override
	protected void paintComponent(Graphics g){
		super.paintComponent(g);

		if(keyInput.keyUp == true){
			//処理なし
		}
		if(keyInput.keyDown == true){
			objectPool.moveBlock(0, 1);
		}
		if(keyInput.keyLeft == true){
			objectPool.moveBlock(-1, 0);
		}
		if(keyInput.keyRight == true){
			objectPool.moveBlock(1, 0);
		}
		break;
	}
       

public void moveBlock(int X, int Y){

		int postX = x + X;
		int postY = y + Y ;

		/**Panel内移動可、外は移動不可*/
		if(0 <= postX && postX <= panelW - width){
			x = postX;
		}
		if(0 < postY && postY < panelH - height){
			y = postY;
		}	
}

この二つのプログラムについてですが、
例えば左キーを１回押すと左に１だけ動いてほしいのですが、このプログラムだと３，４ほど左に動いてしまいます。
この原因はなんなのでしょうか？
また改善方法を教えていただきたいです。
よろしくお願いします。

行動規範の内容に同意します

回答2件

ベストアンサー

paintComponentは「再描画が必要になった際に随時呼び出される」メソッドです。

あなたのコードでは「再描画のメソッド（つまりいついかなる時に呼び出されるかも知れないメソッド）で状態の変更（つまりmoveBlockを呼び出すこと」をしています。そこが問題だと言えます。

moveBlockを呼び出すべきタイミングは「ブロックを移動すべきことがわかったとき」でありそれは「キーが押されたとき」であって「再描画するとき」ではありません。

再描画(paintComponentなど)では「現在のコンポーネントの状態を変化させずに描画だけをする」という考え方で設計するとよいと思います。状態を変えるのはアニメーションであれば「一定時間ごとに定期的イ呼び出されるイベントハンドラー」でありユーザーの操作により状態を変えるなら「ユーザーの操作によって発生するイベントのハンドラー」で行うのが適切です。

投稿2017/03/03 07:32

KSwordOfHaste

総合スコア18394

nakamura-

2017/03/09 02:07

回答ありがとうございます。確かにそうですね！イベントのハンドラというのはどういうことでしょうか？下のようにキーが押されたらブロック移動の処理を行うようにするということでしょうか？ public void keyPressed(KeyEvent e){ /**キー取得*/ int keycode = e.getKeyCode(); if(keycode == KeyEvent.VK_UP){ //objectPool.moveBlock(0, -1); } if(keycode == KeyEvent.VK_DOWN){ objectPool.moveBlock(0, 1); }