JLayeredPanel - Hilfe

[TobYMaier92]

Hi Leute,

ich versuche mich gerade daran, eine Ampel zu programmieren. Mein Vorgehen: JLayeredPanel erzeugen, ein JLabel mit einem Bild darauf legen (welches den Rahmen der Ampel darstellt) und schließlich noch ein JPanel erstellen, auf dem die aktuelle Farbe der Ampel dargestellt wird. Dabei geschieht letzteres durch die Methode paintComponent(Graphics g).
Ich programmiere mit Eclipse. In einer anderen Klasse wird ein Objekt von der hier dargestellten Klasse erzeugt.
Ich kann leider selbst nicht erkennen, was ich falsch mache. Gegoogled habe ich schon genug. Ich vermute es hat irgendwas mit dem JLayeredPanel zu tun. Vielleicht kann jemand kurz über den Quellcode schauen und mir einen Tipp geben. Ich wäre sehr dankbar.

Gruß,
Tobi

 import javax.swing.BorderFactory;
import javax.swing.ImageIcon;
import javax.swing.JLabel;
import javax.swing.JLayeredPane;
import javax.swing.JPanel;
import java.awt.Color;
import java.awt.FlowLayout;
import java.awt.Graphics;



public class JUnitStatusBar extends JPanel
{
	//Erste Komponente der JUnitStatusBar
	JLayeredPane ampelPanel; 
	JLabel ampelRahmen;	
	JPanel ampelLicht;
	
public JUnitStatusBar ()
{	
	//Erzeugung der JUnitStatusBar
	this.setLayout(new FlowLayout(FlowLayout.LEFT));
	this.setBorder(BorderFactory.createLineBorder(Color.gray));
	setBackground(Color.WHITE); 
	
	
	//Erzeugung der 1. Komponente der JUnitStatusBar: Ampel
	ampelPanel = new JLayeredPane ();
	
	ImageIcon ampelbild = new ImageIcon ("C:/ampelrahmen.png");
	ampelRahmen = new JLabel(ampelbild);	
	ampelRahmen.setBounds(20, 20, 10, 10);		
	ampelPanel.add(ampelRahmen, new Integer (3));	
	
	ampelLicht = new Ampel ();
	ampelPanel.add(ampelLicht, new Integer (4));
	
	this.add(ampelPanel);
	
}

class Ampel extends JPanel
{		
	public Ampel ()
	{
		setOpaque(false);
	}
	
	public void paintComponent(Graphics g)
	{		
		g.setColor(Color.GREEN);		
		g.drawOval(0, 0, 5, 5);      
		g.fillOval(0, 0, 5, 5);
	}
}

 

[ChrisKu]

Ich habe Deinen Code mal getestet. Deine paintComponent() Methode wird m.E. gar nicht aufgerufen. Mit folgenden Änderungen scheint es zu gehen.

In der Ampel Klasse die paint() Methode überschreiben

public void paint(Graphics g) {
            super.paint(g);
            g.setColor(Color.GREEN);
            g.drawOval(0, 0, 5, 5);
            g.fillOval(0, 0, 5, 5);
        }

super.paint(g) nicht vergessen, so was fehlte bei Dir. Dann die paint() Methode der JUnitStatusBar überschreiben

public void paint(Graphics g) {
        super.paint(g);
        ampelLicht.paint(g);
    }

 

[Gast2]

In der Ampel Klasse die paint() Methode überschreiben

Bei SwingKomponenten immer paintComponent() überschreiben!

 

[ChrisKu]

Bei SwingKomponenten immer paintComponent() überschreiben

Danke für den Hinweis, aber gibt es dafür auch eine Erklärung? Bei dem Problem des TO scheint es nähmlich genau damit nicht zu gehen....

 

[Gast2]

Danke für den Hinweis, aber gibt es dafür auch eine Erklärung?

Ja, die gibts: Painting in AWT and Swing

Swing programs should override paintComponent() instead of overriding paint(). Although the API allows it, there is generally no reason to override paintBorder() or paintComponents() (and if you do, make sure you know what you're doing!). This factoring makes it easier for programs to override only the portion of the painting which they need to extend. For example, this solves the AWT problem mentioned previously where a failure to invoke super.paint() prevented any lightweight children from appearing.

 

[Michael...]

Vielleicht kann jemand kurz über den Quellcode schauen und mir einen Tipp geben.

Ampel bzw. ampelLicht hat eine Größe von 0x0. Man muss ja mindestens die Größe (bei Bedarf auch die Position) definieren, damit JLayeredPane auch weiss, wo die Komponente zu zeichnen ist.

 

[ChrisKu]

So, ich glaube, nun weiß ich, wo der Fehler im Code des TO liegt.

1. Müssen wohl bei allen Komponenten, die einem JLayeredPane zugefügt werden, die Bounds gesetzt werden. Das ist beim AmpelLicht nicht geschehen. Also nach der Erzeugung von ampelLicht noch ampelLicht.setBounds(....) einfügen.
2. Muss dem JLayeredPane noch ein Größe mitgegeben werden, also ampelPanel.setPreferredSize(...)

Dann funktioniert es, zumindest bei mir, auch ohne überschreiben der paint Methode

Ohne die Bounds wie unter 1. beschrieben zu setzen, wird Ampel.paintComponent() nicht aufgerufen.

Hope this helps.

 

[TobYMaier92]

Hey Leute,

die Ampel funktioniert genau so wie ich es haben wollte. Vielen, vielen Dank für eure Hilfe und noch nen schönen Pfingstmontag.

Viele Grüße,

Tobi