質問をすることでしか得られない、回答やアドバイスがある。

15分調べてもわからないことは、質問しよう!

ただいまの
回答率

88.63%

AWTの画面遷移について

解決済

回答 1

投稿 編集

  • 評価
  • クリップ 0
  • VIEW 1,176

kubo_

score 13

前提・実現したいこと

JavaでAWTを使い、っトランプゲームを作っています。
その中で、タイトル画面からメニュー画面に遷移するボタンがあるのですが押すと遷移するものの、タイトル画面に表示されていたボタンが消えず、本来メニュー画面に表示されるべきものにプラスしてタイトル画面で出ていたボタンが表示されたままになってしまいます。
removeAll()を追加するなど試しましたが変わりませんでした。

発生している問題・エラーメッセージ

画面遷移後、前の画面で表示していたボタンが消えず、遷移後の画面にも表示されてしまいます。

該当のソースコード

※まだプロトタイプ実装途中のため、addActionListenerが一部thisのままになっています。

package test;

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

import javax.swing.BoxLayout;
import javax.swing.JButton;
import javax.swing.JFrame;
import javax.swing.JLabel;
import javax.swing.JPanel;
import javax.swing.JTextField;

public class Title extends JFrame implements ActionListener {
    public static void main(String args[]) {
        new Title();
    }

    public Title() {
        setTitle("トランプコレクション");
        setDefaultCloseOperation(JFrame.EXIT_ON_CLOSE);

        setSize(1280, 720);
        setLocationRelativeTo(null);

        getContentPane().add(getDefPanel());
        setVisible(true);
    }

    private JPanel getDefPanel() {

        JPanel panel = new JPanel();

        String msg0 = "トランプコレクション";
        JLabel label = new JLabel(msg0);
        label.setFont(new Font("MS ゴシック", Font.PLAIN, 60));
        label.setBounds(350, 100, 1600, 100);
        label.setLayout(new BoxLayout(label, BoxLayout.PAGE_AXIS));
        this.add(label);
        JButton stert = new JButton("Start");
        stert.setFont(new Font("Century", Font.ITALIC, 30));
        stert.setSize(200, 50);
        stert.setLocation(400, 500);

        stert.addActionListener(new MenuListener(this));
        this.add(stert);
        JButton end = new JButton("End");
        end.setFont(new Font("Century", Font.ITALIC, 30));
        end.setSize(200, 50);
        end.setLocation(700, 500);
        end.addActionListener(new ExitListener());
        this.add(end);

        return panel;
    }



    @Override
    public void actionPerformed(ActionEvent e) {
        // TODO 自動生成されたメソッド・スタブ

    }


}
package test;

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

import javax.swing.BoxLayout;
import javax.swing.JButton;
import javax.swing.JFrame;
import javax.swing.JLabel;
import javax.swing.JPanel;
import javax.swing.JTextField;

class MenuListener implements ActionListener {
    private JFrame f;
    public MenuListener(JFrame f) {
        this.f=f;
    }
    public void actionPerformed(ActionEvent e) {
        // TODO 自動生成されたメソッド・スタブ

        f.setTitle("トランプコレクション");
        f.setDefaultCloseOperation(JFrame.EXIT_ON_CLOSE);

        f.setSize(1280, 720);
        f.setLocationRelativeTo(null);

        f.getContentPane().add(getPanel());
        f.setVisible(true);
    }
    private JPanel getPanel() {

        JLabel title = new JLabel("トランプコレクション");
        JPanel panel=new JPanel();
        JButton button=new JButton();
        title.setLayout(new BoxLayout(title, BoxLayout.PAGE_AXIS));
        title.setFont(new Font("MS ゴシック", Font.PLAIN, 60));
        title.setBounds(350, 100, 1600, 100);
        panel.add(title);
        // panel.add(new JLabel("トランプコレクション"));

        button = new JButton("神経衰弱");
        button.setSize(120, 50);
        button.setLocation(300, 300);
        button.setActionCommand("3");
        button.addActionListener(this);
        JButton button2 = new JButton("ポーカー");
        button2.setSize(120, 50);
        button2.setLocation(300, 400);
        button2.setActionCommand("3");
        button2.addActionListener(this);
        JButton button3 = new JButton("ソリティア");
        button3.setSize(120, 50);
        button3.setLocation(300, 500);
        button3.setActionCommand("3");
        button3.addActionListener(this);
        JButton button4 = new JButton("タイトルに戻る");
        button4.setSize(150, 50);
        button4.setLocation(550, 600);
        button4.addActionListener(this);
        JTextField txt = new JTextField("基本自分の記憶力が勝負を左右するゲームです。");
        txt.setSize(600, 50);
        txt.setLocation(500, 300);
        txt.setEditable(false);
        JTextField ptxt = new JTextField("5枚の手札の組み合わせで、カードの強さをきそうトランプゲームです。");
        ptxt.setSize(600, 50);
        ptxt.setLocation(500, 400);
        ptxt.setEditable(false);
        JTextField stxt = new JTextField("並べられたカードを数字の順番で列を移動させながら、同じマークのAからKの順番に並べかえるゲームです。");
        stxt.setSize(600, 50);
        stxt.setLocation(500, 500);
        stxt.setEditable(false);
        JTextField sp = new JTextField(" ");
        sp.setEditable(false);
        // txt.setLayout(new BoxLayout(panel, BoxLayout.Y_AXIS));
        panel.removeAll();
        panel.add(title);
        panel.add(button);
        panel.add(button2);
        panel.add(button3);
        panel.add(button4);
        panel.add(txt);
        panel.add(ptxt);
        panel.add(stxt);
        panel.add(sp);

        return panel;

    }
}
package test;

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

class ExitListener implements ActionListener {
    @Override
    public void actionPerformed(ActionEvent e) {
        System.exit(0);
    }
}

試したこと

removeAll()を追加するなど試しましたが変わりませんでした。

補足情報(FW/ツールのバージョンなど)

java 1.8.0

  • 気になる質問をクリップする

    クリップした質問は、後からいつでもマイページで確認できます。

    またクリップした質問に回答があった際、通知やメールを受け取ることができます。

    クリップを取り消します

  • 良い質問の評価を上げる

    以下のような質問は評価を上げましょう

    • 質問内容が明確
    • 自分も答えを知りたい
    • 質問者以外のユーザにも役立つ

    評価が高い質問は、TOPページの「注目」タブのフィードに表示されやすくなります。

    質問の評価を上げたことを取り消します

  • 評価を下げられる数の上限に達しました

    評価を下げることができません

    • 1日5回まで評価を下げられます
    • 1日に1ユーザに対して2回まで評価を下げられます

    質問の評価を下げる

    teratailでは下記のような質問を「具体的に困っていることがない質問」、「サイトポリシーに違反する質問」と定義し、推奨していません。

    • プログラミングに関係のない質問
    • やってほしいことだけを記載した丸投げの質問
    • 問題・課題が含まれていない質問
    • 意図的に内容が抹消された質問
    • 過去に投稿した質問と同じ内容の質問
    • 広告と受け取られるような投稿

    評価が下がると、TOPページの「アクティブ」「注目」タブのフィードに表示されにくくなります。

    質問の評価を下げたことを取り消します

    この機能は開放されていません

    評価を下げる条件を満たしてません

    評価を下げる理由を選択してください

    詳細な説明はこちら

    上記に当てはまらず、質問内容が明確になっていない質問には「情報の追加・修正依頼」機能からコメントをしてください。

    質問の評価を下げる機能の利用条件

    この機能を利用するためには、以下の事項を行う必要があります。

質問への追記・修正、ベストアンサー選択の依頼

  • kubo_

    2019/01/23 19:30

    はい、そうです。
    thisからpanelに直して実行しましたがボタンの位置やラベルの位置がずれるだけで結果は同じでした

    キャンセル

  • jimbe

    2019/01/23 19:31

    なるほど, 了解しました. 構造的な問題もあるので, 全体の書き直し的な回答でも良いですか?

    キャンセル

  • kubo_

    2019/01/23 19:33

    はい、大丈夫です。
    よろしくお願いします

    キャンセル

回答 1

checkベストアンサー

+1

構造を見直して, 1本に纏めました.

元あった getDefPanel を createTitlePanel に変更しています.
MenuListener にあった getPanel を createSelectPanel としてこちらに移動しました.
上記2つのパネルを CardLayout で重ねておき, 初期起動時はタイトル, Startボタンでセレクト, セレクトの"タイトルに戻る"ボタンでタイトルを(切り替え)表示するようにしました.
各ボタンの処理が短くなったため, 無名クラスとしてボタン生成箇所に埋め込むことで MenuListener, ExitListener が要らなくなりました.

package test;

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

import javax.swing.BoxLayout;
import javax.swing.JButton;
import javax.swing.JFrame;
import javax.swing.JLabel;
import javax.swing.JPanel;
import javax.swing.JTextField;

public class MainFrame extends JFrame {
    public static void main(String args[]) {
        new MainFrame();
    }

    private JPanel cardPanel;
    private static final String TITLE_PANEL = "title";
    private static final String SELECT_PANEL = "select";

    public MainFrame() {
        setTitle("トランプコレクション");
        setDefaultCloseOperation(JFrame.EXIT_ON_CLOSE);

        setSize(1280, 720);
        setLocationRelativeTo(null);

        //複数のパネルを乗せ, 名前で切り替える土台となるパネル
        cardPanel = new JPanel();
        cardPanel.setLayout(new CardLayout()); //CardLayoutは重なったパネルを切り替えることができます.

        //1枚目
        JPanel titlePanel = createTitlePanel();
        cardPanel.add(titlePanel, TITLE_PANEL);

        //2枚目
        JPanel selectPanel = createSelectPanel();
        cardPanel.add(selectPanel, SELECT_PANEL);

        //起動時は一枚目を表示
        ((CardLayout)(cardPanel.getLayout())).show(cardPanel, TITLE_PANEL);

        getContentPane().add(cardPanel);
        setVisible(true);
    }

    private JPanel createTitlePanel() {

        JPanel panel = new JPanel(null);

        String msg0 = "トランプコレクション";
        JLabel label = new JLabel(msg0);
        label.setFont(new Font("MS ゴシック", Font.PLAIN, 60));
        label.setBounds(350, 100, 1600, 100);
        label.setLayout(new BoxLayout(label, BoxLayout.PAGE_AXIS));
        panel.add(label);

        JButton stert = new JButton("Start");
        stert.setFont(new Font("Century", Font.ITALIC, 30));
        stert.setSize(200, 50);
        stert.setLocation(400, 500);
        stert.addActionListener(new ActionListener() {
            @Override
            public void actionPerformed(ActionEvent e) {
                //selPanel を表示します
                ((CardLayout)(cardPanel.getLayout())).show(cardPanel, SELECT_PANEL);
            }
        });
        panel.add(stert);

        JButton end = new JButton("End");
        end.setFont(new Font("Century", Font.ITALIC, 30));
        end.setSize(200, 50);
        end.setLocation(700, 500);
        end.addActionListener(new ActionListener() { //こういった定型的で短い処理の場合は省略形で定義します.
            @Override
            public void actionPerformed(ActionEvent e) {
                System.exit(0);
            }
        });
        panel.add(end);

        return panel;
    }

    private JPanel createSelectPanel() {

        JLabel title = new JLabel("トランプコレクション");
        JPanel panel=new JPanel(null);
        JButton button=new JButton();
        title.setLayout(new BoxLayout(title, BoxLayout.PAGE_AXIS));
        title.setFont(new Font("MS ゴシック", Font.PLAIN, 60));
        title.setBounds(350, 100, 1600, 100);
        panel.add(title);
        // panel.add(new JLabel("トランプコレクション"));

        button = new JButton("神経衰弱");
        button.setSize(120, 50);
        button.setLocation(300, 300);
        button.setActionCommand("3");
        //button.addActionListener(this);
        JButton button2 = new JButton("ポーカー");
        button2.setSize(120, 50);
        button2.setLocation(300, 400);
        button2.setActionCommand("3");
        //button2.addActionListener(this);
        JButton button3 = new JButton("ソリティア");
        button3.setSize(120, 50);
        button3.setLocation(300, 500);
        button3.setActionCommand("3");
        //button3.addActionListener(this);
        JButton button4 = new JButton("タイトルに戻る");
        button4.setSize(150, 50);
        button4.setLocation(550, 600);
        button4.addActionListener(new ActionListener() {
            @Override
            public void actionPerformed(ActionEvent e) {
                //defPanel を表示します
                ((CardLayout)(cardPanel.getLayout())).show(cardPanel, TITLE_PANEL);
            }
        });
        JTextField txt = new JTextField("基本自分の記憶力が勝負を左右するゲームです。");
        txt.setSize(600, 50);
        txt.setLocation(500, 300);
        txt.setEditable(false);
        JTextField ptxt = new JTextField("5枚の手札の組み合わせで、カードの強さをきそうトランプゲームです。");
        ptxt.setSize(600, 50);
        ptxt.setLocation(500, 400);
        ptxt.setEditable(false);
        JTextField stxt = new JTextField("並べられたカードを数字の順番で列を移動させながら、同じマークのAからKの順番に並べかえるゲームです。");
        stxt.setSize(600, 50);
        stxt.setLocation(500, 500);
        stxt.setEditable(false);
        JTextField sp = new JTextField(" ");
        sp.setEditable(false);
        // txt.setLayout(new BoxLayout(panel, BoxLayout.Y_AXIS));
        panel.removeAll();
        panel.add(title);
        panel.add(button);
        panel.add(button2);
        panel.add(button3);
        panel.add(button4);
        panel.add(txt);
        panel.add(ptxt);
        panel.add(stxt);
        panel.add(sp);

        return panel;

    }
}

投稿

編集

  • 回答の評価を上げる

    以下のような回答は評価を上げましょう

    • 正しい回答
    • わかりやすい回答
    • ためになる回答

    評価が高い回答ほどページの上位に表示されます。

  • 回答の評価を下げる

    下記のような回答は推奨されていません。

    • 間違っている回答
    • 質問の回答になっていない投稿
    • スパムや攻撃的な表現を用いた投稿

    評価を下げる際はその理由を明確に伝え、適切な回答に修正してもらいましょう。

  • 2019/01/23 20:18

    ご質問の問題は各部品の登録先が混乱していたことです.
    まず, getDefPanel で this に対して add していたこと, それから JFrame の ContentPane の Layout を意識していなかった為と思われます.
    JFrame.getContentPane() で得られるペインのレイアウトは BorderLayout であり, 追加情報が無い状態では中央に重ねられたと思います.
    そのため, 二つが重なって表示されたものと推測しました.
    このコードでは, コメントを入れておきましたが, JFrame(contentPane) の上に CardLayout の JPanel を乗せ, そのパネルに2枚のJPanel を乗せておいて切り替える形にしました.

    キャンセル

  • 2019/01/23 20:27 編集

    すいません, 2つのパネルの配置が乱れたままでした. JPanel のコンストラクタに null を渡すと Layout を無効に出来, 各部品の setLocation() 等が効くようになります.
    本来はレイアウトの機能を使って配置することで, ウインドウのサイズ変更等に対応するようにするのですけれど.

    キャンセル

  • 2019/01/24 08:49

    ありがとうございます。
    無事動作しました。
    今回教えていただいたことを今後に活かしていきたいと思います。
    ありがとうございました。

    キャンセル

  • 2019/01/24 12:37

    構造を変えてしまい申し訳ありません. kubo_ さんのプログラムですから回答としてあまり好ましくない方法だとは分かっているのですが.

    言い訳ですが, クラスファイルを分ける場合は Listener ではなく JFrame や JPanel 等で分けると, 各画面やパネル毎の処理がそれぞれのクラスに入るので分かりやすくなります. Listener の行う処理は, それが登録されるボタンやメニュー, さらのそのボタンやメニューが属するパネルやフレームに関する処理を行う場合が殆どですので, それらのクラスファイルに書いておくほうが良いと思います.
    今回は行いませんでしたが, この意味では createTitlePanel や createSelectPanel で作っているパネルをクラスとして抜き出し, TitlePanel.java として class TitlePanel extends JPanel {}, SelectPanel として class SelectPanel extends JPanel {}なファイルを作ってそれぞれのコンストラクタでボタンやテキストを配置するようにするともっと良いかと思います. ( selectPanel の'タイトルに戻る'ボタンのように 別のクラスのオブジェクト/操作が必要なら, コンストラクタで渡したりで連携します)
    言い訳も長くてすいません.

    キャンセル

15分調べてもわからないことは、teratailで質問しよう!

  • ただいまの回答率 88.63%
  • 質問をまとめることで、思考を整理して素早く解決
  • テンプレート機能で、簡単に質問をまとめられる

関連した質問

同じタグがついた質問を見る