GridBagLayout Problem

[Guest]

Hallo zusammen,

ich habe mal zwei Dinge mit GBL ausprobiert. Eine Sache macht mir aber noch Sorgen/Probleme.

Erste Variante:

....
l1 = new JLabel("Name:");
			l1.setPreferredSize(new Dimension(150,25));
			l1.setHorizontalAlignment(JLabel.CENTER);
			l1.setBackground(Color.red);
			l1.setOpaque(true);
			GridBagConstraints gbc = new GridBagConstraints();
			gbc.gridx = 0;
			gbc.gridy = 0;
			gbc.gridwidth = 1;
			gbc.gridheight = 1;
			gbc.weightx = 1.0;
			gbc.weighty = 1.0;			
			gbc.fill = GridBagConstraints.BOTH;
			gb.setConstraints(l1, gbc);
			panel1.add(l1);
			
			tf1 = new JTextField();
			tf1.setPreferredSize(new Dimension(120,25));
			GridBagConstraints gbcTextField = new GridBagConstraints();
				gbcTextField.gridx = 1;
				gbcTextField.gridy = 0;
				gbcTextField.gridwidth = 1;
				gbcTextField.gridheight = 1;
				gbcTextField.fill = GridBagConstraints.NONE;
				gb.setConstraints(tf1, gbcTextField);
				panel1.add(tf1);

hierbei verhält sich das JLabel l1 so, dass es sich über seine ganze Zellenbreite ausdehnt und das TextField
ganz nach links verschiebt. So weit so gut, das ist mir einleuchtend.

Aber in diesem Code:

....
static void addComponentLabel(JPanel p, JLabel l, int x, int y, int width, int height,
								int padx, int pady, double wx, double wy)
	{
		GridBagLayout gb = new GridBagLayout();
		GridBagConstraints gbcLabel = new GridBagConstraints();
		p.setLayout(gb);
		
		gbcLabel.gridx = x;
		gbcLabel.gridy = y;
		gbcLabel.gridwidth = width;
		gbcLabel.gridheight = height;
		gbcLabel.ipadx = padx;
		gbcLabel.ipady = pady;
		gbcLabel.weightx = wx;
		gbcLabel.weighty = wy;
		gbcLabel.fill = GridBagConstraints.BOTH;//for example: GridBagConstraints.BOTH;		
			gb.setConstraints(l, gbcLabel);
			p.add(l);//p --> Panel, component --> anythink Component (for example: JButton)		
	}//Ende der Methode
	
	static void addComponentTextField(JPanel p, JTextField f, int x, int y, int width, int height,
			int padx, int pady, double wx, double wy)
	{
		GridBagLayout gb = new GridBagLayout();
		GridBagConstraints gbc = new GridBagConstraints();
		p.setLayout(gb);
		
		gbc.gridx = x;
		gbc.gridy = y;
		gbc.gridwidth = width;
		gbc.gridheight = height;
		gbc.ipadx = padx;
		gbc.ipady = pady;
		gbc.weightx = wx;
		gbc.weighty = wy;
		gbc.fill = GridBagConstraints.NONE;//for example: GridBagConstraints.BOTH;
		gb.setConstraints(f, gbc);
		p.add(f);//p --> Panel, component --> anythink Component (for example: JButton)
		
	}
                label1 = new JLabel("Name:");
		label1.setHorizontalAlignment(JLabel.LEFT);
		label1.setPreferredSize(new Dimension(100, 27));
		label1.setOpaque(true);
		label1.setBackground(Color.red);//                         
		GridBagLayout1.addComponentLabel(panel1, label1, 0, 0, 1, 1, 0,  0, 1.0, 1.0);

		tf1 = new JTextField();
		tf1.setPreferredSize(new Dimension(150, 25));
		GridBagLayout1.addComponentTextField(panel1, tf1, 1, 0, 1, 1, 0,  0, 0.0, 0.0);

macht das JLabel label1 nichts. Erst, wenn ich das JTextField tf1 auf weightx/y = 1.0; und
GridBagConstraints.BOTH setze, dann verhält sich das JTextField tf1 wie sich eigentlich das
JLabel label1 verhalten sollte, über beide Richtungen.

Es ist so, als ob JLabel label1 überhaupt nicht auf weightx/y = 1.0 und GridBagConstraints.BOTH
reagieren würde, obwohl doch die Methode "addComponentLabel(....) zuerst aufgerufen wird.

Kann mir jemand sagen warum das so ist und ob es überhaupt sinnvoll ist über Methoden
die Membervariablen von GridBagConstraints zu initialisieren.

Meiner Meinung nach, wäre es schon sinnvoll, da nur immer eine Methode für z.B. fünf
Button oder 10 JTextFields geschireben werden müsste.

Danke, chris

 

[SlaterB]

durch den Aufruf von Zeile 52 und dann Zeile 26 setzt du ein neues zweites GridBagLayout,

das erste Layout, welches sich all die wichtigen Informationen zum JLabel gemerkt hat,
wurde gelöscht

2x GridBagLayout setzen kann nicht gutgehen


streiche Zeile 6, 8, 26 und 28 und setze das Layout woanders vorher GENAU EINMAL

verwende
p.add(component, constraint);
dann brauchst du in den Untermethoden auch keine Referenz auf das Layout

 

[Guest]

Hi SlaterB,

danke für deine Hilfe.

Hab es so ausprobiert...

....
static void addComponentLabel(GridBagLayout g, JPanel p, JLabel l, int x, int y, int width, int height,
								int padx, int pady, double wx, double wy, int contentFill)
	{
		GridBagConstraints gbcLabel = new GridBagConstraints();
		
		gbcLabel.gridx = x;
		gbcLabel.gridy = y;
		gbcLabel.gridwidth = width;
		gbcLabel.gridheight = height;
		gbcLabel.ipadx = padx;
		gbcLabel.ipady = pady;
		gbcLabel.weightx = wx;
		gbcLabel.weighty = wy;
		gbcLabel.fill = contentFill;//for example: GridBagConstraints.BOTH;		
			g.setConstraints(l, gbcLabel);
			p.add(l);//p --> Panel, component --> anythink Component (for example: JButton)		
	}
....
public void gui(){
c =  getContentPane();
	
	panel1 = new JPanel();
	c.add(panel1);
	
	
		gb = new GridBagLayout();//GridBag-Instanz erzeugen
		panel1.setLayout(gb);
	
		label1 = new JLabel("Name:");
		label1.setHorizontalAlignment(JLabel.LEFT);
		label1.setPreferredSize(new Dimension(100, 27));
		label1.setOpaque(true);
		label1.setBackground(Color.red);
       GridBagLayout1.addComponentLabel(gb, panel1, label1, 0, 0, 1, 1, 0,  0, 1.0, 1.0, GridBagConstraints.BOTH);
}

und es hat funktioniert. Das JLabel label1 verhält sich so, wie geplant. :toll:

2x GridBagLayout setzen kann nicht gutgehen

kann auch nicht gut gehen.

setze das Layout woanders vorher GENAU EINMAL

Hab es in die Methode, wo alle swing-Komponenten enthalten sind, hineingelgt.


Lohnt es sich dennüberhaupt solche Methoden zu produzieren (von der Performence)
oder ist das eher nachteilig??



Gruß chris

 

[SlaterB]

Performance wäre völlig schnuppe und Methoden auszulager kostet überhaupt keine Performance,

Hilfsmethode sind im Allgemeinen eine sehr gute Sache

allerdings sind selbst addComponentLabel und addComponentTextField schon zu groß,
beide enthalten einen gleichen Block

gbc.gridx = x;
gbc.gridy = y;
gbc.gridwidth = width;
gbc.gridheight = height;
gbc.ipadx = padx;
gbc.ipady = pady;
gbc.weightx = wx;
gbc.weighty = wy;

bzw unterscheiden sich anscheinend überhaupt nur durch BOTH vs. NONE,
was etwas fragwürdig ist,

zumindest solltest du das ganze dann umbauen in

   static void addComponent(JPanel p, JComponent comp, int x, int y, int width, int height,
                        int padx, int pady, double wx, double wy, int fill)
   {
      GridBagConstraints c = new GridBagConstraints();
      c.gridx = x;
      c.gridy = y;
      c.gridwidth = width;
      c.gridheight = height;
      c.ipadx = padx;
      c.ipady = pady;
      c.weightx = wx;
      c.weighty = wy;
      c.fill = fill;

      p.add(comp,c);
   }


   static void addComponentLabel(JPanel p, JLabel l, int x, int y, int width, int height,
                        int padx, int pady, double wx, double wy)
   {
     addComponent(p, l, x, y, width, height, padx, pady, wx, wy, GridBagConstraints.BOTH)
   }

   static void addComponentTextField(JPanel p, JTextField f, int x, int y, int width, int height,
         int padx, int pady, double wx, double wy)
   {
        addComponent(p, f, x, y, width, height, padx, pady, wx, wy, GridBagConstraints.NONE)
   }

weiterhin: je weniger Parameter, desto besser,
wenn exotische Dinge wie ipadx nicht gebraucht werden, dann Parameter weg, Standardwert 0 setzen,

wenn die Methoden nicht statisch wären sondern in einer abgeleiteten Klasse GridBagPanel extends JPanel,
dann könnte der Parameter p wegfallen usw.

 

[Guest]

hallo,

danke für deinen Vorschlag.

Ich hatte es auch schon mal so versucht:

 static void addComponent(JPanel p, JComponent comp, int x, int y, int width, int height,
                        int padx, int pady, double wx, double wy, int fill) 
//mit JComponent comp

aber irgendwie hatte ich dabei Probleme.

Ich werde es auf jeden Fall versuchen. Ich bin gerade dabei 8 JLabels über eine
for-Schleife auf ein Panel zu bringen... Werde mich bestimmt noch mal melden :roll:

Meine Idee ist, drei Panels anzuordnen und auf diesen dann geweils verschiedene Gridbaglayouts
zu erstellen. Ist nur reine Übungssache.

Danke noch mal,

Gruß chris :roll: