BoxLayout wird auf JPanel nicht erstellt

[Antoras]

Hallo,

ich hab ein kleines Problem mit ein paar Componenten, die auf einem JPanel dargestellt werden sollen. Die JPanel sind mit einem BoxLayout untereinander angeordnet und Fehlermeldungen seitens des Compilers gibt es nicht.

Zuvor hab ich die Panels statt auf ein weiteres Grundpanel auf ein JFrame gezeichnet, da hab ich aber irgendeine Fehlermeldung bekommen, laut der man ein BoxLayout nicht direkt auf ein JFrame zeichnen darf.

Kann mir jemand sagen, warum zwar das JPanel erstellt, aber sonst nichts weiter gezeichnet wird?

import java.awt.Dimension;
import java.awt.EventQueue;
import java.awt.FlowLayout;
import java.awt.event.ActionEvent;
import java.awt.event.ActionListener;

import javax.swing.BoxLayout;
import javax.swing.JButton;
import javax.swing.JFrame;
import javax.swing.JPanel;
import javax.swing.border.TitledBorder;


public class Panel extends JPanel{
	
	private JPanel pnlForm = null;
	private JPanel pnlComponent = null;
	private JButton btn = null;
	
	/**
	 * Launch the application
	 * @param args
	 */
	public static void main(final String args[]) {
		EventQueue.invokeLater(new Runnable() {
			public void run() {
				try {
					new Panel(530, 375);
				} catch (final Exception e) {
					e.printStackTrace();
				}
			}
		});
	}

	/**
	 * Create the frame
	 */
	public Panel(final int w, final int h) {
		super();
		final JFrame f = new JFrame("Panel");
		f.setPreferredSize(new Dimension(w, h));
		f.setLocation(100, 100);
		f.setDefaultCloseOperation(JFrame.EXIT_ON_CLOSE);
		try {
			initGUI();
		} catch (final Throwable e) {
			e.printStackTrace();
		}
		f.pack();
		f.setVisible(true);
	}
	
	private void initGUI() throws Exception {
		this.setLayout(new BoxLayout(this, BoxLayout.Y_AXIS));		
		initPanelForm();
		initPanelComponent();
	}
	
	private JPanel getPanelForm() {
		if (this.pnlForm != null) 
			{ return this.pnlForm; }
		final JPanel p = new JPanel(new FlowLayout());
		p.setBorder(new TitledBorder(null, "Form", 
				TitledBorder.DEFAULT_JUSTIFICATION,
				TitledBorder.DEFAULT_POSITION,
				null, null));		
		add(p);
		this.pnlForm = p;
		return this.pnlForm;
	}
	
	private JPanel getPanelComponent() {
		if (this.pnlComponent != null) 
			{ return this.pnlComponent; }
		final JPanel p = new JPanel(new FlowLayout());
		p.setBorder(new TitledBorder(null, "Component", 
				TitledBorder.DEFAULT_JUSTIFICATION,
				TitledBorder.DEFAULT_POSITION,
				null, null));
		add(p);
		this.pnlComponent = p;
		return this.pnlComponent;
	}

	private void initPanelForm() {
		getPanelForm();
	}

	private void initPanelComponent() {
		getPanelComponent();
		getButton();
	}
	
	private JButton getButton() {
		if (this.btn != null) 
			{ return this.btn; }
		final JButton b = new JButton("Button");
		pnlComponent.add(b);
		this.btn = b;
		return this.btn;
	}
}

 

[SlaterB]

du initialisiert mehr oder wenig gut das eigene this-Panel-Objekt,
dieses wird aber nicht in das angezeigte JFrame eingefügt

 

[Antoras]

Ach verdammt, ich bin bisher immer davon ausgegangen, dass automatisch auf das JFrame gezeichnet wird wenn ich dieses auf einem JPanel erstelle. Eigentlich Schwachsinn, total unlogisch.
Den gleichen Fehler hatte ich aber schon einmal, jetzt weiß ich wenigstens was falsch war. Danke dir.

Einfach folgende Zeile dem Konstruktor hinzufügen:

f.add(this);

Ansonsten, falls es an meinem Code gravierende stilistische Mängel gibt, dann einfach melden.

 

[Ebenius]

Die klassischen Nomenklaturfehler machste schonmal nicht. Die Klasse "Panel" würde ich nicht so benennen, da

1. es schon eine solche gibt, im java.awt-Package, und
2. der Name nicht's weiter aussagt.

Außerdem würde ich nicht im Konstruktor einer Panel-Klasse ein JFrame aufbauen. Das gehört da nicht hin. Der Konstruktor einer Klasse sollte eine Instanz der Klasse erzeugen und sinnvoll initialisieren. Ein Panel ist dazu da, es auf ein anderes Panel zu werfen, oder auf ein Fenster. Aber es sollte es nicht selbst tun, genauso wie Käse kein Brot bäckt, schneidet und sich dann drauf legt. Wenn man plötzlich Butter statt Rama möchte, ist man sprichwörtlich angeschmiert. :lol:

 

[Antoras]

Die klassischen Nomenklaturfehler machste schonmal nicht. Die Klasse "Panel" würde ich nicht so benennen, da

1. es schon eine solche gibt, im java.awt-Package, und
2. der Name nicht's weiter aussagt.

Das ist klar. Die gezeigte Klasse war auch mehr ein Beispielprojekt, das ich schnell zusammengebastelt habe um mein Problem "kompakt" zu erläutern.

Außerdem würde ich nicht im Konstruktor einer Panel-Klasse ein JFrame aufbauen. Das gehört da nicht hin. Der Konstruktor einer Klasse sollte eine Instanz der Klasse erzeugen und sinnvoll initialisieren.

Das mache ich in meiner Mainklasse(die zudem meist mein GUI beheimatet) eigentlich immer so, da ich sonst nichts weiter in dieser Klasse mache. Es hat für mich bisher einfach keinen Sinn gemacht dewegen eine neue Methode zu erstellen, wobei es auch nicht schwer wäre es zu ändern, da ich sowieso immer noch ne Methode hab um das komplette GUI zu initialisieren.

Ein Panel ist dazu da, es auf ein anderes Panel zu werfen, oder auf ein Fenster. Aber es sollte es nicht selbst tun, genauso wie Käse kein Brot bäckt, schneidet und sich dann drauf legt. Wenn man plötzlich Butter statt Rama möchte, ist man sprichwörtlich angeschmiert. :lol:

Du meinst, dass es nicht sinnvoll ist ein Frame auf ein Panel zu zeichnen? Leuchtet ein, ich hab aber immer eine Fehlermeldung bekommen, wenn ich das BoxLayout direkt auf das JFrame gezeichnet habe. Aber gerade eben hab ich das auch gelöst

Besser?

import java.awt.Dimension;
import java.awt.EventQueue;
import java.awt.FlowLayout;
import java.awt.event.ActionEvent;
import java.awt.event.ActionListener;

import javax.swing.BoxLayout;
import javax.swing.JButton;
import javax.swing.JFrame;
import javax.swing.JPanel;
import javax.swing.border.TitledBorder;


public class JFrameDemo extends JFrame{
	
	private JPanel pnlForm = null;
	private JPanel pnlComponent = null;
	private JButton btn = null;
	
	/**
	 * Launch the application
	 * @param args
	 */
	public static void main(final String args[]) {
		EventQueue.invokeLater(new Runnable() {
			public void run() {
				try {
					new JFrameDemo ();
				} catch (final Exception e) {
					e.printStackTrace();
				}
			}
		});
	}

	/**
	 * Create the frame
	 */
	public JFrameDemo() {
		super("Geometry Forms");
		try {
			initGUI();
		} catch (final Throwable e) {
			e.printStackTrace();
		}
	}
	
	private void initGUI() throws Exception {
		getContentPane().setPreferredSize(new Dimension(200, 200));
		setLocation(100, 100);
		setDefaultCloseOperation(JFrame.EXIT_ON_CLOSE);
		setLayout(new BoxLayout(getContentPane(), BoxLayout.Y_AXIS));
		initPanelForm();
		initPanelComponent();
		pack();
		setVisible(true);
	}
	
	private JPanel getPanelForm() {
		if (this.pnlForm != null) 
			{ return this.pnlForm; }
		final JPanel p = new JPanel(new FlowLayout());
		p.setBorder(new TitledBorder(null, "Form", 
				TitledBorder.DEFAULT_JUSTIFICATION,
				TitledBorder.DEFAULT_POSITION,
				null, null));
		add(p);
		this.pnlForm = p;
		return this.pnlForm;
	}
	
	private JPanel getPanelComponent() {
		if (this.pnlComponent != null) 
			{ return this.pnlComponent; }
		final JPanel p = new JPanel(new FlowLayout());
		p.setBorder(new TitledBorder(null, "Component",
				TitledBorder.DEFAULT_JUSTIFICATION,
				TitledBorder.DEFAULT_POSITION,
				null, null));
		add(p);
		this.pnlComponent = p;
		return this.pnlComponent;
	}

	private void initPanelForm() {
		getPanelForm();
	}

	private void initPanelComponent() {
		getPanelComponent();
		getButton();
	}
	
	private JButton getButton() {
		if (this.btn != null) 
			{ return this.btn; }
		final JButton b = new JButton("Button");
		pnlComponent.add(b);
		this.btn = b;
		return this.btn;
	}
}