Swing jComboBox Ausgabe -1/null

[haedock]

Hallo,

ich habe in meinem Programm (unten ein Beispiel) eine ComboBox mit mehreren Auswahlmöglichkeiten eingebaut. Soweit funktioniert dies auch. Wenn ich jedoch versuche die ComboBox auszulesen bekomme ich als Index immer nur "-1" und als Item "null" zurückgegeben. Hat jemand schonmal ein ähnliches Problem gehabt oder kann mir so weiter helfen?

Danke im voraus!

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

public class combobox extends JFrame {
  // Anfang Attribute
  private JComboBox jComboBox1 = new JComboBox();
    private DefaultComboBoxModel jComboBox1Model = new DefaultComboBoxModel();
  // Ende Attribute
  
  public combobox(String title) { 
    // Frame-Initialisierung
    super(title);
    setDefaultCloseOperation(WindowConstants.DISPOSE_ON_CLOSE);
    int frameWidth = 300; 
    int frameHeight = 300;
    setSize(frameWidth, frameHeight);
    Dimension d = Toolkit.getDefaultToolkit().getScreenSize();
    int x = (d.width - getSize().width) / 2;
    int y = (d.height - getSize().height) / 2;
    setLocation(x, y);
    setResizable(false);
    Container cp = getContentPane();
    cp.setLayout(null);
    // Anfang Komponenten
    String[] liste = {"1", "2", "3"};
    jComboBox1.setModel(jComboBox1Model);
    JComboBox jComboBox1 = new JComboBox(liste);
    jComboBox1.setBounds(24, 40, 150, 20);
    cp.add(jComboBox1);
    jComboBox1.addItemListener(new ItemHandler());
    add(jComboBox1);
    // Ende Komponenten   
    setVisible(true);
  } // end of public combobox  
  // Anfang Methoden
  private class ItemHandler implements ItemListener{
    @Override
    public void itemStateChanged(ItemEvent e) {
      System.out.println("Changed Index to: " + jComboBox1.getSelectedIndex());
    }    
  }
  // Ende Methoden 
  public static void main(String[] args) {
    new combobox("combobox");
  } // end of main  
} // end of class combobox

 

[Harry Kane]

Du legst im Konstruktor von combobox eine neue (lokale) Instanz von JComboBox an und packst diese in den JFrame. In deiner itemStateChanged-Methode greifst du aber auf die Instanzvariable zu, die nur ein leeres DefaultComboBoxModel hat.
Die Lösung will ich noch nicht verraten. Mal sehen, ob du da selber drauf kommst.
Agsehen davon:
1. Java-Namenskonventionen beachten. Klassennamen werden nicht klein geschrieben. Ausserdem sollten Klassen nicht so oder so ähnlich heißen wie Java-Standardklassen.
2. Lerne Layoutmanager. Gewöhne dir die verwendung des Null-Layouts gar nicht erst an.

 

[Foxei]

Also mein Junge ich werde dir auch nicht sagen wo dein Fehler liegt mein Freund hier hat das schon gut beschrieben Du kommst selber drauf
Aber Bitte Bitte Formatiere deine Code richtig und schreibe nicht so viel kram den man gar nicht braucht.

Also erstens der Name einer Klasse wird immer am Anfang groß geschrieben.

Dann bist du die im klaren darüber das sich dein Programm nicht beendet? Mit DISPOSE_ON_CLOSE schließt sich nur das Fenster nicht aber die Anwendung. Desweiteren brauchst du keine WindowConstance da deine Klasse von einer JFrame Erbt reicht einfach EXIT_ON_CLOSE.

Um eine Frame in die Mitte des Bildschirms zu setzen muss keine aufwendige Berechnung gemacht werden es reicht eine einzige Zeile Code.

setLocationRelativeTo(null);

Dann bringt es dir nichts deiner ContentPane in einer Variabele zu speichern wenn du sie mal nutzt und mal nicht generell braucht man das nicht da Methoden wie add oder setLayout von der Frame Automatisch weitergegeben werden.

Und als letztes noch kurz angemerkt deinen Variabelen benennung verwirrt total ich habe sie ganz leicht abgeändert

Und abschließend als etwas verspätet Weinachtsgeschenck für dich deinen Code (mit deinem Fehler) in besserer Formatierung.

import java.awt.event.*;
import javax.swing.*;
 
public class Combobox extends JFrame {

  private JComboBox comboBox1 = new JComboBox();
  private DefaultComboBoxModel comboBoxModel1 = new DefaultComboBoxModel();
    
  public Combobox(String title) { 
    super(title);
    setDefaultCloseOperation(EXIT_ON_CLOSE);
    setSize(300,300);
    setLocationRelativeTo(null);
    setResizable(false);
    setLayout(null);
 
    String[] liste = {"1", "2", "3"};
    comboBox1.setModel(comboBoxModel1);
    JComboBox comboBox1 = new JComboBox(liste);
    comboBox1.setBounds(24, 40, 150, 20);
    add(comboBox1);
    comboBox1.addItemListener(new ItemHandler());
    add(comboBox1);
   
    setVisible(true);
  } 
  private class ItemHandler implements ItemListener{
    @Override
    public void itemStateChanged(ItemEvent e) {
      System.out.println("Changed Index to: " + comboBox1.getSelectedIndex());
    }    
  }
  
  public static void main(String[] args) {
    new Combobox("Combobox");
  }
}

Noch als Tipp dein Fehler liegt zwischen Zeile 17 und 25

Gruß Simon

 

[haedock]

danke euch beiden. Die Lösung habe ich noch nicht gefunden und komme leider kein bisschen weiter.

 

[Foxei]

Okay ich erweiche mich
Du hast deine Combobox zweimal erzeugt und auf die eine den Listener gepackt aber den Ausgewählten Index der anderen ausgegeben die noch dazu eine Leere Liste beinhaltete.

Lösung:
Wir erzeugen nur eine Combobox mit Liste und Listener.

import java.awt.event.*;
import javax.swing.*;
 
public class Combobox extends JFrame {

  private JComboBox<String> comboBox1;
  
  public Combobox(String title) { 
    super(title);
    setDefaultCloseOperation(EXIT_ON_CLOSE);
    setSize(300,300);
    setLocationRelativeTo(null);
    setResizable(false);
    setLayout(null);
 
    String[] liste = {"1", "2", "3"};
    
    comboBox1 = new JComboBox<String>(liste);
    comboBox1.setBounds(24, 40, 150, 20);
    comboBox1.addItemListener(new ItemHandler());
    add(comboBox1);
    
    setVisible(true);
  } 
  private class ItemHandler implements ItemListener{
    @Override
    public void itemStateChanged(ItemEvent e) {
      System.out.println("Changed Index to: " + comboBox1.getSelectedIndex());
    }    
  }
  
  public static void main(String[] args) {
    new Combobox("Combobox");
  }
}