Dynamische Erstellung von buttons in Java

Ich versuche, dynamisch zu generieren, eine form. Im Grunde, möchte ich laden Sie eine Liste der Elemente für den Kauf, und erzeugen Sie eine Schaltfläche für jeden. Ich kann bestätigen, dass die buttons generiert werden, mit dem debugger, aber Sie werden nicht angezeigt. Dies ist innerhalb einer Unterklasse von JPanel:

private void generate() {
    JButton b = new JButton("height test");
    int btnHeight = b.getPreferredSize().height;
    int pnlHeight = this.getPreferredSize().height;
    int numButtons = pnlHeight / btnHeight;

    setLayout(new GridLayout(numButtons, 1));

    Iterator<Drink> it = DrinkMenu.iterator();

    for (int i = 0; i <= numButtons; ++i) {
        if (!it.hasNext()) {
            break;
        }
        final Drink dr = it.next();
        b = new DrinkButton(dr);
        b.addActionListener(new ActionListener() {
            public void actionPerformed(ActionEvent e) {
                order.addDrink(dr);
        }});
        add(b);
    }
    revalidate();
}

DrinkButton ist eine Unterklasse von JButton. Irgendwelche Ideen?

Warum sind Sie der Einstellung b, um einen JButton, dann unten Einstellung es wieder zu DrinkButton
stellen Sie sicher, dass diese änderungen geschehen im Event-Dispatch-Thread
Der basic-code sieht gut aus. Sie sind mit revalidate (), die ein Schlüssel ist. Aber wir kennen nicht den Kontext, wie dieser code verwendet wird. Wir nicht, wenn diese Platte tatsächlich wurde Hinzugefügt, um den frame. Posten Sie Ihre SSCCE (sscce.org), die das problem veranschaulicht.

InformationsquelleAutor Kevin Lacquement | 2011-06-17

  1. 3

    Funktioniert auf meinem computer...

    public class Panel extends JPanel {

    public Panel() {
        setLayout(new java.awt.GridLayout(4, 4));
        for (int i = 0; i < 16; ++i) {
            JButton b = new JButton(String.valueOf(i));
            b.addActionListener(new java.awt.event.ActionListener() {
                public void actionPerformed(java.awt.event.ActionEvent e) {
                    //...
                }
            });
            add(b);
        }
    }

    public static void main(String[] args) {
        SwingUtilities.invokeLater(new Runnable(){
            @Override
            public void run(){
                JFrame frame = new JFrame();
                frame.setDefaultCloseOperation(JFrame.EXIT_ON_CLOSE);
                frame.setSize(new Dimension(300, 300));
                frame.add(new Panel());
                frame.setVisible(true);
            }
        });
    }
}

    Soweit ich mich erinnere-Ihre version war so gut arbeiten, obwohl ich hatte zu entfernen, Ihre "Trink" - code. Starten Sie aus diesem Beispiel (es weist ein schönes Gitter von 4x4 Tasten) und bestimmen, was ist falsch mit Ihrem code.

    sollten Sie wirklich die JFrame usw. in der EDT
    Ich wusste nicht, über, dass, nahm ich einfach die erste Swing-tutorial habe ich gefunden, um den code laufen, es ist lange her, seit ich das Letzte mal in diesem Rahmen. Ich Schätze deine Bearbeitung, vielen Dank!
    gerne helfen! 🙂
    Gutes Beispiel, aber es hat nicht wirklich was mit der dynamischen Anforderung.
    Es stellt sich heraus, dass mein problem war das einfädeln habe ich nicht invokeLater().

    InformationsquelleAutor Tomasz Nurkiewicz

  2. 3

    Können Sie revalidate(), wie in diesem Beispiel.

    Nachtrag: Hier ist meine variation auf @mKorbel interessant Antwort , zeigt ein ähnliches Ergebnis für GridLayout. Es sieht aus wie repaint() kann notwendig sein, nach revalidate().

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

/** @see https://stackoverflow.com/questions/6395105 */
public class ValidateRevalidateRepaint {

    private JPanel center;
    private boolean validate = false;
    private boolean revalidate = true;
    private boolean repaint = true;

    public ValidateRevalidateRepaint() {
        center = new JPanel(new GridLayout(1, 0, 10, 10));
        JFrame f = new JFrame();
        f.setTitle("VRR");
        f.setDefaultCloseOperation(JFrame.EXIT_ON_CLOSE);
        f.add(center, BorderLayout.CENTER);
        f.add(getRadioPanel(), BorderLayout.EAST);
        f.add(getCheckBoxPanel(), BorderLayout.SOUTH);
        makeChange(4);
        f.pack();
        f.setLocationRelativeTo(null);
        f.setVisible(true);
    }

    private JPanel getRadioPanel() {
        JPanel panel = new JPanel(new GridLayout(0, 1));
        ButtonGroup group = new ButtonGroup();
        ActionListener l = new ActionListener() {

            @Override
            public void actionPerformed(ActionEvent e) {
                JRadioButton radio = (JRadioButton) e.getSource();
                int n = Integer.parseInt(radio.getActionCommand());
                makeChange(n);
            }
        };
        for (int j = 0; j < 4; j++) {
            String s = String.valueOf(j + 1);
            JRadioButton radio = new JRadioButton(s);
            radio.setActionCommand(s);
            radio.addActionListener(l);
            group.add(radio);
            panel.add(radio);
            if (j == 3) {
                group.setSelected(radio.getModel(), true);
            }
        }
        return panel;
    }

    private JPanel getCheckBoxPanel() {
        final String[] operations = {"validate", "revalidate", "repaint"};
        ActionListener l = new ActionListener() {

            @Override
            public void actionPerformed(ActionEvent e) {
                JCheckBox checkBox = (JCheckBox) e.getSource();
                String ac = checkBox.getActionCommand();
                boolean state = checkBox.isSelected();
                if (ac.equals("validate")) {
                    validate = state;
                }
                if (ac.equals("revalidate")) {
                    revalidate = state;
                }
                if (ac.equals("repaint")) {
                    repaint = state;
                }
            }
        };
        JPanel panel = new JPanel();
        for (int j = 0; j < operations.length; j++) {
            JCheckBox check = new JCheckBox(operations[j]);
            if (j == 0) {
                check.setSelected(false);
            } else {
                check.setSelected(true);
            }
            check.setActionCommand(operations[j]);
            check.addActionListener(l);
            panel.add(check);
        }
        return panel;
    }

    private void makeChange(int number) {
        center.removeAll();
        for (int j = 0; j < number; j++) {
            center.add(getFiller());
        }
        if (validate) {
            center.validate();
        }
        if (revalidate) {
            center.revalidate();
        }
        if (repaint) {
            center.repaint();
        }
    }

    private JPanel getFiller() {
        JPanel panel = new JPanel();
        panel.setBorder(BorderFactory.createLineBorder(Color.blue, 5));
        panel.setBackground(Color.red);
        panel.setPreferredSize(new Dimension(50, 50));
        return panel;
    }

    public static void main(String[] args) {
        EventQueue.invokeLater(new Runnable() {

            @Override
            public void run() {
                new ValidateRevalidateRepaint();
            }
        });
    }
}
    Ich kann nicht widerstehen ..., : -), da sah ich ein paar Zeiten, die Sie wirklich hassen muss repaint(), ist es nicht :-), dein Beispiel ist OP der korrekte Basis des/way, +1
    Ich Schätze Ihre Beobachtungen, erzählen mir immer wenn Sie sehen ein Ort, wo repaint() wäre besser.
    trashgod das ist über meine schnelle Hände, mein Fehler, Wenn ich lese, dass wieder (große Dummheit schrieb ich hier), ein großes sorry Mann, richtig gemeint "revalidate()" + repaint(), Stelle ich, für alle coumpound/composite JComponents f.e. JComboBox (oft mich Total wütend), zurück zu meiner Frage, warum hast ' N verwenden repaint();
    Ah, wenn ich das richtig verstehe revalidate() nicht immer erfordern eine anschließende repaint(); für example.
    trashgod ja, das ist manchmal eigene Gefahr erforderlich/nicht erforderlich, dass die GUI für erfrischend, weil wir es verpasst, klare Beschreibungen aus der API wie/Wann/warum (re -) Validierung erforderlich repaint()

    InformationsquelleAutor trashgod

  3. 3

    Beispiel über validate() revalidate() plus repaint () Aussehen, die als erforderlich für die korrekte Ausgabe auf die GUI gelegt, indem einige der LayourManagers

    EDIT: wie trashgod bemerkte, fügte ich Planen einen job ein, für die EDT

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

    public class ValidateRevalidateRepaint {

        private JPanel panel;
        private GridBagConstraints gbc;
        private boolean validate, revalidate, repaint;

        public ValidateRevalidateRepaint() {
            validate = revalidate = repaint = false;
            panel = new JPanel(new GridBagLayout());
            gbc = new GridBagConstraints();
            gbc.insets = new Insets(0, 20, 0, 20);
            panel.add(getFiller(), gbc);
            JFrame f = new JFrame();
            f.setJMenuBar(getMenuBar());
            f.setDefaultCloseOperation(JFrame.EXIT_ON_CLOSE);
            f.getContentPane().add(panel);
            f.getContentPane().add(getRadioPanel(), "East");
            f.getContentPane().add(getCheckBoxPanel(), "South");
            f.setSize(400, 200);
            f.setLocation(200, 200);
            f.setVisible(true);
        }

        private JMenuBar getMenuBar() {
            JMenu menu = new JMenu("change");
            ActionListener l = new ActionListener() {

                @Override
                public void actionPerformed(ActionEvent e) {
                    JMenuItem item = (JMenuItem) e.getSource();
                    int n = Integer.parseInt(item.getActionCommand());
                    makeChange(n);
                }
            };
            for (int j = 1; j < 5; j++) {
                String s = String.valueOf(j) + " component";
                if (j > 1) {
                    s += "s";
                }
                JMenuItem item = new JMenuItem(s);
                item.setActionCommand(String.valueOf(j));
                item.addActionListener(l);
                menu.add(item);
            }
            JMenuBar menuBar = new JMenuBar();
            menuBar.add(menu);
            return menuBar;
        }

        private JPanel getRadioPanel() {
            JPanel panel1 = new JPanel(new GridBagLayout());
            GridBagConstraints gbc1 = new GridBagConstraints();
            gbc1.insets = new Insets(2, 2, 2, 2);
            gbc1.weighty = 1.0;
            gbc1.gridwidth = GridBagConstraints.REMAINDER;
            ButtonGroup group = new ButtonGroup();
            ActionListener l = new ActionListener() {

                @Override
                public void actionPerformed(ActionEvent e) {
                    JRadioButton radio = (JRadioButton) e.getSource();
                    int n = Integer.parseInt(radio.getActionCommand());
                    makeChange(n);
                }
            };
            for (int j = 0; j < 4; j++) {
                String s = String.valueOf(j + 1);
                JRadioButton radio = new JRadioButton(s);
                radio.setActionCommand(s);
                radio.addActionListener(l);
                group.add(radio);
                panel1.add(radio, gbc1);
            }
            return panel1;
        }

        private JPanel getCheckBoxPanel() {
            final String[] operations = {"validate", "revalidate", "repaint"};
            ActionListener l = new ActionListener() {

                @Override
                public void actionPerformed(ActionEvent e) {
                    JCheckBox checkBox = (JCheckBox) e.getSource();
                    String ac = checkBox.getActionCommand();
                    boolean state = checkBox.isSelected();
                    if (ac.equals("validate")) {
                        validate = state;
                    }
                    if (ac.equals("revalidate")) {
                        revalidate = state;
                    }
                    if (ac.equals("repaint")) {
                        repaint = state;
                    }
                }
            };
            JPanel panel2 = new JPanel();
            for (int j = 0; j < operations.length; j++) {
                JCheckBox check = new JCheckBox(operations[j]);
                check.setActionCommand(operations[j]);
                check.addActionListener(l);
                panel2.add(check);
            }
            return panel2;
        }

        private void makeChange(int number) {
            panel.removeAll();
            for (int j = 0; j < number; j++) {
                panel.add(getFiller(), gbc);
            }
            if (validate) {
                panel.validate();
            }
            if (revalidate) {
                panel.revalidate();
            }
            if (repaint) {
                panel.repaint();
            }
        }

        private JPanel getFiller() {
            JPanel panel3 = new JPanel();
            panel3.setBackground(Color.red);
            panel3.setPreferredSize(new Dimension(40, 40));
            return panel3;
        }

        public static void main(String[] args) {//added Schedule a job for the EDT
        javax.swing.SwingUtilities.invokeLater(new Runnable() {

            @Override
            public void run() {
                ValidateRevalidateRepaint rVR = new ValidateRevalidateRepaint();
            }
        });
    }
}
    Sehr interessant, aber nicht zu vernachlässigen, die event-dispatch-thread. 🙂
    weil ich immer noch sicher, dass, Aktion ist Threadsicher von Hörern 🙂
    Leider ist die thread-sichere API verspricht schwindet, z.B. append() gegen append().
    erhhhhttt ändern, dass Sie im Recht sind +1
    und für Java7 ' s Swing Sie sind geplant zum entfernen von single-treten Regeln für EDT, das sieht aus wie eine gute Idee, auf den ersten Blick, aber mit aktuellen Java-Eigentümer(s) ....

    InformationsquelleAutor mKorbel

Schreibe einen Kommentar