他クラスで設定したメソッドを呼び出しているが機能していない

Question

前提

JavaのGUI（Swing）を使ってポートフォリオを作成しています。
GamePanelクラスで記述したresetGameメソッドを他クラスで呼び出しているつもりですが
機能しません。（GamePanelクラス中では機能しているようです。）

実現したいこと

TitlePanelクラス中でresetGameメソッドを呼び出し機能させスペースキーで画面遷移するたびに
GamePanelが初期化されているようにしたいです。

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

該当のソースコード

package mainPackage;

import java.awt.BorderLayout;
import java.awt.Color;
import java.awt.event.ActionEvent;
import java.awt.event.ActionListener;
import java.awt.event.KeyEvent;
import java.awt.event.KeyListener;

import javax.swing.JPanel;
import javax.swing.Timer;

public class GamePanel extends JPanel implements ActionListener{
	private static final long serialVersionUID = 1L;
	
	//フィールド
	int restTime;
	Timer timer;
	
	//レイアウト
	BorderLayout layout = new BorderLayout();
	//コンポーネント
	MenuBar menuBar;
	FieldPanel fieldPanel;
	
	//リスナー
	MyKeyListener myKeyListener;
	
	GamePanel(){
		this.setLayout(layout);
		this.setBackground(Color.yellow);
		//パネルの生成と設置
		menuBar = new MenuBar();
		fieldPanel = new FieldPanel();
		timer = new Timer(60000,this);
		this.add(menuBar,BorderLayout.NORTH);
		this.add(fieldPanel,BorderLayout.CENTER);
		timer.start();	
	}
	
	public void prepareComponents() {
		
		menuBar.prepareComponents();
		fieldPanel.prepareComponents();
		
		myKeyListener = new MyKeyListener(this);
	}
	
	public void resetGame() {
		this.fieldPanel.removeAll();
		this.fieldPanel.prepareComponents();
	}
	
	public void actionPerformed(ActionEvent e) {
		Main.mainWindow.setFrontScreenAndFocus(ScreenMode.TITLE);
		resetGame();
	}
	
	//内部クラス
	private class MyKeyListener implements KeyListener{
		//貼り付け先の保持
		JPanel panel;
		
		//コンストラクタ
		MyKeyListener(JPanel p){
			super();
			panel = p;
			panel.addKeyListener(this);
		}
		@Override
		public void keyTyped(KeyEvent e) {
			//do nothing
		}
		@Override
		public void keyReleased(KeyEvent e) {
			//do nothing
		}
		@Override
		public void keyPressed(KeyEvent e) {
			switch(e.getKeyCode()) {
			case KeyEvent.VK_H://Hを押したとき
				Main.mainWindow.setFrontScreenAndFocus(ScreenMode.TITLE);
				resetGame();
			}
		}
	}
}

package mainPackage;

import java.awt.Color;
import java.awt.Font;
import java.awt.event.KeyEvent;
import java.awt.event.KeyListener;

import javax.swing.BorderFactory;
import javax.swing.JLabel;
import javax.swing.JPanel;
import javax.swing.SwingConstants;
import javax.swing.border.Border;

public class TitlePanel extends JPanel{
	private static final long serialVersionUID = 1L;
	
	JLabel title;
	JLabel start;
	JLabel exit;
	JLabel select;
	JLabel message;
	Menu checkMenu = Menu.START;
	Border border = BorderFactory.createLineBorder(Color.BLACK,2);//消してもよい
	MyKeyListener myKeyListener;
	GamePanel gamePanel = new GamePanel();
	FieldPanel fieldPanel;
	//列挙型
	public enum Menu{
		START,
		EXIT,
	}
	
	TitlePanel(){
		
		this.setLayout(null);
		this.setBackground(Color.cyan);
	}
	
	public void prepareComponents() {
		title = new JLabel();
		
		
		
		//ImageIcon titleLogo = new ImageIcon(getClass().getClassLoader()getResource("title.png"));//600×300の画像を想定
		//title.setIcon(titleLogo);
		title.setHorizontalAlignment(SwingConstants.CENTER);
		title.setVerticalAlignment(SwingConstants.BOTTOM);
		title.setText("Created by 【TAKAYUKI H】");
		title.setHorizontalTextPosition(JLabel.CENTER);
		title.setVerticalTextPosition(SwingConstants.BOTTOM);
		title.setBounds(90,0,600,350);
		title.setBorder(border);
		//this.add(titleLabel);
		
		//選択肢
		start = new JLabel();
		start.setText("START");
		start.setFont(new Font("MV boli",Font.BOLD,40));
		start.setHorizontalTextPosition(JLabel.CENTER);
		start.setVerticalTextPosition(JLabel.BOTTOM);
		start.setBounds(330,400,150,40);
		start.setBorder(border);//縁取り
		
		exit = new JLabel();
		exit.setText("EXIT");
		exit.setFont(new Font("MV boli",Font.BOLD,40));
		exit.setHorizontalTextPosition(JLabel.CENTER);
		exit.setVerticalTextPosition(JLabel.BOTTOM);
		exit.setBounds(350,450,110,40);
		exit.setBorder(border);//縁取り
		
		//選択アイコン
		select = new JLabel();
		//ImageIcon titleLogo = new ImageIcon(getClass().getClassLoader()getResource("some.png"));
				//title.setIcon(selectIcon);
		select.setBackground(Color.blue);
		select.setOpaque(true);
		select.setBounds(280,400,40,40);
		select.setBorder(border);
		
		//説明
		message = new JLabel();
		message.setHorizontalAlignment(SwingConstants.CENTER);
		message.setText("選択: ↑,↓   決定：SPACE");
		message.setVerticalTextPosition(JLabel.CENTER);
		message.setHorizontalTextPosition(JLabel.CENTER);
		message.setBounds(249,517,300,30);
		message.setBorder(border);
		
		//配置
		this.setLayout(null);
		this.add(title);
		this.add(start);
		this.add(exit);
		this.add(select);
		this.add(message);
		
		//リスナーの設定
		myKeyListener = new MyKeyListener(this);
		
	}
	
	private class MyKeyListener implements KeyListener{
		//貼り付け先を保持
		TitlePanel panel;
		
		//コンストラクタ
		MyKeyListener(TitlePanel p){
		super();
		panel = p;
		panel.addKeyListener(this);
	}
		@Override
		public void keyTyped(KeyEvent e) {
		//do nothing
		}
		
		@Override
		public void keyReleased(KeyEvent e) {
		//do nothing
		}
		
		@Override
		public void keyPressed(KeyEvent e) {
			switch(e.getKeyCode()) {
		
			case KeyEvent.VK_DOWN:
			if(checkMenu == Menu.START) {
				select.setLocation(select.getX(),select.getY()+50);
				checkMenu = Menu.EXIT;
			}
			break;
			case KeyEvent.VK_UP:
				if(checkMenu == Menu.EXIT) {
					select.setLocation(select.getX(),select.getY()-50);
					checkMenu = Menu.START;
				}
			break;
		
			case KeyEvent.VK_SPACE:
			if(checkMenu == Menu.START) {
				Main.mainWindow.setFrontScreenAndFocus(ScreenMode.GAME);
				gamePanel.resetGame();
			}else if(checkMenu == Menu.EXIT) {
				//終了
				System.exit(0);
			}
			break;
			}
		}
	}
}

package mainPackage;

public class Main{
	static MainWindow mainWindow;
	public static void main(String[] args) {
		mainWindow = new MainWindow();
		mainWindow.preparePanels();
		mainWindow.prepareComponents();
		mainWindow.setFrontScreenAndFocus(ScreenMode.TITLE);
		
		mainWindow.setVisible(true);
	}
}

package mainPackage;

import java.awt.CardLayout;
import java.awt.Color;
import java.awt.Dimension;

import javax.swing.JFrame;

public class MainWindow extends JFrame{
	private static final long serialVersionUID = 1L;
	
	ScreenMode screenMode = ScreenMode.GAME;
	
	final int WIDTH = 800;
	final int HEIGHT = 600;
	
	CardLayout layout = new CardLayout();
	
	TitlePanel titlePanel;
	GamePanel gamePanel;
	
	MainWindow(){
		this.setTitle("SheepInComa");
		//ImageIcon icon = new ImageIcon(getClass().getClassLoader().getResource(null));
		//this.setIconImage(icon.getImage());
		
		this.setDefaultCloseOperation(EXIT_ON_CLOSE);
		this.setResizable(false);
		this.getContentPane().setBackground(Color.green);
		this.setLayout(layout);
		this.setPreferredSize(new Dimension(WIDTH,HEIGHT));
		this.pack();
		this.setLocationRelativeTo(null);
	}
	
	public void preparePanels(){
		
		titlePanel = new TitlePanel();
		this.add(titlePanel,"タイトル画面");
		gamePanel = new GamePanel();
		this.add(gamePanel,"ゲーム画面");
		this.pack();
	}
	
	public void prepareComponents() {
			titlePanel.prepareComponents();
			gamePanel.prepareComponents();
			
		}
	
	public void setFrontScreenAndFocus(ScreenMode s) {
		screenMode = s;
		
		switch(screenMode) {
		case TITLE:
			layout.show(this.getContentPane(),"タイトル画面");
			titlePanel.requestFocus();
			break;
		case GAME:
			layout.show(this.getContentPane(),"ゲーム画面");
			gamePanel.requestFocus();
			break;
		}
	}
	
}

試したこと

自分の知識で思いつく限りは試してみたつもりなのですが、なにか初歩的な見逃しをしている気がしてなりません。
上級者の視点からすれば他愛もないようなコードかもしれませんが、ご助言いただけると助かります。

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

1～88行目まではGamePanelクラス、
89行目からTitlePanelクラスになります。

Comments

[jimbe]

コードのご提示はファイル毎で分けてください。コピペしての確認時に編集が必要な場合、そこで間違いが起きて余計な時間が掛かる可能性があります。

[退会済みユーザー]

ご指摘のコメントありがとうございます。
修正を行いました。

[jimbe]

有難うございます。

TitlePanel の先頭20行程が編集に失敗されているようです。
teratail 上で編集するよりも、お手元のファイルの中身をまるごとコピペされたほうが確実ではないでしょうか。

[退会済みユーザー]

重ねて失礼しました。
修正しました。

[int32_t]

> Main.mainWindow.setFrontScreenAndFocus(ScreenMode.GAME);

この関数のコードの開示も必要かと思います。

[退会済みユーザー]

不慣れなものですみません。
Mainクラス　MainWindowクラスを追記しました。

[jimbe]

コードが増えましたね^^;
4ファイルをコピペしても全てにコンパイルエラーが出てますが…FieldPanel と ScreenMode かな? 思った以上に大きなプログラムだったようで。

[TN8001]

こちらを参考にされているんですよね？（参考コードは質問に明示してください！）
[【Java GUI 入門】画面の切り替えってどうやるの？GUIプログラミングの考え方から学べるSwing入門講座！【eclipse/Swing】 - YouTube](https://www.youtube.com/watch?v=FD7_xX-O3v8&list=PLP85E4o2F1o4eQp2u34PRZpTEIsuIc8Ok&index=2)

[退会済みユーザー]

その通りです。申し訳ありませんでした。

Answer 1

かなりクセのある状態になっています。
prepareComponent とか別にメソッドを用意しなくても(ましてコンストラクタの後に呼んで貰わなくても)、コンストラクタに並べてしまえば良いのです。
Main は main メソッドだけですので MainWindow に入れてしまえますし、 MainWindow のメソッドを呼ぶのであれば (static フィールドなど使用せず) その参照をコンストラクタで渡して使うほうが良いです。ScreenMode は無くても それぞれ呼ぶモノは決まっているのですから showTitle() とかで済みます。

MainWindow.java

import java.awt.*;
import java.awt.event.WindowEvent;

import javax.swing.JFrame;
import javax.swing.SwingUtilities;

public class MainWindow extends JFrame {
  public static void main(String[] args) {
    SwingUtilities.invokeLater(() -> new MainWindow().setVisible(true));
  }

  private static final int WIDTH = 800;
  private static final int HEIGHT = 600;

  private static final String TITLE_TAG = "タイトル画面";
  private static final String GAME_TAG = "ゲーム画面";

  private CardLayout layout = new CardLayout();
  private TitlePanel titlePanel;
  private GamePanel gamePanel;

  MainWindow() {
    super("SheepInComa");
    setDefaultCloseOperation(EXIT_ON_CLOSE);
    setPreferredSize(new Dimension(WIDTH, HEIGHT));
    setResizable(false);
    setLayout(layout);

    getContentPane().setBackground(Color.green);

    titlePanel = new TitlePanel(this);
    add(titlePanel, TITLE_TAG);

    gamePanel = new GamePanel(this);
    add(gamePanel, GAME_TAG);

    showTitle();

    pack();
    setLocationRelativeTo(null);
  }

  void showTitle() {
    layout.show(getContentPane(), TITLE_TAG);
    titlePanel.requestFocus();
  }

  void showGame() {
    layout.show(getContentPane(), GAME_TAG);
    gamePanel.requestFocus();
  }

  void exit() {
    dispatchEvent(new WindowEvent(this, WindowEvent.WINDOW_CLOSING));
  }
}

TitlePanel.java

import java.awt.Color;
import java.awt.Font;
import java.awt.event.KeyEvent;
import java.awt.event.KeyListener;

import javax.swing.*;
import javax.swing.border.Border;

class TitlePanel extends JPanel {
  private enum Menu {
    START, EXIT,
  }
  private Menu checkMenu = Menu.START;

  TitlePanel(MainWindow main) {
    super(null);
    setBackground(Color.cyan);

    Font font = new Font("MV boli", Font.BOLD, 40);
    Border selectBorder = BorderFactory.createLineBorder(Color.BLUE, 4);
    Border unselectBorder = BorderFactory.createLineBorder(new Color(0,0,0,0), 4);

    JLabel title = new JLabel("Created by 【TAKAYUKI H】");
    title.setHorizontalAlignment(SwingConstants.CENTER);
    title.setVerticalAlignment(SwingConstants.BOTTOM);
    title.setHorizontalTextPosition(JLabel.CENTER);
    title.setVerticalTextPosition(SwingConstants.BOTTOM);
    title.setBounds(90, 0, 600, 350);
    add(title);

    //選択肢
    JLabel start = new JLabel("START");
    start.setFont(font);
    start.setHorizontalAlignment(SwingConstants.CENTER);
    start.setHorizontalTextPosition(JLabel.CENTER);
    start.setVerticalTextPosition(JLabel.BOTTOM);
    start.setBounds(330, 400, 150, 40);
    start.setBorder(checkMenu == Menu.START ? selectBorder : unselectBorder);
    add(start);

    JLabel exit = new JLabel("EXIT");
    exit.setFont(font);
    exit.setHorizontalAlignment(SwingConstants.CENTER);
    exit.setHorizontalTextPosition(JLabel.CENTER);
    exit.setVerticalTextPosition(JLabel.BOTTOM);
    exit.setBounds(330, 450, 150, 40);
    exit.setBorder(checkMenu == Menu.EXIT ? selectBorder : unselectBorder);
    add(exit);

    //説明
    JLabel message = new JLabel("選択: ↑,↓   決定：SPACE");
    message.setHorizontalAlignment(SwingConstants.CENTER);
    message.setVerticalTextPosition(JLabel.CENTER);
    message.setHorizontalTextPosition(JLabel.CENTER);
    message.setBounds(249, 517, 300, 30);
    add(message);

    setFocusable(true);
    addKeyListener(new KeyListener() {
      @Override
      public void keyPressed(KeyEvent e) {
        switch (e.getKeyCode()) {
        case KeyEvent.VK_DOWN:
          if (checkMenu == Menu.START) {
            start.setBorder(unselectBorder);
            exit.setBorder(selectBorder);
            checkMenu = Menu.EXIT;
          }
          break;
        case KeyEvent.VK_UP:
          if (checkMenu == Menu.EXIT) {
            start.setBorder(selectBorder);
            exit.setBorder(unselectBorder);
            checkMenu = Menu.START;
          }
          break;

        case KeyEvent.VK_SPACE:
          if(checkMenu == Menu.START) {
            main.showGame();
          } else if(checkMenu == Menu.EXIT) {
            main.exit();
          }
          break;
        }
      }

      @Override
      public void keyTyped(KeyEvent e) {}
      @Override
      public void keyReleased(KeyEvent e) {}
    });
  }
}

GamePanel.java(中で使用しているクラスが分からないのでガワだけ)

import java.awt.BorderLayout;
import java.awt.Color;
import java.awt.event.KeyEvent;
import java.awt.event.KeyListener;

import javax.swing.JPanel;
import javax.swing.Timer;

class GamePanel extends JPanel {
  GamePanel(MainWindow main) {
    super(new BorderLayout());
    setBackground(Color.yellow);

    //MenuBar menuBar = new MenuBar();
    //menuBar.prepareComponents();
    //add(menuBar, BorderLayout.NORTH);

    //FieldPanel fieldPanel = new FieldPanel();
    //fieldPanel.prepareComponents();
    //add(fieldPanel, BorderLayout.CENTER);

    setFocusable(true);
    addKeyListener(new KeyListener() {
      @Override
      public void keyPressed(KeyEvent e) {
        switch(e.getKeyCode()) {
        case KeyEvent.VK_H://Hを押したとき
          main.showTitle();
          resetGame();
        }
      }
      @Override
      public void keyTyped(KeyEvent e) {}
      @Override
      public void keyReleased(KeyEvent e) {}
    });

    Timer timer = new Timer(60000, e -> {
      main.showTitle();
      resetGame();
    });
    timer.start();
  }

  void resetGame() {
    //fieldPanel.removeAll();
    //fieldPanel.prepareComponents();
  }
}

Answer 2

詳しくコードを読んだわけではないですが、

• TitlePanel#gamePanel の初期化
• MainWindow#preparePanels() の中のコード

の2箇所で new GamePanel() が呼ばれるのはおかしい気がします。GamePanel のインスタンスは1つだけであるべきではないでしょうか。

TitlePanel#gamePanel は不要で、setFrontScreenAndFocus() の中で gamePanel.resetGame(); を呼ぶべきかもしれません。

Comments

[退会済みユーザー]

ご指摘の通りsetFrontScreenAndFocus()の中にgamePanel.reset()を書き加えることで意図した通りの機能が実装できました。
resetGame()を呼び出し行うメソッド記述先が間違っていたようです。
非常に助かりました。ありがとうございました。

[退会済みユーザー]

jimbeさま
コンストラクタとメソッドの別々に配置することの意味について疑問を感じてはいましたが、私は初心者も初心者なのでまずは参考先に従って書いていこうという方針でした。
今後、学習を進めて行く上でいただいたご意見も参考にしていこうと思います。
貴重なご意見ありがとうございました。