JPanels übereinander legen

[CptK]

Hallo, ich habe ein JPanel auf dem etwas gezeichnet wird und bei einem bestimmten Event soll ein zweites Panel über das erste drüber gelegt werden. Dabei ist das zweite JPanel durchsichtig (also opacity des Hintergrunds = 0.5) und man soll das erste Panel noch sehen. An sich klappt das auch ganz gut, aber es gibt noch 2 Probleme:
1. Wenn ich nicht die vordefinierte Größe des JFrames nutze, sprich die Größe des Frames wird manuell geändert, werden im hinteren Panel die Sachen alle doppelt gezeichnet.

2. Wenn ich die größe des Frames verändere, während das zweite Panel angezeigt wird, wird dann nur noch das vordere Panel gezeichnet, ich hätte ja aber gerne, dass das hintere auch weiterhin sichtbar ist.

Der Code:
[CODE lang="java" title="Hinteres JPanel"]public class GamePanel extends JPanel {
private final static int PILLAR_DISTANCE = 300;
private final GameFrame frame;
private Player player;
private List<Obstacle> pillars;
private ResultPanel resultPanel; // das vordere Pane
private JLabel scoreLabel;
private int score;

public GamePanel(Dimension d, final GameFrame frame) {
this.frame = frame;
setSize(d);
setLayout(null);
init();
setVisible(true);
}

@Override
public void paintComponent(Graphics g) {
super.paintComponent(g);
player.draw(g);
pillars.forEach(p -> p.draw(g));
}

public void onResize() {
player.setY(getHeight() / 2 - player.getHeight() / 2);
pillars.forEach(p -> p.resize(getSize()));
resultPanel.setSize(getSize());
resultPanel.onResize();
}

public void addPillar() {
pillars.add(new Obstacle(pillars.get(pillars.size() - 1).getX() + PILLAR_DISTANCE, getSize()));
}

private void init() {
resultPanel = new ResultPanel(getSize(), this);
resultPanel.setVisible(false);
add(resultPanel);

scoreLabel = new JLabel();
scoreLabel.setFont(new Font("Impact", Font.BOLD, 40));
scoreLabel.setBounds(40, 40, 150, 40);
scoreLabel.setVisible(true);
add(scoreLabel);

initGameObjects();
}

private void initGameObjects() {
score = 0;
scoreLabel.setText("" + score);

pillars = Collections.synchronizedList(new ArrayList<>());
player = new Player("rsc/flappy_bird/player.png");
player.setSize(84, 65);
player.setY(getHeight() / 2 - player.getHeight() / 2);
player.setX((int) (getWidth() * 0.3));

int x = PILLAR_DISTANCE;
for (int i = 0; i < 10; i++) {
pillars.add(new Obstacle(x + 500, getSize()));
x += PILLAR_DISTANCE;
}
pillars.forEach(p -> p.resize(getSize()));
repaint();
}

...
}[/CODE]
[CODE lang="java" title="Vorderes JPanel"]public class ResultPanel extends JPanel {
private final GamePanel panel; // das hintere Panel
private JButton restartBtn;
private JLabel crashedLabel;

public ResultPanel(Dimension d, final GamePanel panel) {
this.panel = panel;
setSize(d);
setLayout(null);
setBackground(new Color(1, 1, 1, 0.5f));
init();
}

public void onResize() {
restartBtn.setBounds(getWidth() / 2 - 50, (getHeight() / 2 - 20), 100, 40);
crashedLabel.setBounds(getWidth() / 2 - 150, (getHeight() / 5), 300, 50);
}

@Override
public void paintComponent(Graphics g) {
super.paintComponent(g);
}

public void activate() {
setVisible(true);
restartBtn.setVisible(true);
crashedLabel.setVisible(true);
}

private void init() {
restartBtn = new JButton("restart");
restartBtn.setBounds(getWidth() / 2 - 50, (getHeight() / 2 - 20), 100, 40);
restartBtn.addActionListener(new ActionListener() {
@Override
public void actionPerformed(ActionEvent arg0) {
panel.reset();
restartBtn.setVisible(false);
crashedLabel.setVisible(false);
}
});
add(restartBtn);
restartBtn.setVisible(false);

crashedLabel = new JLabel("YOU CRASHED");
crashedLabel.setFont(new Font("Impact", Font.BOLD, 50));
crashedLabel.setBounds(getWidth() / 2 - 150, (getHeight() / 5), 300, 50);
crashedLabel.setVisible(false);
add(crashedLabel);
repaint();
}
}[/CODE]
Und zur Sicherheit noch das JFrame:

public class GameFrame extends JFrame {
    private GamePanel panel; // hinteres JPanel

    public GameFrame() {
        setSize(new Dimension(800, 600));
        setLayout(null);
        panel = new GamePanel(getSize(), this);

        addComponentListener(new ComponentAdapter() {
            @Override
            public void componentResized(ComponentEvent e) {
                panel.setSize(getSize());
                panel.onResize();
            }
        });

        addKeyListener(new KeyAdapter() {
            @Override
            public void keyPressed(KeyEvent e) {
                // hier wird u.a. das Event getriggert, mit dem das vordere JPanel aktiviert wird
            }
        });

        add(panel);
        setDefaultCloseOperation(EXIT_ON_CLOSE);
        setVisible(true);
    }

 

[Neumi5694]

Was heißt "doppelt" gezeichnet? Das müsstest du ja noch nicht mal merken. Ich gehe davon aus, dass der Player an 2 verschiedenen Stellen auftaucht? Das klingt danach, als ob der Background nicht neu gezeichnet worden wäre.

Du gehst für Resize-Events einen recht umständlichen Weg (Größen bei Größenänderung des Parents manuell setzen). Wie wär's stattdessen mit der Verwendung von Layoutmanagern? Die lösen dann auch die notwendigen Events aus. Eventuell fehlt der Event, der den Aufruf von paintComponent auslöst.

Im Code hab ich die Stelle nicht finden können, an der du opaque für das Result-Panel setzt.

ps: Mit dem Keylistener wirst du wahrscheinlich nicht glücklich werden, da der nur reagiert, wenn die Komponente den Fokus hat und nicht eine ihrer Subkomponenten. Eventuell wirst du auf eine Inputmap/Actionmap ausweichen müssen.

Ich geh mal davon aus, dass das Result eine Art Overlay darstellen soll. Hier könntest du z.B. das Result als Glasspane für das Frame verwenden (blockiert alles, was darunter ist), wär vielleicht sinnvoller.
Aber ich gehe davon aus, dass die Mustervorgehensweise ein Layeredpane wäre.

How to Use Layered Panes (The Java™ Tutorials > Creating a GUI With Swing > Using Swing Components)

 

[CptK]

Was heißt "doppelt" gezeichnet? Das müsstest du ja noch nicht mal merken. Ich gehe davon aus, dass der Player an 2 verschiedenen Stellen auftaucht? Das klingt danach, als ob der Background nicht neu gezeichnet worden wäre.

Ja genau, alle Objekte tauchen an 2 Stellen auf, ich verstehe aber nicht wieso. Also ich habe ja eine initiale Größe von 800 x 600. Wenn ich nichts ändere funktioniert auch alles. Wenn ich die Größe ändere, bevor das vordere Panel erscheint, so wird alles 2 mal gezeichnet und wenn ich die Größe ändere, während das vordere Panel schon da ist, ist nur noch das vordere Panel sichtbar.

Du gehst für Resize-Events einen recht umständlichen Weg (Größen bei Größenänderung des Parents manuell setzen). Wie wär's stattdessen mit der Verwendung von Layoutmanagern? Die lösen dann auch die notwendigen Events aus. Eventuell fehlt der Event, der den Aufruf von paintComponent auslöst.

Joa damit kenne ich mich nicht wirklich aus, das so zu machen war erstmal der intuitivere Weg. Ich habe schon versucht auf alle Panels repaint() an allen möglichen Stellen aufzurufen, das hat auch nicht geholfen.

Im Code hab ich die Stelle nicht finden können, an der du opaque für das Result-Panel setzt.

Einfach beim Hintergrund:

setBackground(new Color(1, 1, 1, 0.5f));

ps: Mit dem Keylistener wirst du wahrscheinlich nicht glücklich werden, da der nur reagiert, wenn die Komponente den Fokus hat und nicht eine ihrer Subkomponenten. Eventuell wirst du auf eine Inputmap/Actionmap ausweichen müssen.

Also der KeyListner macht genau das, was er soll. Da habe ich keine Probleme.

 

[White_Fox]

Warum kein StackedPane?

 

[kneitzel]

Das klingt etwas nach optimized Drawing.

Returns true if this component tiles its children -- that is, if it can guarantee that the children will not overlap. The repainting system is substantially more efficient in this common case. JComponent subclasses that can't make this guarantee, such as JLayeredPane, should override this method to return false.

Sprich: Bei deinen Panels einfügen:

@Override
public boolean isOptimizedDrawingEnabled()
{
    return false;
}

Das wäre zumindest ein Versuch wert.

Und das repaint erwirkt nichts, weil Du damit ja nur das System aufforderst, die Elemente neu zu zeichnen. Dinge, von denen das System ausgeht, dass diese nicht gezeichnet werden müssen, werden dann natürlich jedes Mal nicht gezeichnet.

 

[Neumi5694]

Was den Layoutmanager angeht. Nimm mal ein Borderlayout und adde das Panel mit .add(panel, SwingConstants.CENTER);
Damit füllt das Panel die gesamte verfügbare Fläche aus, sofern du an den Seitenrändern nichts einfügst.
Das erleichtert das Handling ungemein.
In der Regel gilt: Alles, was du da manuell machst, haben andere schon gemacht, meistens besser.

Was dein setBackground angeht ...
So wie ich Swing kenne, ist es egal, welche Farbe du verwendest. So lange opaque auf true steht, wird das, was darunter liegt, nicht nochmal gezeichnet.
Das dürfte erklären (hab ich jetzt nicht explizit getestet), dass das, was darunter liegt, nur nach dem Einfügen des Panels sichtbar ist und danach - nach Größenordnung - nicht mehr.
Setz den Wert mal auf false.

 

[CptK]

Sprich: Bei deinen Panels einfügen:

@Override
public boolean isOptimizedDrawingEnabled()
{
    return false;
}

Das hat leider nichts bewirkt.

Was dein setBackground angeht ...
So wie ich Swing kenne, ist es egal, welche Farbe du verwendest. So lange opaque auf true steht, wird das, was darunter liegt, nicht nochmal gezeichnet.
Das dürfte erklären (hab ich jetzt nicht explizit getestet), dass das, was darunter liegt, nur nach dem Einfügen des Panels sichtbar ist und danach - nach Größenordnung - nicht mehr.
Setz den Wert mal auf false.

Das opaque hingegen hat alle Probleme gelöst.