JTextField Array im JPanel wird nicht komplett angezeigt

[tritronic]

Hallo zusammen,

nachdem ich eine Pause gemacht habe, bin ich - Anfänger - wieder mal beim Üben. Jetzt habe ich folgendes Problem:

Ich erstelle zwei Array's - ein JLabl und ein JTextfield. Diese sollen abhängig von einer Auswahl (JComboBox) in einem JPanel mit einer for-Schleife generiert werden. Allerdings füllt er das JPanel nur bis zu einer entsprechenden Grenze. Wenn ich also in der JComboBox den letzten Wert (NSC 10) auswähle, läuft die Schleife 10 mal durch (ich sehe das in der Consolen-Ausgabe), es werden allerdings nur je 4 JLable bzw. JTextfield angezeigt.

Anbei mal mein Code:

package ExaltedTool;

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

public class neutest extends JFrame {
    private JPanel contentPane;
    private JMenuBar menueLeiste;
    private JComboBox comboBox;
    private int anzahlNSC;
    private JPanel panelEast;
    private JPanel panelEast2;
   
    public static void main(String[] args)     {
        EventQueue.invokeLater(new Runnable() {
            public void run() {
                try {
                    neutest frame = new neutest();
                    frame.setVisible(true);
                } catch (Exception e) {
                    e.printStackTrace();
                }}});}

    public neutest() {
        setDefaultCloseOperation(JFrame.EXIT_ON_CLOSE);
        setBounds(100, 100, 1000, 600);
        contentPane = new JPanel();
        contentPane.setBorder(new EmptyBorder(5, 5, 5, 5));
        setContentPane(contentPane);
        contentPane.setLayout(new BorderLayout(0, 0));
       
        // Panel East
        panelEast = new JPanel();
        panelEast.setPreferredSize(new Dimension(300, 500));
        panelEast.setBorder(new EtchedBorder(EtchedBorder.LOWERED, Color.BLACK, Color.BLACK));
        panelEast.setLayout(null);
        contentPane.add(panelEast, BorderLayout.EAST);

        JLabel lblUeberschrift2 = new JLabel("NSC");
        lblUeberschrift2.setFont(new Font("Tahoma", Font.PLAIN, 28));
        lblUeberschrift2.setBounds(10, 10, 80, 45);
        panelEast.add(lblUeberschrift2);
       
        panelEast2 = new JPanel();
        panelEast2.setBorder(new EtchedBorder(EtchedBorder.LOWERED, Color.BLACK, Color.BLACK));
        panelEast2.setBounds(10, 59, 280, 410);
        panelEast.add(panelEast2);

        // Menüleiste erzeugen
        menueLeiste = new JMenuBar();
       
        // Checkboxen NSC erzeugen
        String comboBoxListe[] = {"Anzahl NSC", "NSC 1", "NSC 2", "NSC 3", "NSC 4", "NSC 5", "NSC 6",
                "NSC 7", "NSC 8", "NSC 9", "NSC 10"};
        comboBox = new JComboBox(comboBoxListe);
        comboBox.setMaximumRowCount(11);
        comboBox.addItemListener(new java.awt.event.ItemListener() {
             public void itemStateChanged(java.awt.event.ItemEvent e) {
                 panelEast2.removeAll();
                 String selectedItem;
                 anzahlNSC = 0;
                 if(e.getStateChange() == ItemEvent.SELECTED) {
                     selectedItem = (String)comboBox.getSelectedItem() ;
                     try {
                         anzahlNSC = Integer.valueOf(selectedItem.replaceAll("\\D",""));
                         System.out.println(anzahlNSC);

                         int y1 = 20;
                         int plus = 30;

                         JTextField [] tfnsc = new JTextField[10];
                         JLabel [] jlblnsc = new JLabel[10];
                         if(anzahlNSC >= 1) {
                             int i;
                            
                             for (i = 0; i < anzahlNSC; i++) {
                                 jlblnsc[i] = new JLabel("NSC " + (i+1));
                                 jlblnsc[i].setBounds(10, y1, 80, 20);
                                 panelEast2.add(jlblnsc[i]);
                                 System.out.println(jlblnsc[i]);
                                 tfnsc[i] = new JTextField();
                                 tfnsc[i].setBounds(80, y1, 30, 20);
                                 panelEast2.add(tfnsc[i]);
                                 System.out.println("--" +tfnsc[i]);
                                 System.out.println(i);
                                 y1 = y1 + plus;
                             }}
                     } catch (Exception f) {
                         System.out.println(f);
                     }}}});   
        menueLeiste.add(comboBox);
        contentPane.add(menueLeiste, BorderLayout.NORTH);}}

Vielen Danke schon mal für ein paar hilfreiche Antworten

Jens

 

[strußi]

nach dem ende der For-schleife ein panelEast2.updateUI(); einfügen, dass ist alles da, leider nicht mehr ganz so schön, aber die lösung für das problem findest du sicher raus

 

[tritronic]

Danke für die schnelle Antwort. Aber das war es noch nicht ganz.

Anzeigen sollte er mir etwas wie:

JLabel_1 JTextfield_1
JLabel_2 JTextfield_2
JLabel_3 JTextfield_3
JLabel_4 JTextfield_4
JLabel_5 JTextfield_5
JLabel_6 JTextfield_6
JLabel_7 JTextfield_7
JLabel_8 JTextfield_8
JLabel_9 JTextfield_9
JLabel_10 JTextfield_10

Mit Panel panelEast2.updateUI(); wird mir das aber wie folgt angezeigt (Die JTextFields sind abgeschnitten):

JLabel_1 JTF_1 JLabel_2 JTF_2 JLabel_3 JTF_3 JLabel_4 JTF_4 JLabel_5 JTF_5
JLabel_6 JTF_6 JLabel_7 JTF_7 JLabel_8 JTF_8 JLabel_9 JTF_9 JLabel_10 JTF_10

 

[Harry Kane]

Wenn ich richtig sehe, wird bei panelEast2 kein Layout gesetzt, so dass das standardmässige FlowLayout verwendet wird. Und das ordnet die Komponenten nunmal spaltenweise an. Das FlowLayout "überschreibt" die Bounds, die du manuell setzt.

 

[tritronic]

ich habe es auch schon mit panelEast2.setLayout(null); für ein Absolut Layout probiert. Das verändert die Anzeige nicht.

- Mit int plus = 30; werden mir 4 Zeilen angezeigt
- Mit int plus = 10; werden mir alle Zeilen angezeigt, dann stimmen aber die Abstände nicht mehr.

ich habe den Eindruck dass panelEast2 immer nur zu etwa einem drittel gefüllt wird

 

[tritronic]

so... gelöst... ihr habt beide dazu beigetragen:

strußi => panelEast2.updateUI(); => habe ich gemacht, hatte ja nicht funktioniert
Harry Kane => Layout => panelEast2.setLayout(null); => hat alleine auch nichts verändert

beide zusammen waren die Lösung.

Vielen Dank

 

[strußi]

@tritronic es hat funktioniert, aber ohne layout sieht es halt so aus wie es aussieht. für solche strukturen nehme ich gerne das GridBagLayout als Layout und habe eine Methode mit der ich die Components adde
hab diese irgendwo im netz gefunden und für meine Zwecke modifiziert. geht ganz simple, methode aufrufen und component einfügen, fertig

Spoiler: code

/**
     * Interne Methode zum hinzufügen der einzeln Componenten auf das Panel
     * @param gbc GridBagConstraints
     * @param c Component to add
     * @param gridx Colum
     * @param gridy Row
     * @param width count of Colums
     * @param height count of Rows
     * @param weightx percent of the colum for the Component
     * @param weighty percent of the row for the Component
     * @param fill fill orientation
     */
    private void addComponent( GridBagConstraints gbc, Component c, int gridx, int gridy, int width, int height,
                               double weightx, double weighty, int fill){
        gbc.gridx      =gridx;    gbc.gridy     =gridy;
        gbc.weightx    =weightx;  gbc.weighty   =weighty;
        gbc.gridheight =height;   gbc.gridwidth =width;
        gbc.fill =fill;
        add( c, gbc);
    }

 

[tritronic]

Danke. Ich schau mir das mal an