JFrameでのJPanelの切り替えについて

Question

###前提・実現したいこと
(例)Java(UIはJFrame)でゲームを作っているのですが、JPanelの切り替えをJFrameで行おうとしています。
しかし、JPanelの切り替えがうまくいきません。

なお、切り替えにはこのサイトのような切り替え方法を使っています。

###発生している問題

切り替え自体はどうやら成功しているようだが、パネルのボタンの表示がされない

###該当のソースコード
※作成中のゲームのため、一部関数を改変し、コメントはすべて除去した

package panels;

import java.awt.BorderLayout;
import java.awt.Container;
import java.awt.event.WindowAdapter;
import java.awt.event.WindowEvent;

import javax.swing.JFrame;
import javax.swing.JOptionPane;
import javax.swing.JPanel;

public class MainFrame extends JFrame{
	public String[] PanelNames = {"First","Second","Third"};
	FirstPanel fp = new FirstPanel(PanelNames[0]);
	SecondPanel sp = new SecondPanel(PanelNames[1]);
	ThirdPanel tp = new ThirdPanel(PanelNames[2]);
	public JPanel[] jps = {fp,sp,tp};

	int i;

	private static final int VERTICAL = 768;
	private static final int HORIZONTAL = 1024;

	public MainFrame(){
		super("Test");

		this.add(fp);fp.setVisible(true);
		this.add(sp);sp.setVisible(false);
		this.add(tp);tp.setVisible(false);

		setBounds(100, 100, 300, 250);

		setDefaultCloseOperation(JFrame.DO_NOTHING_ON_CLOSE);
		addWindowListener(new WindowClosing());

		setSize(HORIZONTAL,VERTICAL);

		setResizable(false);

		Container contentPane = getContentPane();
		contentPane.add(jps[i], BorderLayout.CENTER);

		setVisible(true);
	}

	class WindowClosing extends WindowAdapter{
		public void windowClosing(WindowEvent e) {
			int ans = JOptionPane.showConfirmDialog
					(MainFrame.this, "本当に終了しますか?");
			if(ans == JOptionPane.YES_OPTION) {
				System.exit(0);
			}
		}
	}

	public void changeKey(JPanel jp, String str){
		System.out.println(jp.getName());
		String name = jp.getName();
		if (name==PanelNames[0]){
			fp = (FirstPanel)jp;
			fp.setVisible(false);
			System.out.println("〇");
		} else if (name==PanelNames[1]){
			sp = (SecondPanel)jp;
			sp.setVisible(false);
			System.out.println("●");
		} else if (name==PanelNames[2]){
			tp = (ThirdPanel)jp;
			tp.setVisible(false);
			System.out.println("◎");
		} else {
			System.out.println("huh?");
		}


		if (str==PanelNames[0]){
			fp.setVisible(true);
			//iで切り替え！
			i = 0;
			System.out.println("△");
		} else if (str==PanelNames[1]){
			sp.setVisible(true);
			i = 1;
			System.out.println("▼");
		} else if (str==PanelNames[2]){
			tp.setVisible(true);
			i = 2;
			System.out.println("▲");
		} else {
			System.out.println("huh?");
		}
	}
}

package panels;

import java.awt.event.ActionEvent;
import java.awt.event.ActionListener;

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

package panels;

import java.awt.Font;
import java.awt.event.ActionEvent;
import java.awt.event.ActionListener;

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

public class FirstPanel extends JPanel implements ActionListener{
	String str;

	public FirstPanel(String s){
		str = s;
		this.setName("First");
		//setLocationRelativeTo(null);
		setLayout(null);

		JLabel titleLabel = new JLabel("FirstPanel");

		titleLabel.setBounds(270, 100, 512, 100);

		titleLabel.setFont(new Font("ＭＳ ゴシック", Font.BOLD, 64));
		titleLabel.setHorizontalTextPosition(JLabel.CENTER);

		JButton buttonOne = new JButton("One");
		buttonOne.setBounds(256, 350, 512, 50);
		buttonOne.addActionListener(this);
		buttonOne.setActionCommand("One");

		JButton buttonTwo = new JButton("Two");
		buttonTwo.setBounds(256, 400, 512, 50);
		buttonTwo.addActionListener(this);
		buttonTwo.setActionCommand("Two");

		JButton buttonThree = new JButton("Three");
		buttonThree.setBounds(256, 4050, 512, 50);
		buttonThree.addActionListener(this);
		buttonThree.setActionCommand("Three");

		JButton buttonFour = new JButton("Four");
		buttonFour.setBounds(256, 500, 512, 50);
		buttonFour.addActionListener(this);
		buttonFour.setActionCommand("Four");

		JButton buttonFive = new JButton("Five");
		buttonFive.setBounds(256, 550, 512, 50);
		buttonFive.addActionListener(this);
		buttonFive.setActionCommand("Five");

		add(titleLabel);
		add(buttonOne);
		add(buttonTwo);
		add(buttonThree);
		add(buttonFour);
		add(buttonFive);

	}


	@Override
	public void actionPerformed(ActionEvent e) {
		String cmd = e.getActionCommand();
		System.out.println(cmd);

		switch(cmd){

		case "One":
			break;
		case "Two":
			break;
		case "Three":
			break;
		case "Four":
			break;
		case "Five":
			System.exit(0);
			break;

			default:
				//何もない場合はreturnで切る
				return;
		}

	}
}
※SecondPanel、ThirdPanelはこれと同じなので省略

###試したこと

• SecondPanelのみをMainにして起動→動作する
• MainFrameのFirstPanelをfalseにしてSecondPanelをtrueにする→動作しない(ThirdPanelでも動作しない)
• 切り替え時にコンソール上に文字を表記→切り替え時の文字は表記される
• FirstPanelからFirstPanelに転送→表示される

###補足情報(言語/FW/ツール等のバージョンなど)
Java8(Eclipse)

Comments

[umyu]

多分、行いたいことはゲームのシーン遷移だと思いますがー。。質問文のコードではコンパイルエラーになります！ 1,FirstPanelのコンストラクタが一致してません。2,MainFrameコンストラクタのsuper(Main.title);3,同コンストラクタdp.setVisible(false);

[Hinantius246]

失礼しました…

Answer 1

JPanelの切り替えにCardLayoutを使用。

MainFrame クラス

package panels;

import java.awt.Container;
import java.awt.event.WindowAdapter;

import java.awt.CardLayout;
import java.awt.event.WindowEvent;

import javax.swing.JFrame;
import javax.swing.JOptionPane;
import javax.swing.JPanel;

import java.util.Arrays;

public class MainFrame extends JFrame {
	public String[] PanelNames = { "First", "Second", "Third" };
	JPanel fp = new FirstPanel(this, PanelNames[0]);
	JPanel sp = new SecondPanel(this, PanelNames[1]);
	JPanel tp = new ThirdPanel(this, PanelNames[2]);
	public JPanel[] jps = { fp, sp, tp };
	private final CardLayout layout = new CardLayout();//カードレイアウト
	
	private static final int VERTICAL = 768;
	private static final int HORIZONTAL = 1024;
	
	public MainFrame() {
		super("Test");
		setBounds(100, 100, 300, 250);
		setDefaultCloseOperation(JFrame.DO_NOTHING_ON_CLOSE);
		addWindowListener(new WindowClosing());
		setSize(HORIZONTAL, VERTICAL);
		// CardLayoutを使用。
		Container contentPane = getContentPane();
		JPanel mainPanel = new JPanel();
		mainPanel.setLayout(layout);
		for (JPanel panel : jps){
			mainPanel.add(panel, panel.getName());
		}
		contentPane.add(mainPanel);
		
		setResizable(false);
		setVisible(true);
	}

	class WindowClosing extends WindowAdapter {
		public void windowClosing(WindowEvent e) {
			int ans = JOptionPane.showConfirmDialog(MainFrame.this, "本当に終了しますか?");
			if (ans == JOptionPane.YES_OPTION) {
				System.exit(0);
			}
		}
	}

	// changeKey=>changePanelにメソッド名を変更して、引数を1個に
	public void changePanel(String str) {
		int index = Arrays.asList(PanelNames).indexOf(str);
		layout.show(jps[index].getParent(), jps[index].getName());
	}
}

FirstPanelクラス

package panels;

import java.awt.Font;
import java.awt.event.ActionEvent;
import java.awt.event.ActionListener;

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

public class FirstPanel extends JPanel implements ActionListener{
    private final String str;
    private final MainFrame frame; // インスタンス変数でMainFrameの参照を保持
    public FirstPanel(MainFrame f, String s){
        str = s;
        frame = f;
        this.setName(s);//固定文字列ではなく、コンストラクタで渡された文字列に変更
        //setLocationRelativeTo(null);
        setLayout(null);

        JLabel titleLabel = new JLabel(s);//固定文字列ではなく、コンストラクタで渡された文字列に変更

        titleLabel.setBounds(270, 100, 512, 100);

        titleLabel.setFont(new Font("ＭＳ ゴシック", Font.BOLD, 64));
        titleLabel.setHorizontalTextPosition(JLabel.CENTER);

        JButton buttonOne = new JButton("One");
        buttonOne.setBounds(256, 350, 512, 50);
        buttonOne.addActionListener(this);
        buttonOne.setActionCommand("One");

        JButton buttonTwo = new JButton("Two");
        buttonTwo.setBounds(256, 400, 512, 50);
        buttonTwo.addActionListener(this);
        buttonTwo.setActionCommand("Two");

        JButton buttonThree = new JButton("Three");
        // setBounds の引数y 4050=> 450
        buttonThree.setBounds(256, 450, 512, 50);
        buttonThree.addActionListener(this);
        buttonThree.setActionCommand("Three");

        JButton buttonFour = new JButton("Four");
        buttonFour.setBounds(256, 500, 512, 50);
        buttonFour.addActionListener(this);
        buttonFour.setActionCommand("Four");

        JButton buttonFive = new JButton("Five");
        buttonFive.setBounds(256, 550, 512, 50);
        buttonFive.addActionListener(this);
        buttonFive.setActionCommand("Five");

        add(titleLabel);
        add(buttonOne);
        add(buttonTwo);
        add(buttonThree);
        add(buttonFour);
        add(buttonFive);

    }


    @Override
    public void actionPerformed(ActionEvent e) {
        String cmd = e.getActionCommand();
        System.out.println(cmd);

        switch(cmd){

        case "One":
        	//MainFrame#changePanelを呼び出す。
        	frame.changePanel("First");
            break;
        case "Two":
        	frame.changePanel("Second");
            break;
        case "Three":
            break;
        case "Four":
        	frame.changePanel("Third");
            break;
        case "Five":
            System.exit(0);
            break;

            default:
                //何もない場合はreturnで切る
                return;
        }

    }
}

---

MainFrame#changeKeyメソッドでFirstPanelをsetVisible(false);にすると親のcontentPaneが表示されて遷移したように見えるだけなのです。

Swingのコンポーネントのデバックをする時は、
JFrameをクリックしてフォーカスを当てた状態でCTRL+Shit+F1キーを押下すると、コンソールにSwingのコンポーネントツリーが出力されます。

---

画面遷移の管理について
0. 遷移を管理するためのシーンクラスを作成（シングルトン）
0. 画面の遷移元と遷移先をツリー構造で管理する。

JFrameを継承したクラスにいろいろな処理を書くのは個人的に好きではないのでこれがベターだと思います。

Comments

[Hinantius246]

ありがとうございました。これで動くようになりました。CardLayoutも一度は考えたのですが難しくてやめてしまいましたが、これでできるようになりました。これからも頑張りたいです。