Swing Repaint Problem - mal wieder

[blauerninja]

Hallo,
ich hab in den Weiten des Webs keine Lösung für mein Problem gefunden, dh. die Lösung funktioniert nicht, finden tut man viel. Es geht mal wieder um das leidige Thema repaint().

Mein Szenario:
Beim Spielstart wird das Hauptfenster StartFrame gestartet. Dieses hat 2 JPanels einen oberen für die Graphik und einen unten für die Steuerungselemente, die aber von außen gesetzt werden, da sie im Spielvelauf ausgetauscht werden sollen. So weit so gut. Was nicht funktioniert, ist, dass ich dem StartFrame ein JPanel zuweise, dieses trotz

validate()

und

repaint()

nicht angezeigt wird.

Dazu der Code:

package game.gui;


import java.awt.BorderLayout;

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

@SuppressWarnings("serial")
public final class StartFrame extends JFrame {
  public static final int WIDTH  = 480;
  public static final int HEIGHT = 320;

  private static final String TITLE = "RPG Game";
  private static final StartFrame FRAME = new StartFrame();

  private JPanel screen;
  private JPanel touchScreen;


  private StartFrame() {
    this.setSize(WIDTH, 2*HEIGHT);
    this.setLocationRelativeTo(null);
    this.setTitle(TITLE);
    this.setResizable(false);
    this.setUndecorated(true);
    this.screen = new JPanel(); [COLOR="Green"]// Hier werden sie schon mal instatiiert[/COLOR]
    this.touchScreen = new JPanel();
    this.add(this.screen, BorderLayout.NORTH);
    this.add(this.touchScreen, BorderLayout.SOUTH);
    this.setVisible(true);
  }


  public static void initialize() { }


  public static void setMainScreen(JPanel panel) {
    FRAME.screen = panel;
   // FRAME.screen.validate();
   // FRAME.screen.repaint();
    FRAME.validate(); [COLOR="Green"]// Sollte doch reichen - oder?[/COLOR]
    FRAME.repaint();
  }


  public static void setTouchScreen(JPanel panel) {
    FRAME.touchScreen = panel; [COLOR="Green"]// Wird wie setMainScreen() sein, wenn es oben funktioniert[/COLOR]
  }
}

package game;


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

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

import game.gui.StartFrame;

// Viel Testcode
public final class Game {
  public static void main(String[] args) throws Exception {
    StartFrame.initialize();

    System.out.println("Sleep");
    Thread.sleep(2000);

    System.out.println("Set new panel");

    JPanel panel = createPanel();
    panel.setBackground(new Color(255, 200, 200));
    StartFrame.setMainScreen(panel);

    System.out.println("Finished");
  }


  private static JPanel createPanel() {
    Dimension size = new Dimension(StartFrame.WIDTH, StartFrame.HEIGHT);
    JPanel panel = new JPanel();
    panel.setSize(size);
    panel.setPreferredSize(size); [COLOR="Green"]// Erzwingt, dass das Panel so groß ist, wie das Fenster (mit halber Höhe)[/COLOR]

    [COLOR="Green"]// Ein bisschen was zum Füllen[/COLOR]
    for (int i = 0; i < 20; i++) {
      panel.add(new JLabel("                                                                                                  " + i));
    }
    return panel;
  }
}

Da ich ein neues

Component

dem Fenster hinzufüge, so muss doch

validate()

und

repaint()

reichen. Oder lieg ich falsch? (OK, ich weiß ich lieg falsch, aber wo?)

Hoffe, dass jemand es besser weiß, als ich und sorry, für die Umstände.

 

[SlaterB]

ein Attribut deiner Klasse neu zuzuweisen hat wenig Auswirkung,
da kannst du dir auch einen Kassenbon ins Portmonnaie legen und hast trotzdem keine neuen Reifen am Auto

lasse screen bestehen,
add()e dein Paramter-Panel in screen hinein,
falls zu vermuten ist, dass schon was drin war, alles vorherige mit removeAll() entfernen

evtl. noch experimentieren ob screen ein anderes Layout braucht, damit alles gut klappt

validate() + repaint() auf das JFrame ist genau mein Vorschlag, richtig

 

[blauerninja]

Super, funktioniert. Was ich aber noch nicht ganz verstehe: wieso funktioniert es nicht, wenn ich einfach die Variable neu referenziere? Die validate() sorgt doch dafür, dass der Frame seine Komponenten auf Änderung prüft. Wird dabei nur geprüft, ob Komponenten hinzugefügt oder entfernt wurden? Sprich wenn ich keine add() oder remove() aufrufen, passiert nichts?

 

[Michael...]

Was ich aber noch nicht ganz verstehe: wieso funktioniert es nicht, wenn ich einfach die Variable neu referenziere?

Grundsätzlich finde ich den gesamten Code eher etwas fragwürdig.
In einem graphischen Komponente (z.B. JFrame) sind keine Variablen verbaut, sondern weitere graphische Komponenten (z.B. JPanel, JButton...). Nur weil dann eine Variable - die vorher genutzt wurde, um ein JPanel auf das sie (zum damaligen Zeitpunkt) referenziert, im JFrame zu verbauen - jetzt auf eine andere Komponente verweist, heißt das ja nicht, das diese Komponente automatisch im JFrame verbaut wird.
Die Variable ist ja nur ein Hilfsmittel um das Objekt zu "greifen". Wenn Du einen Hund an der Leine in einen Zwinger führst und die Leine abmachst und an einem anderen Hund einhakst, ist dieser ja nicht automatisch im Zwinger.

Die validate() sorgt doch dafür, dass der Frame seine Komponenten auf Änderung prüft. Wird dabei nur geprüft, ob Komponenten hinzugefügt oder entfernt wurden? Sprich wenn ich keine add() oder remove() aufrufen, passiert nichts?

Genau, denn ohne ein add oder ein remove hat sich ja am Frame und seinen Komponenten nichts geändert.

 

[blauerninja]

Puh, jetzt funktioniert es nicht mehr. Habs auf ohne Variablen umgestellt und beim Ausführen wird kein Panel angezeigt. Keine Ahnung wieso.

package game.gui;


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

@SuppressWarnings("serial")
public final class GameFrame extends JFrame {
  public static final int WIDTH  = 480;
  public static final int HEIGHT = 320;

  private static final String TITLE = "RPG Game";
  private static final GameFrame FRAME = new GameFrame();


  private GameFrame() {
    this.setSize(WIDTH, HEIGHT);
    this.setLocationRelativeTo(null);
    this.setTitle(TITLE);
    this.setResizable(false);
    this.setUndecorated(true);
    this.setVisible(true);
  }


  public static void initialize() { }


  public static void setScreen(JPanel screen) {
    FRAME.removeAll();
    FRAME.add(screen);
    FRAME.validate();
    FRAME.repaint();
  }
}

package game;


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

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

import game.gui.GameFrame;

public final class Game {
  public static void main(String[] args) throws Exception {
    GameFrame.initialize();

    System.out.println("Sleep");
    Thread.sleep(2000);
    System.out.println("New Panel");

    JPanel panel = new JPanel();
    panel.setBackground(new Color(255, 200, 200));
    panel.setPreferredSize(new Dimension(GameFrame.WIDTH, GameFrame.HEIGHT));
    panel.add(new JLabel(panel.getSize().toString()));
    GameFrame.setScreen(panel);

    System.out.println("Finished");
  }
}

Die

setScreen()

wird aufgerufen und abgearbeitet. Wo soll ein Fehler sein?
???:L:shock:

=============

Ok, bissl versucht den Fehler zu finden. Folgendes kam raus:
hab vor der Änderung das Frame erstmal mit

setVisible(false)

versteckt und nach der Änderung wieder angezeigt.
Das Panel wird entfernt, das neue wird nicht eingefügt. Sodann wird das Fenster auf Minimalgröße verkleinert.
Und somit nur noch mehr Rätsel. :bahnhof:

====
Toll, jetzt funktioniert es mit Variablen auch nicht. ;(
Ein Component wurde aber hinzugefügt, was man mit

getComponentCount()

prüfen kann. Dennoch wird es nicht angezeigt. zefix nochmal.

 

[blauerninja]

Ok, hab den Fehler gefunden. Und darüber hinaus darf man bei der Methode ohne Variablen die

removeAll()

vom

JFrame

nicht, ich betone NICHT!!! aufrufen, sonst wird keine Komponente mehr hinzugefügt. Stattdessen muss man mit

FRAME.getContentPane().removeAll()

arbeiten. Dann funktioniert alles wie gewünscht^^

Ein großes Danke an alle.