GUI Absturz nach 3x benutzen einer JComboBox

[Asumanji]

Hallo Comm,
ich bin grade dabei einen Kalender grafisch zu Programmieren. Der stellt beim starten den aktuellen Monat dar.
Mit 2 JComboBoxen ändere ich das ausgewählte Jahr mit der einen und den ausgewählten Monat mit der anderen.
Das heißt in diesem Fall das alle Elemente entfernt und passend für den neuen Monat erstellt werden.
Das Problem das ich nun habe ist, das sich das Programm aufhängt und sich nur noch über den Taskmanager schließen lässt, wenn ich 3x einen Eintrag in der JComboBox ändere.
Dabei ist die Reihenfolge der Änderung egal, also 3x Jahr ändern hat den selben Effekt wie 3x Monat ändern oder 2x Monat und 1x Jahr ändern.
Hat jemand schon mal ein ähnliches Problem gehabt?
Vielen Dank für eure Zeit
Gruß ich

 

[Joose]

Wenn die GUI hängen bleibt gibt es meist einen Programmierfehler.
Ohne Code kann man nicht wirklich helfen.

PS: Abstürzen ist etwas anderes als "hängen bleiben"

 

[Asumanji]

Danke für die schnelle Antwort. Du hast natürlich Recht, es hätte ja sein können das es ein bekanntes Problem ist.

this.cJahr.addActionListener(new ActionListener(){
			@Override
			public void actionPerformed(ActionEvent e){
				tKalender.removeAll();
				String strJahr = (String) cJahr.getSelectedItem();
				currYear = Integer.parseInt(strJahr);
				firstDay = new GregorianCalendar(currYear, currMonth, 1);
				DoW = firstDay.get(Calendar.DAY_OF_WEEK);
				mt = 31;
				initComponents();
				initListeners();
				tabpane.setSelectedIndex(6);
			}
		});
		
		this.cMonat.addActionListener(new ActionListener(){
			@Override
			public void actionPerformed(ActionEvent e){
				tKalender.removeAll();
				String strMonat = (String) cMonat.getSelectedItem();
				switch(strMonat){
					case "Januar": currMonth = 0;
					break;
					case "Februar": currMonth = 1;
					break;
					case "März": currMonth = 2;
					break;
					case "April": currMonth = 3;
					break;
					case "Mai": currMonth = 4;
					break;
					case "Juni": currMonth = 5;
					break;
					case "Juli": currMonth = 6;
					break;
					case "August": currMonth = 7;
					break;
					case "September": currMonth = 8;
					break;
					case "Oktober": currMonth = 9;
					break;
					case "November": currMonth = 10;
					break;
					case "Dezember": currMonth = 11;
					break;
				}
				firstDay = new GregorianCalendar(currYear, currMonth, 1);
				DoW = firstDay.get(firstDay.DAY_OF_WEEK);
				mt = 31;
				initComponents();
				initListeners();
				tabpane.setSelectedIndex(6);
			}
		});
	}

Das sind meine Listener für die JComboBoxen.
Ich habe da schon dran rumgebastelt, aber es ändert an meinem Problem nix.
Ich hoffe das ist übersichtlich genug.

 

[Asumanji]

Anhang anzeigen 7393
So sieht es aus und so soll es auch aussehen.
Eclipse zeigt noch ein paar Warnungen an:

	private JLabel		lblJahr			= new JLabel("Jahr:");
	private JComboBox	cJahr			= new JComboBox(strJahr);
	private JLabel		lblMonat		= new JLabel("Monat:");
	private JComboBox	cMonat			= new JComboBox(strMonate);

Warnung:

Description	Resource	Path	Location	Type
JComboBox is a raw type. References to generic type JComboBox<E> should be parameterized	Verwaltung.java	

Description	Resource	Path	Location	Type
Type safety: The constructor JComboBox(Object[]) belongs to the raw type JComboBox. References to generic type JComboBox<E> should be parameterized	Verwaltung.java

In strJahr sind die Jahre 2014, 2015 und 2016 gespeichert und in strMonate die Monate Januar bis Dezember, jeweils als

private String[] strMonate = {"Januar", "Februar", "März", "April", "Mai", "Juni", "Juli", "August", "September", "Oktober", "November", "Dezember"};

	private String 		lastYear		= String.valueOf(currYear - 1);
	private String 		thisYear		= String.valueOf(currYear);
	private String 		nextYear		= String.valueOf(currYear + 1);
	private String[] 	strJahr			= {lastYear, thisYear, nextYear};

Die erste Warnung taucht für jede der beiden JComboBox 2x auf und die 2. Warnung einmal für jede JComboBox.
Das Programm lässt sich auch ausführen, zumindest bis man zum 3. mal Monat/Jahr ändern will.

 

[Harry Kane]

Von drei möglichen Listenern (Jahr, Monat, Tag) zeigst du zwei. Was ist mit dem dritten? Und auch bei den beiden gezeigten Listenern fehlt viel zu viel, als das ich da was gescheites zu sagen könnte. In welcher Klasse befinden sich die Listener? Was ist tKalender und tabPane? Was pasisert in initComponents und initListeners?
Die Namen der zuletzt genannten Methoden legen die Vermutung nahe, daß du deine Gui in den actionPerformed() Methoden jedesmal neu aufbaust, was wiederum ein sehr schlechtes Design wäre.

 

[Asumanji]

Von drei möglichen Listenern (Jahr, Monat, Tag) zeigst du zwei.

Wozu brauche ich einen für den Tag? Die beiden Listener prüfen die Änderungen in den JComboBoxen um festzustellen welchen Monat und welches Jahr der Nutzer gewählt hat. Standardmäßig ist der aktuelle Monat im aktuellen Jahr offen, also Februar 2015.

Was ist mit dem dritten? Und auch bei den beiden gezeigten Listenern fehlt viel zu viel, als das ich da was gescheites zu sagen könnte. In welcher Klasse befinden sich die Listener?

Wenn ich das richtig im Blick hab, denn hab ich nur eine Klasse für alles.

Was ist tKalender und tabPane?

Ich hab einen JTabbedPane(left) tabPane und tKalender ist der Tab in dem der Kalender ist.

Was pasisert in initComponents und initListeners?

initComponents() bezogen nur auf den Kalender und in der Reihenfolge:
- GridLayout(0,7) da ich ja 7 Wochentage habe, Anzahl der Zeilen sind dynamisch je nach Monat
- Label für die 7 Wochentage werden hinzugefügt
- Variable mt (Tage im Monat) wird per Switch auf 30, 29 oder 28 gesetzt, Standard ist 31
- per Schleife werden leere Label erstellt, sodass der erste Tag im Monat auch den richtigen Wochentag zugeordnet bekommt
- per Schleife wird dann für jeden Tag ein JButton erstellt und Samstage und Sonntage gelb gefärbt
- nach dem letzen JButton eine Schleife für leere Label, damit die JComboBoxen auch wirklich immer unten rechts stehen
- dann werden Label für Jahr, ComboBox für Jahr, Label für Monat und ComboBox für Monat hinzugefügt
- dann nur noch die Einstellungen das der Text in den Labeln zentriert ist und die ComboBoxen auch den aktuelen Monat und Jahr anzeigen

initListeners():hier stehen die Listener für die beiden ComboBoxen drin. Für die einzelnen Tage (sind ja alles Buttons) sollen auch noch Listener dazukommen.

Die Namen der zuletzt genannten Methoden legen die Vermutung nahe, daß du deine Gui in den actionPerformed() Methoden jedesmal neu aufbaust, was wiederum ein sehr schlechtes Design wäre.

Jop, wie beschrieben werden halt alle Elemente entfernt und wieder komplett neu aufgebaut.
Wie kann man es besser machen? Ich bin ja davon ausgegangen das jeder Monat anders ist.

Danke für deine Antwort

 

[Harry Kane]

initListeners():hier stehen die Listener für die beiden ComboBoxen drin. Für die einzelnen Tage (sind ja alles Buttons) sollen auch noch Listener dazukommen.

Die Comboboxen haben doch schon anonyme Listener. Warum brauchen sie noch mehr?

 

[Asumanji]

Was meinst du mit anonymen Listenern? Ich muss doch für jedes Objekt das ein Event auslösen soll einen Listener erstellen, oder sehe ich das falsch?

 

[Harry Kane]

Was meinst du mit anonymen Listenern?

In deinem ersten Codeschnipsel aus Post 3 legst du zwei anonyme ActionListener an bzw. zwei anonyme Klassen, die das ActionListener interface implementieren und als ActionListener für die JComboBoxen verwendet werden. Der Begriff "anonyme Klasse" ist dir aber schon geläufig?

Ich muss doch für jedes Objekt das ein Event auslösen soll einen Listener erstellen, oder sehe ich das falsch?

Nein, das siehst du richtig. Nur wie oben bereits geschrieben, haben die beiden JComboBoxen für Auswahl des Jahres und des Monats jeweils einen anonymen ActionListener. Dann schreibst du, dass in "initListeners() die Listener für die beiden ComboBoxen drinstehen". Entweder dein Schnipsel oben mit den anonymen Klassen ist aus der initListeners-methode, oder du legst verpasst den JComboBoxen womöglich zwei ActionListener, die sich irgewndwie in die Quere kommen.
Poste mal ein SSCCE. Dann erübrigt sich das Rumraten und spekulieren.

 

[Asumanji]

Danke, du hast mir echt geholfen.
Das Problem war das ich die Listener bereits initialisiert hatte und dachte sie werden wie die Objekte wieder entfernt.
Das führte dazu das ich bei jedem Ändern von den ComboBoxen die Listener wieder neu (oder zusätzlich?) initialisiert habe.
Das Problem ließ sich also dadurch beheben das ich den Methodenaufruf für die Listener aus den Events der ComboBoxen raus genommen habe.

Aber um zu meiner Frage zurückzukommen: du sagtest ja, dass das entfernen der Elemente und das neu aufbauen ein schlechtes Design ist. Wie kann man das besser machen?
Was heißt SSCCE? Meine du damit das "Kurzes, Selbständiges, Kompilierbares Beispiel"?
Und nein ich weiß nicht, was anonyme Klassen sind, kann mir aber denken das damit, wie du schon sagst, die Listener gemeint sind, da diese anders aufgerufen werden.

 

[Harry Kane]

Wie man das besser machen kann, ist angesichts der Tatsache, dass ich nicht genau weiß, wie du es aktuell implementiert hast, schwer zu sagen.
Anstatt neue JLabel und JButtons zu erzeugen, könntest du ihnen z. B. einen neuen Text verpassen und JButtons, die für den aktuellen Monat nicht gebraucht werden, auf disabled oder invisible setzen. Dafür bräuchtest du aber einen LayoputManager, der sich um das visible-flag einer JComponent schert. ich weiss nicht, ob Gridlayout das tut (ich verwende seit Jahren für praktisch alle Layoutaufgaben das MigLayoutm und dort gibt es verschiedene Möglichkeiten, wie eine unsichtbare Component behandelt werden soll).
So wie du es beschrieben hast, zeigt dein JPanel mit den buttons und labeln nur die Daten für den aktuellen Monat. Das macht es zum einen schwierig zu programmieren, weil sich die Anzahl von Monat zu Monat ändert, und zum anderen finde ich es auch weniger übersichtlich als ein paar Tage vom vorangehenden und folgenden Monat mit anzuzeigen.
Du könnest z. B. ein GridLayout(7, 7) verwenden und die erste Reihe von Montag bis Sonntag grundsätzlich für den vorangehenden Monat verwenden. In der zweiten Reihe würde dann erstmals ein Tag aus dem aktuellen Monat auftauchen (in Spalte 1 bis 7, je nachdem auf welchen Wochentag der 1. des Monats fällt), dei Reihen 3, 4, 5 und ev. 6 würden die weiteren Tage des aktuellen Monats anzeigen, und ein Teil der 6. und die gesamte 7. Reihe die Tage des Folgemonats.

 

[Asumanji]

So wie du es beschrieben hast, zeigt dein JPanel mit den buttons und labeln nur die Daten für den aktuellen Monat. Das macht es zum einen schwierig zu programmieren, weil sich die Anzahl von Monat zu Monat ändert, und zum anderen finde ich es auch weniger übersichtlich als ein paar Tage vom vorangehenden und folgenden Monat mit anzuzeigen.

Ja, es wird nur der aktuelle Monat angezeigt.
Schwierig fand ich es nicht, die meisten Monate haben ja 31 Tage, also hab ich eine Variable "int mt = 31" deklariert. Dann hab ich den ausgewählten Monat geholt und per switch case gesagt welcher Monat 30 Tage und welcher 28 Tage hat. Für das Schaltjahr auch noch eine If-Anweisung geschrieben, die besagt nur Jahr % 4 == 0 und dann die Variable auf 29 gesetzt. Nun kann ich die Variable mt als Obergrenze für die Schleife nehmen. Funktioniert auch alles problemlos. Muss bloß die Variable mt in jedem Event wieder auf 31 setzen.

Ich glaube es wird schwieriger den letzten Monat mit einzubringen wie du es beschreibst, da der letzte Tag ja der 31., 30., 29. oder 28. sein kann und ich somit 2 Monate betrachten muss.
Beim Folgemonat wirds nicht so problematisch da jeder Monat mit dem 1. beginnt.
Vielleicht stell ich mir das auch grad komplizierter vor als es eigentlich ist.

Ich werd es mir auf jeden Fall mal ansehen und auch die Variante mit dem Visible = false ausprobieren.