Mehrere Panels auf JFrame

[HerrInfo]

Hallo zusammen,

ich habe mehrere Panels auf einem JFrame und würde gerne zwischen ihnen mittels einer Methode wechseln. Leider funktioniert es nicht richtig, wenn ich setVisible(false) setze. Ich habe auch versucht, sie zu entfernen, aber dann werden die neuen Panels nicht angezeigt, sondern laufen nur im Hintergrund. Habt ihr eine Idee für eine Lösung? Hier ist der entsprechende Code:

package net.tim.gui;

import net.tim.core.Verwaltung;

import javax.swing.*;

public class Frame extends JFrame {
    Verwaltung verwaltung;

    public GamePanel gamePanel = new GamePanel(this);
    GlassPanel glassPanel = new GlassPanel(gamePanel);
    MenuPanel menuPanel = new MenuPanel(this);
    public Frame(Verwaltung verwaltung) {
        this.verwaltung = verwaltung;

        setTitle("A* Pathfinding");
        setDefaultCloseOperation(JFrame.EXIT_ON_CLOSE);
        setSize(GamePanel.screenWidth, GamePanel.screenHeight);
        setResizable(false);

        add(gamePanel);

        setGlassPane(glassPanel);
        getGlassPane().setVisible(true);

        pack();
        setLocationRelativeTo(null);
        setVisible(true);
    }

    public void gameEnd(){
        glassPanel.gameEnd = true;
        glassPanel.repaint();

        try {
            Thread.sleep(5000);
        } catch (InterruptedException e) {
            throw new RuntimeException(e);
        }

        switchToMenu();
    }

    public void switchToMenu(){
        remove(gamePanel);
        glassPanel.setVisible(false);
        add(menuPanel.panel);

        System.out.println("Switched to menu");
    }

    public void switchToGame(){
        menuPanel.panel.setVisible(false);

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

        remove(menuPanel.panel);



        glassPanel.setVisible(true);
        glassPanel.gameEnd = false;
        glassPanel.repaint();

        Verwaltung.gameRunning = true;
        try {
            Thread.sleep(5000);
        } catch (InterruptedException e) {
            throw new RuntimeException(e);
        }
        verwaltung.startGame();
    }

    public void switchToShop(){
        remove(menuPanel.panel);
        glassPanel.setVisible(false);
    }

    public void switchToSettings(){
        remove(menuPanel.panel);
        glassPanel.setVisible(false);
    }

    public void switchToCredits(){
        remove(menuPanel.panel);
        glassPanel.setVisible(false);
        menuPanel.panel.setVisible(false);
    }
}

 

[KonradN]

Also generell sollte es funktionieren. Aber was mir etwas ins Auge fällt ist dein Thread.sleep mit den Aufrufen von Methoden in verwaltung.

Kann es sein, dass Du den UI Thread blockierst und daher wird einfach das Fenster nicht neu gemalt?
Bei UI Anwendungen gilt generell:
a) Nie den UI Thread blockieren - länger laufende Dinge immer in einem separaten Thread durchführen.
b) Nie UI Komponenten außerhalb des UI Threads verändern

Und gegen eine der Regeln verstößt Du in switchToGame auf jeden Fall.

 

[KonradN]

Neben den ersten Hinweis noch zusätzlich ein weiterer Hinweis:
Evtl. reicht das unsichtbar machen nicht - nicht dass es da noch Probleme gibt, weil ein unsichtbares Control womöglich noch den Fokus hat oder einen Mausclick abfangen kann. Daher würde ich vorsichtshalber auch setEnabled aufrufen mit false beim verstecken und true beim sichtbar machen.
Aus der Dokumentation:

Sets whether or not this component is enabled. A component that is enabled may respond to user input, while a component that is not enabled cannot respond to user input.

 

[mihe7]

Wieso nicht einfach ein CardLayout? https://docs.oracle.com/en/java/javase/19/docs/api/java.desktop/java/awt/CardLayout.html

 

[HerrInfo]

Erst einmal vielen Dank für die schnelle Antwort. Ich habe das gesamte Layout jetzt mit CardLayout umgesetzt, aber es passiert immer noch nichts, nachdem der Start-Game Button gedrückt wurde. Das Panel wechselt immer noch nicht. Hier ist der aktualisierte Code:

package net.tim.gui;

import net.tim.core.Verwaltung;

import javax.swing.*;
import java.awt.*;

public class Frame extends JFrame {
    Verwaltung verwaltung;

    public GamePanel gamePanel = new GamePanel(this);
    GlassPanel glassPanel = new GlassPanel(gamePanel);
    MenuPanel menuPanel = new MenuPanel(this);
    CardLayout cardLayout = new CardLayout();
    public Frame(Verwaltung verwaltung) {
        this.verwaltung = verwaltung;
        setLayout(cardLayout);

        setTitle("A* Pathfinding");
        setDefaultCloseOperation(JFrame.EXIT_ON_CLOSE);
        setSize(GamePanel.screenWidth, GamePanel.screenHeight);
        setResizable(false);

        add(gamePanel, "game");
        add(menuPanel.panel, "menu");

        setGlassPane(glassPanel);
        getGlassPane().setVisible(true);

        pack();
        setLocationRelativeTo(null);
        setVisible(true);
    }

    public void gameEnd(){
        glassPanel.gameEnd = true;
        glassPanel.repaint();

        try {
            Thread.sleep(5000);
        } catch (InterruptedException e) {
            throw new RuntimeException(e);
        }

        switchToMenu();
    }

    public void switchToMenu(){
        glassPanel.setVisible(false);
        cardLayout.show(getContentPane(), "menu");

        System.out.println("Switched to menu");
    }

    public void switchToGame(){
        glassPanel.setVisible(true);
        glassPanel.gameEnd = false;
        glassPanel.repaint();

        cardLayout.show(getContentPane(), "game");

        Verwaltung.gameRunning = true;
        verwaltung.startGame();
    }

    
    //Ab hier ignorieren
    public void switchToShop(){
        remove(menuPanel.panel);
        glassPanel.setVisible(false);
    }

    public void switchToSettings(){
        remove(menuPanel.panel);
        glassPanel.setVisible(false);
    }

    public void switchToCredits(){
        remove(menuPanel.panel);
        glassPanel.setVisible(false);
        menuPanel.panel.setVisible(false);
    }
}

So sieht es dann aus und die GUI reagiert nicht mehr.

 

[KonradN]

Den Hinweis bezüglich blockieren des UI Threads hast Du gelesen? Das war meine Befürchtung in meiner ersten Antwort und deine Aussage

und die GUI reagiert nicht mehr.

scheint das auch noch einmal zu bestätigen!

 

[mihe7]

Beim CardLayout musst Du nichts removen. Du addest einmal Deine Panels und kannst dann einfach hin und her wechseln.

Und ja, wie @KonradN schon geschrieben hat: UI blockieren = nix gut.

import javax.swing.*;
import java.awt.*;

public class Frame extends JFrame {

    private JPanel gamePanel = new JPanel(new BorderLayout());
    JPanel menuPanel = new JPanel(new BorderLayout());
    CardLayout cardLayout = new CardLayout();
    Thread endGameThread;

    public Frame() {
        setLayout(cardLayout);

        JButton startGameButton = new JButton("Start Game");
        startGameButton.addActionListener(e -> switchToGame());
        menuPanel.add(startGameButton, BorderLayout.SOUTH);
        menuPanel.add(createLabel("MENU"));

        JButton endGameButton = new JButton("End Game");
        endGameButton.addActionListener(e -> gameEnd());
        gamePanel.add(endGameButton, BorderLayout.SOUTH);
        gamePanel.add(createLabel("GAME"));

        setTitle("A* Pathfinding");
        setDefaultCloseOperation(JFrame.EXIT_ON_CLOSE);
        setSize(800, 600);
        setResizable(false);

        add(gamePanel, "game");
        add(menuPanel, "menu");

        setLocationRelativeTo(null);
        setVisible(true);
    }

    private JLabel createLabel(String text) {
        JLabel label = new JLabel(text);
        label.setFont(label.getFont().deriveFont(72f));
        label.setHorizontalAlignment(JLabel.CENTER);
        return label;
    }

    public void gameEnd() {
        if (endGameThread != null) {
            return;
        }

        endGameThread = new Thread(() -> {
            try {
                Thread.sleep(2000);
            } catch (InterruptedException e) {
                // hier mal ignoieren...
            }

            endGameThread = null;
            switchToMenu();
        });

        endGameThread.start();
    }

    public void switchToMenu() {
        cardLayout.show(getContentPane(), "menu");
    }

    public void switchToGame() {
        cardLayout.show(getContentPane(), "game");
    }

    public static void main(String[] args) {
        SwingUtilities.invokeLater(() -> new Frame());
    }
}

 

[HerrInfo]

Vielen Dank für die schnelle Antwort! Der blockierende Thread war in einer anderen Klasse, daher war er nicht direkt zu finden. Mit dem Code kann ich auch etwas anfangen, da ich nirgendwo ein sauberes Beispiel gefunden habe. Nochmals vielen Dank! Jetzt werde ich mich daran machen, den Code etwas aufzuräumen.

 

[mihe7]

Sorry, ich vergaß zu erwähnen, dass ich Deine Panels kurzerhand ausgetauscht habe