JComboBox macht nicht immer was sie soll

[Marfir]

Hallo,

abhängig von dem was der Nutzer in der JComboBox auswählt, soll das Formular inhaltlich zusammengestellt werden.

//Label erzeugen
JLabel lname = new JLabel("Name:");

//Label beschriften
if (objekt.getSelectedItem().toString().equals("a"))
{
            lname.setText("Vorname:");
            lname.setBounds(10, 70, 200, 20);
            this.add(lname);
            ...
}
else if (objekt.getSelectedItem().toString().equals("b"))
{
            lname.setText("Nachname:");
            ...
}

Beim starten des JFrames wird Vorname angezeigt. Wenn ich aber b auswähle ändert sich der Name nicht!
Wieso?

Ich hab mal testweise ein Button aufs Frame gesetzt dass den Vornamen auf Nachname ändern soll. Das funktioniert auch, aber nur solange wie ich in der JComboBox nichts anderes ausgewählt habe.
Da er aber 100% in die if-Anweisung reingeht (habe ich mit System.out.println() ermitteln können), muss die JComboBox irgendwas blockieren...

Eine Exception tritt nicht auf. Sieht auch sonst alles rund aus...

Ich hoffe jmd. von euch hat eine Idee woran das liegen könnte.

 

[Illuvatar]

Probier mal einen Aufruf von validate() am Ende der if Anweisung.

 

[Marfir]

Ich habs jetzt.
Es lag an

this.add(lname);

Man sollte ein Steuerelement besser nur einmal hinzufügen...
:lol:

 

[Marfir]

So jetzt gibts ne Exception in thread "AWT-EventQueue-0" java.lang.NullPointerException die mir zu schaffen macht...

Ich verstehe echt nich was java da macht. Ich hab doch nur code kopiert und schon krachts. Naja und anhand der Fehlermeldungen kann man ja nichts ersehen woran es liegen könnte...

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


public class eingabe extends javax.swing.JFrame
{
    JLabel lname;
    JTextField name;
    JLabel lentfernung;
    JTextField entfernung;
    
    /** Creates new form eingabe */
    public eingabe()
    {
        initComponents();
        
        try
        {
            lname = new JLabel("test");
            lname.setBounds(10, 70, 200, 20);
            lname.setVisible(true);
            this.add(lname);
            name = new JTextField("test");
            name.setBounds(250, 70, 120, 20);
            name.setVisible(true);
            this.add(name);
            JLabel lentfernung = new JLabel("test");
            lentfernung.setBounds(10, 90, 200, 20);
            lentfernung.setVisible(true);
            this.add(lentfernung);
            JTextField entfernung = new JTextField("test");
            entfernung.setBounds(250, 90, 120, 20);
            entfernung.setVisible(true);
            this.add(entfernung);
            
            //objekt-kategorien
            objekt.addItem("Planeten");
            objekt.addItem("Monde");
            objekt.addItem("Asteroiden");
            objekt.addItem("Kometen");
            objekt.addItem("Meteoriten");
            objekt.addItem("Braune Zwerge");
            objekt.addItem("Sterne");
            objekt.addItem("Galaxien / Nebel");
            objekt.addItem("Galaxienhaufen");
            objekt.addItem("Sternhaufen");
            objekt.addItem("sonstiges");
            objekt.addItem("Lexikon");
            
        }
        catch (Exception e)
        {
            System.out.println(e.toString());
        }
        
    }
    
    /** This method is called from within the constructor to
     * initialize the form.
     * WARNING: Do NOT modify this code. The content of this method is
     * always regenerated by the Form Editor.
     */
    // <editor-fold defaultstate="collapsed" desc=" Generated Code ">
    private void initComponents() {
        jLabel1 = new javax.swing.JLabel();
        objekt = new javax.swing.JComboBox();
        jLabel2 = new javax.swing.JLabel();
        rnotiz = new javax.swing.JScrollPane();
        notiz = new javax.swing.JTextArea();

        getContentPane().setLayout(null);

        setDefaultCloseOperation(javax.swing.WindowConstants.EXIT_ON_CLOSE);
        setTitle("neuer Eintrag");
        setResizable(false);
        jLabel1.setText("Objektklasse:");
        getContentPane().add(jLabel1);
        jLabel1.setBounds(10, 40, 100, 14);

        objekt.addActionListener(new java.awt.event.ActionListener() {
            public void actionPerformed(java.awt.event.ActionEvent evt) {
                objektActionPerformed(evt);
            }
        });

        getContentPane().add(objekt);
        objekt.setBounds(250, 40, 170, 22);

        jLabel2.setText("Bitte w\u00e4hlen Sie die Objektklasse aus, die Sie hinzuf\u00fcgen m\u00f6chten.");
        getContentPane().add(jLabel2);
        jLabel2.setBounds(10, 10, 390, 14);

        rnotiz.setViewportView(notiz);

        getContentPane().add(rnotiz);
        rnotiz.setBounds(180, 140, 270, 120);

        java.awt.Dimension screenSize = java.awt.Toolkit.getDefaultToolkit().getScreenSize();
        setBounds((screenSize.width-652)/2, (screenSize.height-738)/2, 652, 738);
    }
    // </editor-fold>

    private void objektActionPerformed(java.awt.event.ActionEvent evt) {                                       
        //gui generieren
        
        //objekte unsichtbar machen
        rnotiz.setVisible(false);
        notiz.setVisible(false);
        
        //elemente beschriften
        if ("Planeten".equals(objekt.getSelectedItem().toString()))
        {
            
            lname.setText("Name:");
            lname.setBounds(10, 70, 200, 20);
            name.setText("Erde");
            name.setBounds(250, 70, 120, 20);
            /*
            lentfernung.setText("Entfernung von seinem Stern in AE:");
            lentfernung.setBounds(10, 90, 200, 20);
            entfernung.setText("1,0");
            entfernung.setBounds(250, 90, 120, 20);
            */
            /*
            
        }
        else if ("Monde".equals(objekt.getSelectedItem().toString()))
        {
            lname.setText("Name:");
            lname.setBounds(10, 70, 120, 20);
            name.setText("Io");
            name.setBounds(250, 70, 120, 20);
            lentfernung.setText("Entfernung von seinem Planeten in km:");
            lentfernung.setBounds(10, 90, 200, 20);
            entfernung.setText("1,0");
            entfernung.setBounds(250, 90, 120, 20);
            
        }
        else if ("Asteroiden".equals(objekt.getSelectedItem().toString()))
        {
            lname.setText("Name:");
            lname.setBounds(10, 70, 120, 20);
            name.setText("2003 UB313");
            name.setBounds(250, 70, 120, 20);
            
        }
        else if ("Galaxien / Nebel".equals(objekt.getSelectedItem().toString()))
        {
            lname.setText("Name / Katalog & ID:");
            lname.setBounds(10, 70, 120, 20);
            name.setText("NGC 1234");
            name.setBounds(250, 70, 120, 20);
            
        }
        
    }                                      
    
    /**
     * @param args the command line arguments
     */
    public static void main(String args[])
    {
        java.awt.EventQueue.invokeLater(new Runnable()
        {
            public void run()
            {
                new eingabe().setVisible(true);
            }
        });
    }
    
    // Variables declaration - do not modify
    private javax.swing.JLabel jLabel1;
    private javax.swing.JLabel jLabel2;
    private javax.swing.JTextArea notiz;
    private javax.swing.JComboBox objekt;
    private javax.swing.JScrollPane rnotiz;
    // End of variables declaration
    
}

Wenn ich überall lentfernung und entfernung auskommentiere funzt es. Wenn nichts gibts diese 0-Pointer-Exc.
Aber wieso?
Ach und das verrückteste ist, die JComboBox zeigt nur Planeten an...

Danke für eure Hilfe!

 

[kaie]

Du darfst das Ergebnis-Label und -Textfield nicht als lokale Variable im Konstruktor deklarieren (Zeile 29 bzw. 33). Verwende Deine beiden schon vorhandenen Attribute, dann klappt's auch mit der ComboBox. Also statt

JLabel lentfernung = new JLabel("test");

einfach

lentfernung = new JLabel("test");

schreiben. Ebenso

entfernung = new JTextField("test");

statt

JTextField entfernung = new JTextField("test");

Schönen Sonntagabend noch!
kaie

 

[Marfir]

Huch stimmt. Das hab ich vor lauter Code nicht mehr gesehen :lol:

Danke dir auch!