repaintの処理

Question

コード
public class MyPanel extends JPanel{

	Player player1;
	int gun_x = -100;
	int gun_y = -100;

	/**
	 * MyPanelのコンストラクタ
	 */
	public MyPanel(){
		setBackground(Color.BLACK);		//背景色設定

		try{
			player1 = new Player("jiki.gif", 0, 0, 32, 32);
		}
		catch(IOException ioe){
			System.out.println("画像がありません");
		}
	}

	/**
	 * キー入力処理
	 */
	protected void processKeyEvent(KeyEvent ke){
		/*何かキーが押された場合*/
		if(ke.getID() == KeyEvent.KEY_PRESSED){
			System.out.println("キー" + ke.getKeyCode() + "が押された");

			player1.move1(ke.getKeyCode());

			move2(ke.getKeyCode());

			repaint();
		}
	}

	/**
	 * 描画メソッド
	 */
	protected void paintComponent(Graphics g){
		super.paintComponent(g);
		requestFocusInWindow();			//キー入力有効化

		player1.draw1(g);

		draw2(g);
	}

	/**
	 *
	 */
	public void draw2(Graphics g){
		g.setColor(Color.WHITE);
		g.fillRect(gun_x, gun_y, 5, 5);
	}

	/**
	 *
	 */
	public void move2(int keyCode){
		if(keyCode == KeyEvent.VK_ENTER){
			gun_x = player1.m_x;
			gun_y = player1.m_y;

			Thread thread = new Thread(new Runnable(){
				public void run(){
					for(int i = 0; i < 50; i++){
						try{
							Thread.sleep(50);
	                    }
	                    catch(InterruptedException e){
	                    }
						gun_y -= 10;
						repaint();
					}
				}
			});
			thread.start();
		}
	}
}

これはプログラムの一部です。
今このプログラムでは、ENTERキーを押すと、0.05秒ごとにy座標が10ずつ動く四角形を描画しています。
このmove2メソッドを下のように別クラスに移したいのですが、

コード
public class Gun{

	int gun_x = -100;
	int gun_y = -100;

	/**
	 *
	 */
	public void move2(int keyCode){
		if(keyCode == KeyEvent.VK_ENTER){
			gun_x = player1.m_x;
			gun_y = player1.m_y;

			Thread thread = new Thread(new Runnable(){
				public void run(){
					for(int i = 0; i < 50; i++){
						try{
							Thread.sleep(50);
	                    }
	                    catch(InterruptedException e){
	                    }
						gun_y -= 10;
						repaint();
					}
				}
			});
			thread.start();
		}
	}
}

このときrepaint();でエラーが出てしまいます。
MyPanelクラスにあったときと違ってGunクラスはJPanelを継承していないからだと思うのですがよくわかりませんでした。
repaintはどう対処したらいいのでしょうか？

Answer 1

いくつか対処方法はありますが、
方法１．インナークラスで実施する
MyPanel クラスの中にインナークラスを作成する

private class Gun {
    public void move2(int keyCode){
    ～～
    }
}

方法２．GunクラスのMoive2メソッドの引数にMyPanelを渡す

 public void move2(int keyCode, MyPanel panel){
  ～～
 panel.repaint();
 }

もし自分なら方法1のほうを実装すると思います。

本件とは関係ないので簡単に気になる点を挙げます。
・キーを連打した場合に速度がおかしくなると思います。
(スレッドの無駄な生成が発生する。)

Comments

[nakamura-]

回答ありがとうございます。
参考にしてどちらもやってみたいと思います！

確かに連打したとき速度がおかしくなりました。
ENTERキーを押すたびにスレッドが生成されてしまうからおかしくなるということでしょうか？

[Yossi_1982]

そういうことです。
スレッドが複数本立ち、gun_y -= 10が不必要に計算されているからです。

この辺はGoogleなどで「Java 再描画 無限ループ」などのキーワードで探してみると色々と対処方法が書いてあるページがあるので参考にしてみてください。

[nakamura-]

ご指摘ありがとうございました。
調べてみたいと思います！