Positionieren von Elementen in GridBagLayout

[CptK]

Hallo zusammen, ich habe folgenden Code:

private final Dimension size = new Dimension(250, 300);

private final String title = "Add Point";

private JLabel[] labels;
private String[] labelNames = { title, "x", "y", "z" };
private JTextField[] inputFields;
private JButton addButton;

public AddPointFrame(int dimenion) {
    labels = new JLabel[labelNames.length];
    inputFields = new JTextField[3];

    setPreferredSize(size);
    setResizable(false);
    setDefaultCloseOperation(DISPOSE_ON_CLOSE);
    GridBagLayout layout = new GridBagLayout();
    GridBagConstraints gbc;
    setLayout(layout);

    // header
    labels[0] = new JLabel(labelNames[0]);
    labels[0].setFont(new Font("Arial", 5, 20));
    GridBagConstraints gbcHeader = makegbc(0, 0, 2, 0);
    gbcHeader.anchor = GridBagConstraints.NORTH;
    gbcHeader.insets = new Insets(0, 20, 30, 20);
    layout.setConstraints(labels[0], gbcHeader);
    add(labels[0]);

    for (int i = 1; i < dimenion + 1; i++) {
        // Label
        gbc = makegbc(0, i, 1, 1);
        gbc.insets = new Insets(20, 40, 0, 0);
        labels[i] = new JLabel(labelNames[i]);
        layout.setConstraints(labels[i], gbc);
        add(labels[i]);

        // Text-field
        gbc = makegbc(1, 1, 1, 1);
        gbc.weightx = 100;
        gbc.insets = new Insets(20, 20, 0, 40);
        gbc.fill = GridBagConstraints.HORIZONTAL;
        inputFields[i - 1] = new JTextField();
        layout.setConstraints(inputFields[i - 1], gbc);
        add(inputFields[i - 1]);
        System.out.println(i - 1);
    }

    // Ende-Button
    addButton = new JButton("Add");
    addButton.setPreferredSize(new Dimension(size.width / 2, 20));
    gbc = makegbc(0, 4, 2, 0);
    gbc.insets = new Insets(40, 20, 40, 20);
    layout.setConstraints(addButton, gbc);
    add(addButton);

    pack();

    setVisible(true);
    repaint();
}

private GridBagConstraints makegbc(int x, int y, int width, int height) {
    GridBagConstraints gbc = new GridBagConstraints();
    gbc.gridx = x;
    gbc.gridy = y;
    gbc.gridwidth = width;
    gbc.gridheight = height;
    return gbc;
}

Für dimension = 1,2,3 bekomme ich nun folgende Ergebnisse (andere Dimensionen sind nicht vorgesehen, das Handling dafür kommt noch):

Direkt fällt natürlich auf, dass der immer nur ein JTextfield anzeigt, obwohl mehrere da sein sollten, ich verstehe aber nicht, wieso das nicht gemacht wird.
Außerdem habe ich ein paar Probleme beim anordnen der Komponenten: Der Titel soll bei allen auf der Höhe sein, auf der er bei dimension = 3 ist. Der Button soll bei allen auf der selben Höhe sein und zwar ungefähr seine jetzige Position bei dimension = 3 minus seiner Höhe und die Labels + dazugehörige TextFields sollen zwischen Header und Button horizontal zentriert werden.

 

[LimDul]

Deine TExtfelder werden alle in 1,1 erzeugt - da fehlt ein ein i als Parameter beim GBC.

 

[LimDul]

Bezüglich der vertikalen Anordnung. Du setzt nirgendwo weighty. Das führt dazu, dass der verbleibende Platz, den die Komponenten nicht brauchen gleichmäßig oben und unten ergänz wird

Unless you specify at least one non-zero value for weightx or weighty, all the components clump together in the center of their container. This is because when the weight is 0.0 (the default), the GridBagLayout puts any extra space between its grid of cells and the edges of the container.

Das heißt, du musst du überlegen, welche Komponente wie den Platz zugeschlagen bekommt (z.B. bei der obersten mit dem Label weighty setzen)

 

[CptK]

So ich habe die Sachen mal versucht umzusetzen:

// header
labels[0] = new JLabel(labelNames[0]);
labels[0].setFont(new Font("Arial", 5, 20));
GridBagConstraints gbcHeader = makegbc(0, 0, 2, 0);
gbcHeader.anchor = GridBagConstraints.NORTH;
gbcHeader.insets = new Insets(0, 20, 50, 20);
gbcHeader.weighty = 20;
layout.setConstraints(labels[0], gbcHeader);
add(labels[0]);

for (int i = 1; i < dimension+ 1; i++) {
    // Label
    gbc = makegbc(0, i, 1, 1);
    gbc.anchor = GridBagConstraints.CENTER;
    gbc.weighty = 60 / dimension;
    gbc.fill = GridBagConstraints.VERTICAL;
    gbc.insets = new Insets(20, 40, 0, 0);
    labels[i] = new JLabel(labelNames[i]);
    layout.setConstraints(labels[i], gbc);
    add(labels[i]);

    // Text-field
    gbc = makegbc(1, i, 1, 1);
    gbc.anchor = GridBagConstraints.CENTER;
    gbc.weightx = 100;
    gbc.weighty = 60 / dimension;
    gbc.insets = new Insets(20, 20, 0, 40);
    gbc.fill = GridBagConstraints.HORIZONTAL;
    inputFields[i - 1] = new JTextField();
    layout.setConstraints(inputFields[i - 1], gbc);
    add(inputFields[i - 1]);
}

// Ende-Button
addButton = new JButton("Add");
addButton.setPreferredSize(new Dimension(size.width / 2, 20));
gbc = makegbc(0, 4, 2, 0);
gbc.weighty = 20;
gbc.insets = new Insets(40, 20, 40, 20);
layout.setConstraints(addButton, gbc);
add(addButton);

es sieht auch schon wesentlich schöner aus:

Wenn ich jetzt aber versuche dem Header noch einen Margin zu geben, also sowas wie

gbcHeader.insets = new Insets(20, 20, 20, 20);

macht der das nicht so, wie es sein sollte:

 

[mihe7]

Wenn ich Dich richtig verstehe, willst Du etwas wie (Achtung: es folgt quick & dirty-Code):

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

public class Test {

    public void run() {
        String[] labels = {"Add Point", "x", "y", "z"};

        Insets labelInsets = new Insets(5,20,5,5);
        Insets fieldInsets = new Insets(5,0,5,20);

        JPanel panel = new JPanel(new GridBagLayout());
        GridBagConstraints gbc = new GridBagConstraints();

        gbc.fill = GridBagConstraints.HORIZONTAL;

        for (int i = 1; i < labels.length; i++) {
            JLabel label = new JLabel(labels[i]);
            gbc.gridy = i;
            gbc.gridx = 0;
            gbc.weightx = 0;
            gbc.insets = labelInsets;
            panel.add(label, gbc);

            gbc.gridx = 1;
            gbc.weightx = 1;
            gbc.insets = fieldInsets;
            panel.add(new JTextField(30), gbc);
        }

        JLabel header = new JLabel(labels[0]);
        header.setFont(header.getFont().deriveFont(20f));
        gbc.gridx = 0;
        gbc.gridy = 0;
        gbc.gridwidth = 2;
        gbc.fill = GridBagConstraints.NONE;
        gbc.anchor = GridBagConstraints.PAGE_END;
        gbc.insets = new Insets(20,0,20,0);
        panel.add(header, gbc);

        JButton button = new JButton("Add");
        gbc.gridx=0;
        gbc.gridy=labels.length;
        gbc.gridwidth=2;
        gbc.weighty=1;
        gbc.anchor = GridBagConstraints.PAGE_START;
        panel.add(button, gbc);        


        JFrame frame = new JFrame();
        frame.setDefaultCloseOperation(JFrame.DISPOSE_ON_CLOSE);
        frame.add(panel);
        frame.pack();
        frame.setVisible(true);
    }

    public static void main(String[] args) {
        SwingUtilities.invokeLater(() -> new Test().run());
    }
}