Pb de mise à jour d'un JComboBox

Version imprimable

Salut à tous,

Voilà j'ai codé un calendrier qui marche et maintenant je suis en train de l'améliorer afin de pouvoir changer la locale du calendrier et ainsi le mettre à jour dynamiquement. Le changement est effectif pour les jours de la semaine et du mois mais pas pour le JComboBox des mois. Soit je récupère l'exception : Exception in thread "main" java.lang.NullPointerException at Calendar.actionPerformed(Calendar.java:95) ou soit j'ai les mois dans les 2langues que j'utilise qui s'affichent dans le JComboBox. Je sais où est le problème mais je n'arrive pas à le corrgier. Voilà mon code :

Code:

1
2
3
4
5
6
7
8
9
10
11
12
13
14
15
16
17
18
19
20
21
22
23
24
25
26
27
28
29
30
31
32
33
34
35
36
37
38
39
40
41
42
43
44
45
46
47
48
49
50
51
52
53
54
55
56
57
58
59
60
61
62
public class Calendar {
    ...
    static JComboBox monthsCombo;
    static String[] months;
    static Month[] onlyMonths;
    ...
 
    public Calendar(JPanel westMainWindowPanel, Locale locale){ 
 	...
	monthsCombo = new JComboBox();
        updateMonth();
  	...	
	monthsCombo.addActionListener(new ActionListener(){
             public void actionPerformed(ActionEvent e){
		System.out.println(monthsCombo.getSelectedItem()); // Affichage de null			
                System.out.println(((Month)monthsCombo.getSelectedItem()).getMonthIndex()); // J'ai l'exception qui se fini ici
	            ...
	     }
            }); 
    }
 
    // MAJ du calendrier
    protected static void updateCalendarLocale(Locale locale){ 
            ...	
            updateMonth();
	    // Les autres MAJ
            ...
    }
 
    // MAJ du combo box des mois
    private static void updateMonth(){ 
        // Je récupère les mois en fonction de la locale
    	months = dateSymbols.getMonths();
    	onlyMonths = new Month[Calendar.UNDECIMBER];
    	String month = "";
 
        // Ce for permet d'enlever le 13ième mois vide retourné par la méthode getMonths() et d'avoir des couples (numéro du mois, nom du mois)
 	for (int i = Calendar.JANUARY; i < Calendar.UNDECIMBER; i++){
 	         month = months[i].substring(0,1).toUpperCase()+months[i].substring(1);
 	         onlyMonths[i] = new Month(i, month);
 	}
 
        // Je test si le combo a déjà été affiché
        // Si c'est le cas je le met à jour, du moins j'essaye parce que c'est ici que ça bloque
	if (monthsCombo.getItemCount() == 12) {
                 // J'ai l'exception qui commence ici
 	         monthsCombo.removeAllItems();			
 
                 // Je rempli le JcomboBox avec les nouvelles valeurs
 	         for (int i = Calendar.JANUARY; i < Calendar.UNDECIMBER; i++){
                          monthsCombo.addItem(onlyMonths[i]);
                          System.out.println(monthsCombo.getItemAt(i));
 	         }
	}
        // Sinon je le crée
	else {
		monthsCombo = new JComboBox(onlyMonths);
	}
 
	monthsCombo.setSelectedIndex(selectedMonth);
    }
}

Sinon, je ne sais pas si je devrais utiliser un ComboBoxModel. Comment ça marche et quel serait son utilité dans mon cas ?

Merci d'avance pour vos réponses

08/12/2006, 17h54
Core8

Personne n'a d'idées pour résoudre mon problème ? C'est surement une erreur bête mais je n'arrive pas à la touver.
12/12/2006, 20h44
Core8

Franchement un petit coup de pouce serait le bienvenu ;) .

J'ai encore essayé aujourd'hui mais je n'arrive pas à mettre dynamiquement le contenu de mon JComboBox.

J'ai réussi à le faire dans un autre cas dans une fenêtre pop-up parce que dans ce cas le JComboBox est recrée à chaque affichage de la fenêtre donc il n'y a pas de problème pour la mise à jour du contenu.

Mais sans le recréer j'ai soit l'exception (dans mon post initial) soit mon JComboBox après une sélection reste "bloqué" sur le même item : getSelectedItem() retroune toujours le même item peut importe celui sélectionné. Voilà mon code de la méthode updateMonth pour ce cas :
Code:

1 2 3 4 5 6 7 8 9 10 11 12 13 14 15 16 17 // MAJ du combo box des mois private static void updateMonth(){ // Je récupère les mois en fonction de la locale months = dateSymbols.getMonths(); onlyMonths = new Month[Calendar.UNDECIMBER]; String month = ""; // Ce for permet d'enlever le 13ième mois vide retourné par la méthode getMonths() et d'avoir des couples (numéro du mois, nom du mois) for (int i = Calendar.JANUARY; i < Calendar.UNDECIMBER; i++){ month = months[i].substring(0,1).toUpperCase()+months[i].substring(1); onlyMonths[i] = new Month(i, month); } monthsCombo = new JComboBox(onlyMonths); monthsCombo.setSelectedIndex(selectedMonth); }

13/12/2006, 20h43
BainE

Salut,

Ben oui mais t'as qu'une seule combo...
Peut etre en en mettant plusieurs au sein d'un GroupButton, et d'appeler le getselectedIndex sur lui...
14/12/2006, 22h12
Core8

Citation:

Envoyé par BainE

Salut,

Ben oui mais t'as qu'une seule combo...
Peut etre en en mettant plusieurs au sein d'un GroupButton, et d'appeler le getselectedIndex sur lui...

Oui, j'ai qu'un seul combo box mais je ne vois ce que celà va apporter à mon problème si j'en rajoute un et que j'appelle getselectedIndex sur lui.

Personne d'autre aurait une idée pour résoudre mon problème svp.
15/12/2006, 10h55
BainE

oui mais si t'as qu'en seul combo, il sera forcement celui selectionné.
Et l'appel du getSelectedIndexdevrait etre fait sur le GroupButton.

Je comprend pas bien comment tu pourrais avoir un indice different de 1 si t'es qu'un seul combo
15/12/2006, 11h55
Core8

Citation:

Envoyé par BainE

oui mais si t'as qu'en seul combo, il sera forcement celui selectionné.
Et l'appel du getSelectedIndexdevrait etre fait sur le GroupButton.

Je comprend pas bien comment tu pourrais avoir un indice different de 1 si t'es qu'un seul combo

Je crois que tu n'as pas bien compris mon problème.

En fait j'ai un JComboBox qui a pour items les 12 mois de l'année donc ces mois on chacun un indice différent.
Le problème que je rencontre c'est de mettre à jour dynamiquement ce JComboBox : c'est-à-dire dans mon cas de le re-remplir avec les 12 mois de l'année dans une nouvelle langue (dynamiquement).
J'ai réussi à le faire dans un autre cas dans une fenêtre pop-up parce que dans ce cas le JComboBox est recrée à chaque affichage de la fenêtre donc il n'y a pas de problème pour la mise à jour du contenu.

Mon code qui est censé faire ça et qui ne marche est dans mon post initial.

J'ai essayé de résoudre mon problème en utilisant un ComboBoxModel mais j'ai le même problème que mon post du 13/12/2006. Je dois surement mal mis prendre mais où là j'ai vraiment besoin de votre aide parce que je suis largué. Ca fait maintenant plus de 2 semaines que j'ai entre autre ce problème et je tourne en rond. Si ça peut vous aider voilà mon modèle (c'est mon premier modèle alors il doit surement il y avoir des fautes :( ) :

Code:

1
2
3
4
5
6
7
8
9
10
11
12
13
14
15
16
17
18
19
20
21
22
23
24
25
26
27
28
29
30
31
32
33
34
35
36
37
38
39
40
41
42
43
44
45
46
47
48
49
50
51
52
53
54
55
56
57
58
59
public static class MonthComboBoxModel extends AbstractListModel implements ComboBoxModel{
 
	private GCMonth[] months;
	private GCMonth item;
	private final ArrayList<ListDataListener> listeners = new ArrayList<ListDataListener>();
 
	public MonthComboBoxModel(GCMonth[] months){
		this.months = months;
	}
 
	public GCMonth getSelectedItem(){
		return item;
	}
 
	public void setSelectedItem(Object monthSelected){
		item  = (GCMonth)monthSelected;
	}
 
	public GCMonth getElementAt(int index){
		return months[index];
	}
 
	public int getSize(){
		return months.length;
	}
 
	public void addListDataListener(ListDataListener l){
		listeners.add(l);
	}
 
	public void removeListDataListener(ListDataListener l){
		listeners.remove(l);
	}
 
	public void removeAllItems(GCMonth[] newMonths){
		months = newMonths;
	}
}
 
// Ma méthode updateMonth est maintenant la suivante
private static void updateMonth(){ 
	String[] months = dateSymbols.getMonths();
	GCMonth[] onlyMonths = new GCMonth[Calendar.UNDECIMBER];
	String month = "";
 
	for (int i = Calendar.JANUARY; i < Calendar.UNDECIMBER; i++){
		month = months[i].substring(0,1).toUpperCase()+months[i].substring(1);
		onlyMonths[i] = new GCMonth(i, month);
	}
 
	final MonthComboBoxModel monthModel = new MonthComboBoxModel(onlyMonths);
	monthsCombo = new JComboBox(monthModel);
 
	if (monthsCombo.getItemCount() == 12) {			
		monthModel.removeAllItems(onlyMonths);
	}
 
	monthsCombo.setSelectedIndex(selectedMonth);
}