Darstellungsfehler auf der Oberfläche

[Snapshot]

Hallo,
ich habe folgendes Problem. Ich verwende für meine Anwendung ein OverlayLayout. Dabei sind die Elemente der eigentlichen Anwendung alle in einem Panel im Vordergrund. Im Hintergrund lieg ein einzelnes ImagePanel. Das ImagePanel ist eine von JPanel abgeleitete Klasse die ein Bild zeichnet und soll als Hintergrundbild für die Oberfläche dienen.
Probleme macht dabei aber immer der Programmstart. Denn die Steuerelemente der Oberfläche, Buttons, Toolbar etc. werden nur zum Teil dargestellt, und zwar bei jedem Programmstart anders! Also mal erscheint der eine Button und mal nicht!? Erst wenn man dann zur Laufzeit die Größe des Fensters leicht verändert oder mit der Maus über die Oberfläche fährt wird diese richtig gezeichnet. Ich habe versucht mit einem repaint() das Problem zu lösen, aber der Effekt bleibt immer der selbe. Nur wenn ich in das im Hintergrund liegende Panel kein Bild lade baut sich die Oberfläche direkt fehlerfrei auf.
Was kann die Ursache für dieses Problem sein? Gibt es irgendwelche besonderen Eigenheiten die man bei der Verwendung von OverlayLayouts beachten muss?

 

[Marco13]

Kann es sein, dass du AWT (Panel) und Swing (JPanel) mischst? Du solltest nur EINS von beidem verwenden (z.B. NUR Swing, also NUR JPanels). Wenn nicht, poste mal etwas (kompilierbaren!) Code.

 

[Snapshot]

Also ich verwende nur JPanels habe aber trotzdem einen import auf AWT für die Layout Objekte (BorderLayout etc.). Kann das etwas damit zu tun haben?

Hier mal ein paar Codeauszüge aus der View:

    public MasterView()
    {
        this.setDefaultCloseOperation(WindowConstants.EXIT_ON_CLOSE);
        this.getContentPane().setLayout(new OverlayLayout(this.getContentPane()));
        
        //Toolbar erstellen
        this.ControlBar = new JToolBar();
        this.ControlBar.setOrientation(JToolBar.VERTICAL);
        this.SelectButtons = new JButton[6];
        for( int i = 0; i < 6; i++ )
        {
            this.SelectButtons[i] = new JButton();
            this.ControlBar.add(this.SelectButtons[i]);
        }
       
        //Steuerelemente der Oberfläche anlegen
        CreatePanels();
        
        //Hintergrund anlegen
        this.imgBackground = new ImagePanel();
        this.imgBackground.setImage("back.png"); //Wenn dieser Aufruf auskommentiert wird geht's!??
        this.panBackground.add(this.imgBackground);
        
        //View zusammensetzen
        this.panGame.add(this.panCenter, BorderLayout.CENTER);
        this.panGame.add(this.panRight, BorderLayout.EAST);
        this.panGame.add(this.ControlBar, BorderLayout.WEST);
        this.panGame.add(this.panBottom, BorderLayout.SOUTH);
        this.getContentPane().add(panGame);
        this.getContentPane().add(panBackground);
    }

    private void CreatePanels()
    {
        //Game Panels für die Anzeige anlegen
        this.panGame = new JPanel();
        this.panBackground = new JPanel();
        this.panCenter = new JPanel();
        this.panRight = new JPanel();
        this.panBottom = new JPanel();
        
        //Layout der Game Panels setzen
        this.panGame.setLayout(new BorderLayout());
        this.panBackground.setLayout(new BorderLayout());
        this.panCenter.setLayout(new BoxLayout(this.panCenter, BoxLayout.Y_AXIS));
        this.panRight.setLayout(new BoxLayout(this.panRight, BoxLayout.Y_AXIS));
        this.panBottom.setLayout(new FlowLayout());

        //Vordergrund-Panel transparent machen
        this.panGame.setOpaque(false);
        this.panCenter.setOpaque(false);
        this.panRight.setOpaque(false);
        this.panBottom.setOpaque(false);
    }

Und die Klasse ImagePanel:

public class ImagePanel extends JPanel implements Serializable
{
    private Image image;
    private String strImage;
    
    /** Creates a new instance of ImagePanel */
    public ImagePanel()
    {
        //Transparent machen
        this.setOpaque(false);
        
        //Bild auf null setzen
        this.image = null;
        this.strImage = "";
    }

    public void paintComponent(Graphics g)
    {
        super.paintComponent(g);

        int iWidth  = this.getWidth();
        int iHeight = this.getHeight();
        
        //Falls ein Bild hinterlegt wurde zeichnen wir es
        if( this.image != null )
        {
            g.drawImage(this.image, 0, 0, iWidth, iHeight, this);
        }        
    }

    public String getImage()
    {
        return this.strImage;
    }

    public void setImage(String strImage)
    {
        if( strImage != null && !strImage.equals("") )
        {
            //Bild laden
            this.strImage = strImage;
            this.image = Toolkit.getDefaultToolkit().getImage(strImage);
        }
        
        //neuzeichnen
        this.repaint();
    }
}

Ohne den setImage Aufruf im Konstruktor der View wird die Oberfläche fehlerfrei gezeichnet

 

[Marco13]

Tja, compilierbar ist das ja nun nicht. Kannst ja mal ein bißchen mit "hauptfenster.validate()" rum-probieren.

 

[Marco13]

OK, so viel hat ja nicht gefehlt. Mir ist allerdings nicht ganz klar, warum du das vollkommen un-Kommentierte OverlayLayout dafür verwendest. WIE man dort die Alignements setzen muss, damit das gewünschte passiert, weiß ich nicht (und das läßt sich auch nicht aus der API-Doku rauslesen).

Was würde denn gegen sowas sprechen....

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


class MasterView extends JFrame
{
	public static void main(String args[])
	{
		MasterView mv = new MasterView();
		mv.setSize(400,400);
		mv.setVisible(true);
	}

	JToolBar ControlBar;
	JButton SelectButtons[];
	ImagePanel imgBackground;
	JPanel panGame;
	JPanel panCenter;
	JPanel panRight;
	JPanel panBottom;

    public MasterView()
    {
        this.setDefaultCloseOperation(WindowConstants.EXIT_ON_CLOSE);

        //Toolbar erstellen
        this.ControlBar = new JToolBar();
        this.ControlBar.setOrientation(JToolBar.VERTICAL);
        this.SelectButtons = new JButton[6];
        for( int i = 0; i < 6; i++ )
        {
            this.SelectButtons[i] = new JButton();
            this.ControlBar.add(this.SelectButtons[i]);
        }


        //Hintergrund anlegen
        this.imgBackground = new ImagePanel();
        this.imgBackground.setImage("back.png"); //Wenn dieser Aufruf auskommentiert wird geht's!??
        imgBackground.setLayout(new GridLayout(1,1));
        this.setContentPane(imgBackground);

        //Steuerelemente der Oberfläche anlegen
        CreatePanels();

        //View zusammensetzen
        this.panGame.add(this.panCenter, BorderLayout.CENTER);
        this.panGame.add(this.panRight, BorderLayout.EAST);
        this.panGame.add(this.ControlBar, BorderLayout.WEST);
        this.panGame.add(this.panBottom, BorderLayout.SOUTH);
        this.getContentPane().add(panGame);

    }

    private void CreatePanels()
    {
        //Game Panels für die Anzeige anlegen
        this.panGame = new JPanel();
        this.panCenter = new JPanel();
        this.panRight = new JPanel();
        this.panBottom = new JPanel();

        //Layout der Game Panels setzen
        this.panGame.setLayout(new BorderLayout());
        this.panCenter.setLayout(new BoxLayout(this.panCenter, BoxLayout.Y_AXIS));
        this.panRight.setLayout(new BoxLayout(this.panRight, BoxLayout.Y_AXIS));
        this.panBottom.setLayout(new FlowLayout());

        //Vordergrund-Panel transparent machen
        this.panGame.setOpaque(false);
        this.panCenter.setOpaque(false);
        this.panRight.setOpaque(false);
        this.panBottom.setOpaque(false);
    }

}

class ImagePanel extends JPanel
{
    private Image image;
    private String strImage;

    /** Creates a new instance of ImagePanel */
    public ImagePanel()
    {
        //Transparent machen
        this.setOpaque(false);

        //Bild auf null setzen
        this.image = null;
        this.strImage = "";
    }

    public void paintComponent(Graphics g)
    {
        super.paintComponent(g);

        int iWidth  = this.getWidth();
        int iHeight = this.getHeight();

        //Falls ein Bild hinterlegt wurde zeichnen wir es
        if( this.image != null )
        {
            g.drawImage(this.image, 0, 0, iWidth, iHeight, this);
        }
    }

    public String getImage()
    {
        return this.strImage;
    }

    public void setImage(String strImage)
    {
        if( strImage != null && !strImage.equals("") )
        {
            //Bild laden
            this.strImage = strImage;
            this.image = Toolkit.getDefaultToolkit().getImage(strImage);
        }

        //neuzeichnen
        this.repaint();
    }
}

(das ist übrigens compilierbar....)

 

[Snapshot]

Yepp, so gehts. Der entscheidende Tipp war die Oberflächenelemente direkt in das ImagePanel zu hängen und das OverlayLayout ganz wegzulassen. Danke!! Wobei es mit OverlayLayout eigentlich auch gehen sollte... egal.

P.S.: Das OverlayLayout ist in der aktuellen JDK-Dokumentation vollständig dokumentiert. Hier der Link: OverlayLayout

 

[Marco13]

Mit "nicht dokumentiert" meinte ich, dass die Dokumentation sich darauf beschränkt, dass neben den üblichen (vmtl. halbautomatisch generierten) "Kommentaren"

public void addLayoutComponent(Component comp,
                               Object constraints)
    Adds the specified component to the layout, using the specified constraint object. Used by this class to know when to invalidate layout.

(Aha!)
dort im wesentlichen nur steht: "Mit dem OverlayLayout kann man Layouten" :wink:


RICHTIG dokumentiert sind die anderen:

http://java.sun.com/docs/books/tutorial/uiswing/layout/layoutlist.html