JComboBox

[Endymion]

Hi, ich habe ein kleines Problem: Ich setze bei meinem Programm die Sprache über eine JComboBox. Die möglichen Sprachen sind deutsch und englisch. Wenn nun englisch eingestellt ist, und ich in der JComboBox deutsch einstelle, wird die Sprache korrekt geändert. Allerdings bleibt in der JCombobox weiterhin der Begriff "Englisch" ausgewählt (er ändert sich einfach von dem englischen "English" auf "Englisch". Wie kann ich das ausgewählte Objekt auch anzeigen lassen, wenn die JComboBox geschlossen wird? Folgender Code:

languageBox = new JComboBox<String>();
		for (Locale locale : main.getSupportetLanguages()) {
			languageBox.addItem(locale.getDisplayLanguage());
		}
		languageBox.addActionListener(new ActionListener() {
			@Override
			public void actionPerformed(ActionEvent ae) {
				JComboBox<?> source = (JComboBox<?>) ae.getSource();
				main.setLanguage(LocaleUtils.getLanguage(main,
						(String) source.getSelectedItem()));
				languageBox.setSelectedItem(source.getSelectedItem());
			}
		});
		languagePanel.add(languageBox);

 

[so0jvpwemm]

Abgesehen davon, dass ich es nicht schlimm finde und ich grundsätzlich die möglichen Sprachen auch in der jeweiligen Sprache auflisten würde. Ist hierfür entscheidend, was in der

main.setLanguage(...)

mit der ComboBox passiert.

 

[Endymion]

public void setLanguage(Locale locale) {
		Locale.setDefault(locale);
		language = locale;
		fireLanguageEvent(locale);
	}

private void fireLanguageEvent(Locale locale) {
		for (LanguageListener languageListener : languageListeners) {
			languageListener.languageChanged();
		}
	}

public class LanguageEvent {
	private Locale language;

	public LanguageEvent(Locale language) {
		this.language = language;
	}

	public Locale getLocale() {
		return language;
	}
}

 

[so0jvpwemm]

LanguageListener usw. sind ja alles keine Standard Java Klassen.
Wie ist das ganze denn mit der JComboBox gekoppelt? Wo bekommt diese also die Änderung mit bzw. wird manipulilert?

 

[Endymion]

public interface LanguageListener {
	public void languageChanged();
}

Die JComboBox ruft setLanguage in main auf, das ruft fireLanguageEvent auf, das ruft languageChanged in GUI auf, das entfernt dann alle Elemente aus der JComboBox, und ruft initializeLanguageBox auf:

public void initializeLanguageBox() {
		for (Locale locale : main.getSupportetLanguages()) {
			languageBox.addItem(locale.getDisplayLanguage());
		}
	}

 

[so0jvpwemm]

OK. Du baust die GUI komplett neu auf.
Sicher, dass die richtige ComboBox angezeigt bzw. befüllt wird. Lass Dir doch mal in der Schleife die Namen per System.out ausgeben, um zu überprüfen, ob hier die Sprachen wie gewünscht ausgelesen werden.

 

[bERt0r]

Du machst das viel zu kompliziert. Pack einfach die Locales in die Combobox und schreib dir einen Renderer:

import java.awt.BorderLayout;
import java.awt.Component;
import java.awt.EventQueue;
import java.awt.event.ItemEvent;
import java.awt.event.ItemListener;
import java.util.Locale;

import javax.swing.DefaultListCellRenderer;
import javax.swing.JComboBox;
import javax.swing.JFrame;
import javax.swing.JList;
import javax.swing.JPanel;
import javax.swing.border.EmptyBorder;


public class LocalComboBox extends JFrame
{
	
	private JPanel contentPane;
	private JPanel panel;
	private JComboBox<Locale> comboBox;
	
	/**
	 * Launch the application.
	 */
	public static void main(String[] args)
	{
		EventQueue.invokeLater(new Runnable()
			{
				public void run()
				{
					try
					{
						LocalComboBox frame = new LocalComboBox();
						frame.setVisible(true);
					} catch (Exception e)
					{
						e.printStackTrace();
					}
				}
			});
	}
	
	/**
	 * Create the frame.
	 */
	public LocalComboBox()
	{
		setDefaultCloseOperation(JFrame.EXIT_ON_CLOSE);
		setBounds(100, 100, 450, 300);
		contentPane = new JPanel();
		contentPane.setBorder(new EmptyBorder(5, 5, 5, 5));
		contentPane.setLayout(new BorderLayout(0, 0));
		setContentPane(contentPane);
		
		panel = new JPanel();
		contentPane.add(panel, BorderLayout.CENTER);
		
		comboBox = new JComboBox<Locale>(Locale.getAvailableLocales());
		comboBox.setRenderer(new LocaleListCellRenderer());
		comboBox.addItemListener(new ItemListener()
		{

			@Override
			public void itemStateChanged(ItemEvent e)
			{
				Locale l=(Locale)e.getItem();
				if(l!=null)
				{
					Locale.setDefault(l);
				}
			}
			
		});
		panel.add(comboBox);
	}
	
	class LocaleListCellRenderer extends DefaultListCellRenderer
	{
		@Override
		public Component getListCellRendererComponent(JList list, Object value, int index, boolean isSelected, boolean cellHasFocus)
		{
			Locale l=(Locale)value;
			return super.getListCellRendererComponent(list, l.getDisplayName(), index, isSelected, cellHasFocus);
		}
	}
	
}

 

[Endymion]

Darum geht es doch nicht. Es geht darum, dann da anstatt "english" und "german" zwar "englisch" und "deutsch" steht, in der Auswahl aber das Wort "englisch" und nicht "deutsch" steht. Problem habe ich aber grade gelöst: in languageChanged von GUI musste ich am ende noch die Zeile

languageBox.setSelectedItem(Locale.getDefault().getDisplayLanguage());

einfügen.

 

[bERt0r]

Oh doch um genau das ging es. Wenn du nämlich alle Items aus der ComboBox entfernst und dann neue hinzufügst ist die vorherige Selektion futsch. Durch den Renderer sparst du dir dieses Entfernen und wieder Hinzufügen und der Fehler tritt gar nicht auf.