ボタンを押した後のKeyEventの取得がうまくいかない

Question

現在ゲームの枠組みを作っています。MainPanelを用意し、そこにstartpanelやgamepanel,gameclearpanelなどをMainpanel上で入れ替えるといったプログラムを作りたいと思っています。初めはMainpanelにはstartpanelが載せられており、「ｓ」キーを押すとゲームがスタートしてgemapanelに行くようになっているのですが、StartPanelに設置した「遊び方」のボタン（押すと標準出力に遊び方と表示される。）を押すとkeyEventが取得されず、ｓキーを押してもゲームが始まらなくなってしまいます。原因がわかる方がいましたら、解決策を教えてください。

//GameFrame ファイル
import javax.swing.JFrame;

public class GameFrame extends JFrame {

	public GameFrame() {

		setLayout(null);
		setDefaultCloseOperation(JFrame.EXIT_ON_CLOSE);
		setSize(500,500);
		setLocationRelativeTo(null);
		MainPanel pn =new MainPanel();
		add(pn);
		setVisible(true);

	}

	public static void main(String[] args) {
		GameFrame GF=new GameFrame();



	}

}

//StartPanelファイル
import java.awt.Color;
import java.awt.Font;
import java.awt.FontMetrics;
import java.awt.Graphics;
import java.awt.event.ActionEvent;
import java.awt.event.ActionListener;
import java.awt.event.KeyAdapter;
import java.awt.event.KeyEvent;

import javax.swing.JButton;
import javax.swing.JPanel;

public class StartPanel extends JPanel implements ActionListener{
      boolean in_game=true;
      MainPanel mp;
      JButton bt;

      public StartPanel(MainPanel mp1) {

    	  mp=mp1;
    	  setLayout(null);
    	  setBackground(Color.white);
    	  Font f=new Font("SansSerif",Font.BOLD,20);
    	  FontMetrics fm=getFontMetrics(f);
    	  String str="遊び方";
    	  int w=fm.stringWidth(str)+50;
    	  int h=fm.getHeight()+20;
    	  bt=new JButton(str);
    	  bt.setFont(f);
    	  bt.addActionListener(this);
    	  bt.setSize(w,h);
    	  bt.setLocation(250-w/2,400);
    	  add(bt);
    	  addKeyListener(new Key());
    	  setFocusable(true);



      }
     @Override
      public void paint(Graphics g) {
    	  super.paint(g);
    	  FontMetrics fm;
    	  Font f;
    	  String str1,str="Game Title";
    	  int w,h;

    	  f=new Font("Serif",Font.BOLD,25);
    	  fm=g.getFontMetrics(f);
    	  str1=str+"(Serif)";
    	  w=fm.stringWidth(str1);
    	  h=fm.getHeight();
    	  g.setFont(f);
    	  g.drawString(str1,250-w/2,h);

    	  f=new Font("SansSerif",Font.BOLD,25);
    	  fm=g.getFontMetrics(f);
    	  str1=str+"(SansSerif)";
    	  w=fm.stringWidth(str1);
    	  h=h+fm.getHeight()+10;
    	  g.setFont(f);
    	  g.drawString(str1,250-w/2,h);


    	  f=new Font("Dialog",Font.BOLD,25);
    	  fm=g.getFontMetrics(f);
    	  str1=str+"(Dialog)";
    	  w=fm.stringWidth(str1);
    	  h=h+fm.getHeight()+10;
    	  g.setFont(f);
    	  g.drawString(str1,250-w/2,h);

    	  f=new Font("DialogInput",Font.BOLD,25);
    	  fm=g.getFontMetrics(f);
    	  str1=str+"(DialogInput)";
    	  w=fm.stringWidth(str1);
    	  h=h+fm.getHeight()+10;
    	  g.setFont(f);
    	  g.drawString(str1,250-w/2,h);

    	  f=new Font("Serif",Font.PLAIN,20);
    	  fm=g.getFontMetrics(f);
    	  str1="ゲーム開始: 「s」キー";
    	  w=fm.stringWidth(str1);
    	  h=500-fm.getHeight()-10;
    	  g.setFont(f);
    	  g.drawString(str1,250-w/2,h-200);
    	  //requestFocusInWindow();





      }

      public void actionPerformed(ActionEvent e) {
    	  if(e.getSource()==bt) {
    		 System.out.println("遊び方");
    	  }
      }

      class Key extends KeyAdapter{

    	  public void keyPressed(KeyEvent ke) {
    		  if(ke.getKeyCode()==KeyEvent.VK_S) {
    			  mp.state=1;

    		  }
    	  }
      }

}

//MainPanel ファイル
import java.awt.BorderLayout;

import javax.swing.JPanel;

public class MainPanel extends JPanel implements Runnable {
    StartPanel sp;
    GamePanel gp;
    GameClearPanel gcp;
    GameOverPanel gop;
    boolean in_game=true;//ゲーム実行中はtrue
    public   int state=0;//ゲーム状態
    public int level=1;//ゲームレベル
    int old_state=0;//直前のゲーム状態
    Thread td;
    int width=500;
    int hight=500;

    public MainPanel() {

    	setSize(width,hight);
    	setLayout(new BorderLayout());
    	sp=new StartPanel(this);
    	add(sp,BorderLayout.CENTER);
    	td=new Thread(this);
    	td.start();

    }

    public void run() {
    	while(in_game) {
    	try {
    		td.sleep(100);
    	}catch(InterruptedException e) {}
    	if(state!=old_state) {
    		if(old_state==0) {
    			remove(sp);
    		}
    			else if(old_state==1) {
    				remove(gp);
    			}
    			else if(old_state==2) {
    				remove(gcp);
    			}
    			else {
    				remove(gop);
    			}

    	//新しいパネルの追加
    	if (state==4) {
    		in_game=false;
    	}else if(state==0) {
    		sp=new StartPanel(this);
    		add(sp);
    	}else if(state==1) {
    		gp=new GamePanel(this);
    		add(gp);
    	}
    	else if(state==2) {
    		gcp=new GameClearPanel(this);
    		add(gcp);
    	}
    	else if(state==3){
    		gop=new GameOverPanel(this);
    		add(gop);
    	}
    	validate();
    	old_state=state;
    }
    	}
}
}

Comments

[退会済みユーザー]

どれ試そうかとったらコンパイルが通らないソースを貼るのは勘弁してください。
markdownのコード【```】で囲むのは、ソースファイル単位に区切ってください。
質問文の修正お願いします。

[etcetera]

すみません、これでどうでしょうか？

[退会済みユーザー]

GameClearPanel
GameOverPanel
GameLanel
を型に解決できません。

…とは出たんですが、今回はStartPanelのKey#keyPressed()に進めればいいと思うので、未定義エラーはコメントアウトします。

ってこれswingだからJDK8以前かな。今15使っているのでJDK入れ直して見てみますね。

[退会済みユーザー]

まだ読み切れていませんが、この現象が起きてる気がする
https://stackoverflow.com/questions/58014435/putting-a-keylistener-inside-of-an-actionlistener

Answer 1

とりあえず遊び方ボタン押下後にキーイベントが取れない理由は

• ① ボタンを押下したことにより、ボタンにフォーカスが移ってしまい
• ② パネル側のキーイベントがボタンに吸われるようになってしまった

のが原因のようです。

対症療法的にボタンにもキーイベントを仕掛けると、
ボタン押下後もキーイベントを拾い続ける事が出来ました。
このコードは根本解決している訳ではないのでちゃんとした対策が必要かと思いますが、
後は etcetera さんがどう直すかだと思います。

// ～（略）～

public StartPanel(MainPanel mp1) {

    // ～（略）～

    String str="遊び方";
    int w=fm.stringWidth(str)+50;
    int h=fm.getHeight()+20;
    bt=new JButton(str);
    bt.setFont(f);
    bt.addActionListener(this);
    bt.addKeyListener(new Key()); // これを追加
    bt.setSize(w,h);
    bt.setLocation(250-w/2,400);

    add(bt);
    addKeyListener(new Key());

    // ～（略）～
}

頑張ってください。

Comments

[etcetera]

軽く調べてみましたが、この方法が一番手っ取り早そうなので、これでいきたいと思います。ありがとうございました。

[m.ts10806]

そういう理由で採用した手法は大抵すぐこけます。
そのあたりも含めたアドバイスであることは認識されたほうが良いです＞質問者さん

[退会済みユーザー]

> etcetera さん
多分、この対症療法を採用した場合、今後コンポーネントを追加すると、追加したコンポーネント自身と、それ以外の操作する可能性のあるすべてのコンポーネントに追加でイベントを入れる必要が出てきます。
コンポーネント1つなら【1イベント】で済むけど、2つなら【4イベント】、3つなら【9イベント】と指数関数的に増えます。
他のゲーム画面にどれだけのコンポーネントがあるかは分かりませんが、ごくごく小規模なゲームを作るのが限界かと思います。リークの問題もありそうですし。

一つ鼻につくとしたら【軽く調べて～】の所ですが、
貴方のソースの書き方、swing系オブジェクトを格納した変数の状態、JDKソース、StackOverFlowなどのフォーラム、これらを総合的に見て【対症療法的】と私の経験と勘から言っている訳です。
私より調査、検証能力が高い人間に言われるならまだしも現時点で貴方が調べてたどり着けるとは思えません。
元々現象確認用のコードですしね。

初心者ごときが【本気で調べたって早々出てこない】です。んなもん頭で思ってても回答者に言わないで欲しいです。
失礼しました。

> m.ts10806さん
私が言いたい事を代弁して頂いてありがとうございます。
一度こけてみるのも勉強だと思うので良いかと思います。
仕事でこんなコード出して来たらキレますがｗ

[m.ts10806]

採用理由からして、こけたときに回答者のせいにしそうだなと懸念しての指摘でした。
厚意の場所ですが回答者は基本無責任ですからね。
選んだ質問者自身の責任において十分検証してもらいたいもんです。手法ファーストでいくとすぐ詰まりますし。