Swing JComboBox-Problem

[Landei]

Wenn ich in einer JComboBox ein Item mit dem Wert null habe, bekomme ich dafür keine SELECTED- und DESELECTED-Events (also das vorherige Item wird deselected, und erst wenn ich wieder ein nicht-Null-Item anklicke, bekomme ich wieder ein selected). Irgendeine Idee wie ich das ändern kann?

KSKB:

import java.awt.event.ItemEvent;
import java.awt.event.ItemListener;
import javax.swing.JComboBox;
import javax.swing.JOptionPane;

public class ComboBoxTest {
   public static void main(String... args) {
       JComboBox cb = new JComboBox(new String[]{null, "eins","zwei","drei"});
       cb.addItemListener(new ItemListener(){
            public void itemStateChanged(ItemEvent e) {
                System.out.println(e);
            }
       });
       JOptionPane.showMessageDialog(null, cb);
   }
}

 

[MQue]

Das

JComboBox cb = new JComboBox(new String[]{"", "eins","zwei","drei"});

geht nicht?
Man müsste halt abfragen auf "null" und dann "" setzen, ist zwar umständlich aber wieso nicht.

 

[mariusbopp]

moin

also ich fange auch grade mit java an und habe mich die tage mal mit dem thema beschäftigt... :rtfm:
also ich habe das ganze über einen switch gemacht... vill rede ich auch grade totalen müll???:L
is es ein eine combo box in die man auch rein schreiben kann? oder nur dinge auswählt??

fals englisch kein problem is guck doch mal auf der seite :Combo Box
gruß

 

[Landei]

Ich würde es gerne als null lassen. Im eigentlichen Code habe ich eine andere Klasse als String, und ich habe ein "generifizierte" Subklasse von JComboBox, die den richtigen Itemtyp zurückgibt.

 

[Marco13]

setSelectedItem aus DefaultComboBoxModel

    public void setSelectedItem(Object anObject) {
        if ((selectedObject != null && !selectedObject.equals( anObject )) ||
	    selectedObject == null && anObject != null) {
	    selectedObject = anObject;
	    fireContentsChanged(this, -1, -1);
        }
    }

Vielleicht kann man das passend überschreiben? Ein flag oder so wüßte ich nicht. Vermutlich wäre der leere String die einfachere Lösung - ob sie auch eleganter ist, hängt vom Anwendungsfall ab...

 

[SlaterB]

> falls englisch kein problem is



also Schuld ist die Methode in JComboBox

    /**
     * This protected method is implementation specific. Do not access directly
     * or override. 
     */
    protected void selectedItemChanged() {
	if (selectedItemReminder != null ) {
	    fireItemStateChanged(new ItemEvent(this,ItemEvent.ITEM_STATE_CHANGED,
					       selectedItemReminder,
					       ItemEvent.DESELECTED));
	}
	
	// set the new selected item.
	selectedItemReminder = dataModel.getSelectedItem();

	if (selectedItemReminder != null ) {
	    fireItemStateChanged(new ItemEvent(this,ItemEvent.ITEM_STATE_CHANGED,
					       selectedItemReminder,
					       ItemEvent.SELECTED));
	}
    }

man kann sie probeweise doch überschreiben und die if-Einschränkungen herausnehmen, bei mir klappte es,

ansonsten den Focus aufs Model richten, ListDataListener.contentsChanged() wird irgendwie bei jeder Selektion genau einmal aufgerufen, aber enthält weder das letzte noch das neue Item,
wenn man die Combobox oder das Model abfragt und das alte Element nicht braucht oder erinnert, dann reicht das vielleicht,

ansonsten kann man noch setSelectedItem(Object anObject) (im Model, nicht in der Box) überschreiben bzw. ein komplett eigenes Model implementieren und dann eigene gute Listener agieren lassen

ich habe

public abstract class SelectedItemListener<T>
{
    public abstract void selectedItemChanged(T oldItem, T newItem);
}

(edit: ok, könnte aktuell ein Interface sein)

 

[vladimir]

Hi, ich habe es gerade ausprobiert, anscheinend geht es wirkilich nicht mit null;
ich würde die ComboBox alternativ mit "" befühlen. Am sonsten würde mich die Lösung auch interesieren.

import java.awt.event.ItemEvent;
import java.awt.event.ItemListener;
import javax.swing.JComboBox;
import javax.swing.JOptionPane;

public class Main {

    public static void main(String... args) {
        JComboBox cb = new JComboBox(new String[]{"", "eins", "zwei", "drei"});
        cb.addItemListener(new ItemListener() {

            public void itemStateChanged(ItemEvent e) {
                if (e.SELECTED == e.getStateChange()) {
                    System.out.println("SEL " + e);
                }

                if (e.DESELECTED == e.getStateChange()) {
                    System.out.println("DES " + e);
                }
            }
        });
        JOptionPane.showMessageDialog(null, cb);
    }
}

Gruß Vladimir

 

[Landei]

Slaters Lösung funktioniert. Manchmal stehe ich auch ganz schön auf dem Schlauch...

Hier ist übrigens meine "typsichere" ComboBox (allerdings noch nicht ausgiebig getestet):

import java.awt.event.ItemEvent;
import java.util.Collection;
import java.util.Vector;

import javax.swing.JComboBox;

public class TypedComboBox<A> extends JComboBox {

    public TypedComboBox(Collection<? extends A> data) {
        super(new Vector<A>(data));
    }

    public TypedComboBox(A... data) {
        super(data);
    }

    @Override
    public A getItemAt(int index) {
        return (A) super.getItemAt(index);
    }

    @Override
    public A getSelectedItem() {
        return (A) super.getSelectedItem();
    }

    @Override
    public A[] getSelectedObjects() {
        return (A[]) super.getSelectedObjects();
    }

    @Deprecated
    @Override
    public void addItem(Object o) {
        throw new UnsupportedOperationException();
    }

    public void add(A a) {
        super.addItem(a);
    }

    @Deprecated
    @Override
    public void insertItemAt(Object anObject, int index) {
        throw new UnsupportedOperationException();
    }

    public void insertAt(A a, int index) {
        super.insertItemAt(a, index);
    }

    public void remove(A a) {
        removeItem(a);
    }

    @Override
    protected void selectedItemChanged() {
        fireItemStateChanged(new ItemEvent(this, ItemEvent.ITEM_STATE_CHANGED,
                selectedItemReminder,
                ItemEvent.DESELECTED));
        selectedItemReminder = dataModel.getSelectedItem();

        fireItemStateChanged(new ItemEvent(this, ItemEvent.ITEM_STATE_CHANGED,
                selectedItemReminder,
                ItemEvent.SELECTED));
    }

    @Override
    public int getSelectedIndex() {
        Object sObject = dataModel.getSelectedItem();
        for (int i = 0, c = dataModel.getSize(); i<c; i++ ) {
            Object obj = dataModel.getElementAt(i);
            if ((obj == null && sObject == null) || 
                (obj != null && obj.equals(sObject)))
                return i;
        }
        return -1;
    }
}

Es fehlt allerdings noch die Möglichkeit, ein Model zu übergeben, allerdings müsste das ja dann auch irgendwie typsicher sein.

[edit] getSelectedIndex muss man auch überschreiben...

 

[SlaterB]

ich empfehle doch dringend, nur mit dem Model zu arbeiten, die ComboBox dazu kann man erzeugen und in die GUI einfügen, wird danach nicht mehr benötigt,
add/remove/get/reset/Selektion (setzen + auslesen), alles geht über das eine Model,
besonders günstig wenn mehrere Boxen oder gar unterschiedliche GUI-Elemente dasselbe Model anzeigen

bei JTable/ JList ist es komplizierter, da muss man die Tabelle nach den (mehreren) selektierten Einträgen abfragen bzw. das zusätzliche SelectionModel