Swing JComboBox mit Listener aktualisieren

[Sherry]

Hallo zusammen,

mit folgenden Programm erzeuge ich eine JComboBox, die Städtenamen aus einer XML-Datei erhält. Die Städteliste wir über die Klasse XMLCreator und deren Methode CityList() erzeugt.
Ich habe es soeit geschafft, dass in dem Dropdown-Menü alle Städte aus der XML-Datei angezeigt werden.

Mein Ziel geht noch weiter. Und zwar soll die XML-Datei während der laufzeit dieses Programmes geändert werden, zum Beispiel neue Städte hinzu gefügt oder entfernt werden. Und das ComboboxModel sollte während der Laufzeit beim Herunterklappen des Dropdown-Menüs die Werte aktuallisieren.

Mit welchem Listener kann das realsiert werden?

Viele Grüße

Sherry

import java.awt.BorderLayout;
import java.awt.EventQueue;

import javax.swing.JFrame;
import javax.swing.JPanel;
import javax.swing.border.EmptyBorder;
import javax.swing.event.ListDataEvent;
import javax.swing.event.ListDataListener;

import java.awt.Window.Type;
import java.util.Vector;

import javax.swing.BoxLayout;
import javax.swing.DefaultComboBoxModel;
import javax.swing.GroupLayout;
import javax.swing.GroupLayout.Alignment;
import javax.swing.JComboBox;

import XMLCreator;

public class ComboBoxTest extends JFrame implements ListDataListener {

	private JPanel contentPane;
	private JComboBox dropDown;
	private DefaultComboBoxModel comboBoxModel1;
	

	/**
	 * Launch the application.
	 */
	public static void main(String[] args) {
		EventQueue.invokeLater(new Runnable() {
			public void run() {
				try {
					ComboBoxTest frame = new ComboBoxTest();
					frame.setVisible(true);
				} catch (Exception e) {
					e.printStackTrace();
				}
			}
		});
	}

	/**
	 * Create the frame.
	 */
	public ComboBoxTest() {
		setType(Type.UTILITY);
		setTitle("Test für JComboBox");
		setDefaultCloseOperation(JFrame.EXIT_ON_CLOSE);
		setBounds(100, 100, 450, 300);
		contentPane = new JPanel();
		contentPane.setBorder(new EmptyBorder(5, 5, 5, 5));
		setContentPane(contentPane);
		
		dropDown = new JComboBox();
		/*
		 * Städte in Liste einlesen
		 */
		Vector<String> cities = new Vector<String>();
		XMLCreator cityList = new XMLCreator();
		for (int i = 0; i < cityList.CityList().size(); i++) {
			cities.add(cityList.CityList().get(i));
		}
		
		/*
		 * ComboBoxModel erzeugen
		 */
		comboBoxModel1 = new DefaultComboBoxModel<String>(cities);
		comboBoxModel1.addListDataListener(this);
		
		/*
		 * ComboBoxModel setzen
		 */
		dropDown.setModel(comboBoxModel1);
		
		GroupLayout gl_contentPane = new GroupLayout(contentPane);
		gl_contentPane.setHorizontalGroup(
			gl_contentPane.createParallelGroup(Alignment.LEADING)
				.addGroup(gl_contentPane.createSequentialGroup()
					.addGap(76)
					.addComponent(dropDown, GroupLayout.PREFERRED_SIZE, 223, GroupLayout.PREFERRED_SIZE)
					.addContainerGap(137, Short.MAX_VALUE))
		);
		gl_contentPane.setVerticalGroup(
			gl_contentPane.createParallelGroup(Alignment.LEADING)
				.addGroup(gl_contentPane.createSequentialGroup()
					.addGap(101)
					.addComponent(dropDown, GroupLayout.PREFERRED_SIZE, GroupLayout.DEFAULT_SIZE, GroupLayout.PREFERRED_SIZE)
					.addContainerGap(137, Short.MAX_VALUE))
		);
		contentPane.setLayout(gl_contentPane);
	}

	@Override
	public void contentsChanged(ListDataEvent arg0) {
		// TODO Auto-generated method stub
		
	}

	@Override
	public void intervalAdded(ListDataEvent arg0) {
		// TODO Auto-generated method stub
		
	}

	@Override
	public void intervalRemoved(ListDataEvent arg0) {
		// TODO Auto-generated method stub
		
	}
}

 

[turtle]

Und zwar soll die XML-Datei während der laufzeit dieses Programmes geändert werden

Die Daten, die in einer Combobox angezeigt werden, sind in einem ComboBoxModel gespeichert.

Also musst du das Model neu "laden", indem du die Daten aus der XML-Datei wieder einliest. Über jComboBox.setModel(...) kannst du dann deiner ComboBox mitteilen, das es neue Model-Daten gibt.

Unter java-7 gibt es mit WatchService die Möglichkeit, informiert zu werden, wenn sich eine Datei geändert hat.

 

[Sherry]

Hmm,

ich dachte das könnte ich mit einem Listener machen, der auf das Herunterklappen des Menüs lauscht.

Also musst du das Model neu "laden", indem du die Daten aus der XML-Datei wieder einliest. Über jComboBox.setModel(...) kannst du dann deiner ComboBox mitteilen, das es neue Model-Daten gibt.

 

[Harry Kane]

Warum willst du prophylaktisch bei jedem Öffnen der JComboBox prüfen, ob sich der Umfang der anzuzeigenden Daten geändert hat? Klingt für mich seeeehr fragil.
Andersrum wird ein Schuh draus: lege dir ein ListModel an welches AbstractListModel erweitert. Dieses nennen wir es mal XMLContentListModel überprüft in regelmässigen Abständen einen Abschnitt einer Datei auf Änderungen. Wenn Änderungen festgestellt werden, werden die Daten im ListModel durch Einlesen/Neuparsen des Abschnitts aktualisiert. Anschliessend rufst du in deinem Model fireContentsChanged auf.

 

[Sherry]

Warum willst du prophylaktisch bei jedem Öffnen der JComboBox prüfen, ob sich der Umfang der anzuzeigenden Daten geändert hat? Klingt für mich seeeehr fragil.

Was meinst Du mit fragil? Sinn und Zweck des Überprüfens ist, das Hauptfenster benötigt diese Auswahlliste. Wenn jetzt der Anwender feststellt, die gewünschte Stadt ist noch nicht eingepflegt, dann soll er es mit einem AddFrame einpflegen können. Nach Schliessen dieses Frames steckt die Stadt in der XML-Datei, aber noch nicht in der JComboBox. Das Programm müsste neugestartet werden.

Was wäre denn der Unterschied zu Deinem Vorschlag mit einem ListModel und meiner Lösung? Meine parst doch die XML-Datei ebenfalls. Und die XML-Datei müsste doch in beiden Fällen komplett geparst werden, sonst werden die Änderungen nicht erfasst.

 

[turtle]

Daher würde ich die Daten in einer Datenbank halten (Aber mich fragt ja keiner;()

Das Laden/Parsen ist aus meiner Sicht unnötig und kostet "nur" Performance.

 

[Sherry]

Okay, jetzt sind wir ganz von dem abgewichen, was ich wissen wollte.
Wie heißt der Listener bei der JComboBox, der das kann was ich erzielen will?
Ob Datenbank oder XML-Datei spielt an diesem Punkt keine Rolle, auch wäre meine Frage dieselbe.

 

[turtle]

Du hast Recht

Aber ich finde, Harry Kane und ich haben es bereits beantwortet. Wir finden, du solltest ein eigenes Model schreiben, welches die XML Datei parsed und die Einträge verwaltet.

Ob du das ganze Model tauschen musst oder über eine fireXXX-Methode dies triggern kannst, richtet sich auch danach welche Änderungen im Model gemacht werden.

Meines Wissens gibt es keinen Listener, der das, was du beabsichtigst, ermöglicht.

Du kannst BasicComboBoxUI extenden und createPopup() überschreiben. Ist aber abhängig vom Look-and-Feel und ich schätze das als nicht trivial ein. Ich habs noch nicht gemacht

 

[Sherry]

Okay, dann muss ich das eben mal intensiver erarbeiten:

Du hast Recht
Wir finden, du solltest ein eigenes Model schreiben, welches die XML Datei parsed und die Einträge verwaltet.

 

[turtle]

Vielleicht hilft folgendes Mini-Beispiel weiter?

import java.awt.FlowLayout;
import java.awt.event.ActionEvent;
import java.awt.event.ActionListener;
import java.util.Random;

import javax.swing.DefaultComboBoxModel;
import javax.swing.JButton;
import javax.swing.JComboBox;
import javax.swing.JFrame;
import javax.swing.JPanel;

public class MyComboBox extends JPanel {
    private MyDefaultComboBoxModel myModel;
    private JComboBox<String> myCombobox;

    public MyComboBox(JFrame frame) {
	myModel = new MyDefaultComboBoxModel();
	myCombobox = new JComboBox<String>();
	myCombobox.setMaximumRowCount(10);
	myModel.reload();
	myCombobox.setModel(myModel);
	JButton load = new JButton("Reload");
	load.addActionListener(new ActionListener() {

	    @Override
	    public void actionPerformed(ActionEvent e) {
		myModel.reload();
		myCombobox.setModel(myModel);
	    }
	});
	frame.add(myCombobox);
	frame.add(load);
    }

    public static void main(String[] args) {
	JFrame frame = new JFrame("Test Combobox");
	frame.setDefaultCloseOperation(JFrame.EXIT_ON_CLOSE);
	frame.setLayout(new FlowLayout());
	frame.add(new MyComboBox(frame));
	frame.pack();
	frame.setVisible(true);
    }

}

class MyDefaultComboBoxModel extends DefaultComboBoxModel<String> {
    private static Random random = new Random();

    public void reload() {
	removeAllElements();
	for (int i = 0; i < 10; i++) {
	    int nextInt = random.nextInt(20);
	    addElement("" + nextInt);
	}
    }
}

 

[Harry Kane]

Wenn du es unbedingt mit einem Listener versuchen möchtest:
Nimm einen PopupMenuListener und implementiere popupMenuWillBecomeVisible(PopupMenuEvent pme).
Mit "fragil" meine ich, daß du in dem Moment, in dem die JList eingeblendet wird, noch schnell die Liste aktualisierst oder der JComboBox gar ein neues Modell verpasst. Je nachdem was da wie zu welcher Millisekunde aktualisiert wird, wäre mit die Gefahr zu hoch, daß es dabei zu Artefakten bei der Darstellung kommt oder gar eine Exception fliegt, je nachdem ob alles im EDT abläuft oder irgendein Aktualisierungsvorgang doch nebenläufig programmiert ist. Alles in allem wäre mir das viel zu heikel. Je nachdem wie lang die Überprüfung ist, müsstest du ausserdem mit einem Zeitverzug rechnen zwischen Klick auf den entsprechenden JComboBox Button und dem Moment wo das Popup wirklich erscheint.

 

[Sherry]

Ich denke schon. Denn alternativ hatte ich jetzt auch daran gedacht einen "Auffrisch"-Button, oder wie Du es als Reload bezeichnest, einzusetzen und dort das Model neuzusetzen.

Du merkst, ich bin noch am Lernen.

Da ich aus dem Hauptfenster mittels "Hinzufügen" ein Unterfenster zum Hinzufügen starten kann und eigentlich nur nach dem Beenden im Hauptfenster die Liste aufgefrischt werden braucht, dachte ich daran, den Aufruf dazu ins Unterfester zu legen.
Also wenn ich auf Beenden im Unterfenster klicke, müsste das Model im Hauptfenster neugesetzt werden.

Kann ich so etwas fensterübergreifend realisieren? Also der ActionListener im Unterfenster des Beenden-Buttons ruft setModel() im anderen Fenster auf. Etwa so ähnlich:

JButton closeButton = new JButton("Beenden");
		closeButton.addActionListener(new ActionListener() {
			public void actionPerformed(ActionEvent arg0) {
                             targetCityComboBox.setModel(comboBoxModel);
			}
		});

 

[turtle]

Also der ActionListener im Unterfenster des Beenden-Buttons ruft setModel() im anderen Fenster auf

Klar geht das

Ich habe das in meinem Mini-Programm NICHT im Reload gemacht, da ich der Meinung bin, das dies nicht Aufgabe vom Reload ist. (Sollte nur das Model neu bauen).

Daher wird dieses ausserhalb gemacht durch myCombobox.setModel(myModel) unter der Annahme, das neue Daten ins Model gepumpt wurden.

 

[Sherry]

Prima, dann versuche ich das ersteinmal. Besten Dank für Deine Unterstützung.