Problem mit Inhalt-Aktualisierung

[Igäl]

Guten Morgen allerseits

Ich habe mich in den letzten paar Stunden (des gestrigen Tages) intensiv mit der Struktur meines Programmes beschäftigt. Dabei ist folgendes raus gekommen:

- Die Klasse Main ruft zwei andere Klassen auf, welche je ein JPanel mit Navigation und Content zurück geben.
- Wenn ein entsprechendes Flag gesetzt wurde, so geht ein neues Fenster auf. Ansonsten passiert: getContentPane().repaint();

import java.awt.*;
import java.awt.event.*;
import javax.swing.*;

public class Main extends JFrame {
	private JPanel pMain = new JPanel();
	private SetNav pNav = new SetNav();
	private SetContent pCont = new SetContent();
	
	public Main(boolean openWindow, int loadId)	{
		pMain.setLayout(new BoxLayout(pMain, BoxLayout.Y_AXIS));
		pMain.add(pNav.getNav());  //Navigation hinzufügen
		pMain.add(pCont.getCont(loadId));  //Content hinzufügen
		
		add(pMain, BorderLayout.PAGE_START);
		
		if(openWindow == true)	{
			addWindowListener(new CloseWindow(true));
			setSize(800, 600);
			setResizable(false);
			setVisible(true);
		}else{
			getContentPane().repaint();
		}
	}
	
	public static void main(String[] args) {
		Main main = new Main(true, 1);
	}

}

- Beim erstmaligen Aufruf der Klasse wird nun also die Startseite (mit der Id 1) geladen und ein neues Fenster geöffnet.
- In der Klasse SetNav() werden nun drei Buttons kreiert und auf jeden dieser Buttons ein ActionListener registriert.
- Bei Klick auf Button 2 rufe ich die Klasse Main wieder auf und zwar so: new Main(false, 2);

import javax.swing.*;

public class SetContent extends JFrame {
	private JPanel pMid = new JPanel();
	private ContentObjects cobj = new ContentObjects();
	
	public JPanel getCont(int pageId)	{
		pMid.removeAll();
		
		switch(pageId)	{
		case 1:		pMid.add(cobj.createLabel("Ich bin das Label 1"));
		break;
		case 2:		pMid.add(cobj.createLabel("Ich bin das Label 2"));
		break;
		default:	pMid.add(cobj.createLabel("Default Label"));
		}	
		
		pMid.validate();
		
		return pMid;
	}
}

In meinem Verständnis wird bei Klick auf Button 2 Main wieder aufgerufen und darin von der Klasse SetContent ein neues JPanel angefordert. Zurückgegeben wird ein pMid, dass das neue JLabel beinhaltet. Da openWindow auf false ist, soll mir kein neues Fenster geöffnet werden, sondern nur die contentPane neu gemalt werden. In einem Test - Script funktioniert dieser Ansatz. Da ist allerdings alles in der gleichen Klasse.

Weiss einer was schief läuft? Ich schaffe es nicht, dass "Ich bin das Label 2" erscheint. Obwohl im Switch case 2 aufgerufen wird bleibt "Ich bin das Label 1" bestehen. Bin grad etwas ratlos.

Danke für die Hilfe.

Gruss Igäl

 

[SlaterB]

mit new Main(false, 2); erzeugst du in jedem Falle ein zweiten Fenster,
völlig unabhängig vom ersten,
das zweite zeigst du nicht an, weshalb es gleich wieder im Mülleimer verschwindet,
auf keinem Fall wird dadurch das erste Fenster in irgendeiner Weise beeinflusst,

statt
new Main(false, 2);
musst du schreiben
referenzAufAltesMain.aendereDeinenInhalt()

notfalls wieder mit so einem merkwürdigen int-Parameter,
besser aber mit einer verständlichen Logik 😉

und wichtig: dieser Änderungscode muss auf jeden Fall in einer Operation stehen,
der Konstruktor steht nicht mehr zur Verfügung

------


ganz andere Variante wäre, ein neues Main-Fenster mit dem zweiten Label zu erzeugen
(so wie du es schon machst, aber auch anzeigen),
und das alte Fenster zu löschen,
sinnvoller ist aber, einfach das vorhandene Fenster zu bearbeiten

 

[Igäl]

Wie kann die Content-Klasse auf das JFrame in der Main-Klasse zugreifen? Muss ich da über den Methodenaufruf eine Referenz übergeben oder kann das die Klasse irgendwie durch einen Kniff, der mir bis anhin verborgen blieb? 🙂

Der Int-Parameter erscheint mir Zweckmässig. Aber vlt sind das alte Gewohnheiten aus PHP. Werde wohl erst mit der Zeit eine Kopfinterne Java-Logik entwickeln.

 

[SlaterB]

> Wie kann die Content-Klasse auf das JFrame in der Main-Klasse zugreifen?

falsch,

richtig ist:
Wie kann die Content-Klasse auf das Main-Objekt zugreifen?
das Main-Objekt ist zufällig auch das JFrame, aber selbst wenn das JFrame eine interne Variable wäre, dann hätte das außerhalb von Main niemanden zu interessieren

Objekt als Parameter übergeben, ja

private SetNav pNav = new SetNav(this);
oder
pMain.add(pNav.getNav(this));

 

[Igäl]

Ok das hab ich soweit verstanden und auch umgesetzt bis zu dem Punkt:

Wie kann ich in der Methode actionPerformed() auf Übergabeparameter zugreifen?

Entweder übergeb ich die Referenz auf das JFrame an die Navigation, dann habe ich sie wohl in der Methode wo ich die ActionListener adde, nicht aber in der Methode actionPerformed, wo ich sie dann bräuchte.

Die andere Lösung wäre eine Methode in der Main-Klasse, die mir die Referenz auf das JFrame zurück gibt:

public JFrame getFrameRef()    {
    return this;
}

Um diese Methode ansprechen zu können, müsste ich aber in der Navigations-Klasse ein neues Objekt der Main-Klasse erstellen, was dazu führt, dass ein neues Fenster aufgeht.

Das Einfachste wäre also, wenn ich, wie du gesagt hast, die JFrame-Referenz als Funktionsparameter übergeben könnte. Nur müsste ich dazu wissen, wie ich in actionPerformed() darauf zurgreifen kann.

Edit: Dafür weise ich den übergebenen Wert wohl einer Klassenvariable zu, auf die auch actionPerformed() zugriff hat...

 

[SlaterB]

immer noch zwei verrückte Punkte:
1.
das Main IST das JFrame, was um alles in der Welt bring eine Operation, die nur return this; enthält?
alles was du mit dem JFrame machen willst, kannst du direkt mit Main machen
2.
du brauchst das JFrame überhaupt nicht,
rufe nicht main.setContentPane() (oder gar main.getFrameRef().setContentPane()) oder ähnliches auf,
überlasse es Main, das JFrame (also zufälligerweise sich selbst) zu ändern!,
du musst nur main.aendereGUI() aufrufen, der Rest passiert INTERN in Main!

im ActionListener brauchst du eine Referenz ja,
je nachdem wie das bei dir aufgebaut ist, muss Main eine Exemplarvariable in der Navigation oder im ActionListener sein

falls der ActionListener eine anonyme innere Klasse ist,
reicht auch eine finale lokale Variable in der Operation, die den ActionListener definiert

 

[Igäl]

Ok. Ich denke jetzt habe ich genügend Infos um das Problem zu lösen. Allerdings habe ich jetzt meinen Quellcode mit dem Rumpröbeln so zerschossen, dass ich erstmal was neues mache 😉

Danke dir Slater... Bei einigen Dingen wird mir langsam klar, dass sie mir eigentlich klar sein müssten. Aber irgendwie sieht man am Anfang vor lauter Bäumen den Wald nicht mehr.

 

[Igäl]

Nochmals was kleines:

Ich hab ein Hauptpanel. Dem sind zwei andere Panels zugefügt. Eines dieser zugefügten Panels veränder ich. Reicht es da, einfach einen repaint() auf das JFrame zu machen oder muss ich die mit den Panels noch was spezielles anfangen?

 

[SlaterB]

du hast doch schon eine JPanel-verändernde Operation mit

public JPanel getCont(int pageId) {
....
pMid.validate();
return pMid;
}

da validate() ist also meist nötig, wenn die Struktur verändert wird

 

[Igäl]

Das hat irgendwie nicht hingehauen...

Nun habe ich ein Testprogramm geschrieben. Alles in der selben Klasse. Vier Buttons: Zwei färben den Hintergrund und zwei blenden ein eigenes Panel ein. Das funktioniert wunderbar. Jetzt wollte ichs in verschiedene Klassen verschieben und halt einfach die Referenzen anpassen und jetzt harkt es wieder.


Da wo alle in der selben Klasse sind, siehts so aus:

public void actionPerformed(ActionEvent ae)   {
	if(ae.getSource() == color)   {
		getContentPane().setBackground(Color.red);
	}
[...]
}

Bei der Version wo ich mehrer Klassen habe so:

JFrame frmRef;
[...]
public JPanel getTopPan(JFrame frmRef)   {
	[...]
	this.frmRef = frmRef;
	[...]
}

[...]

public void actionPerformed(ActionEvent ae)   {
	if(ae.getSource() == color1)   {
		frmRef.getContentPane().setBackground(Color.red);
	}
	[...]
}
[...]

Das obere Beispiel funktioniert, das untere nicht. Aber eigentlich zeigen doch beide auf den selben Ort. Häng grad wieder zünftig -.-

 

[SlaterB]

tja, ich zumindest sehe den Fehler nciht auf Anhieb,
baue für beide oder zumindest die fehlerhafte Variante ein komplett lauffähiges Programm (<100 Zeilen),
dann kann man den Fehler haargenau angeben

 

[Igäl]

Aber ist folgendes Szenario überhaupt möglich?


- Panel A enthält Panel B und C und wird in Klasse A generiert.
- Panel B und C bekommt es aus Klasse B und C
- Panel A wird nun dem JFrame zugeordnet und angezeigt
- User Klickt auf Button, Panel C wird in Klasse C geleert [removeAll()], neu gefüllt [irgendwelcher Inhalt] und aktualisiert [validate()]. Danach wird das JFrame aus Klasse A über die mitgegeben Referenz auf das JFrame aktualisiert und in Panel C sollte nun der neue Inhalt angezeigt werden.


Also geht das generell, dass ich ein Panel, welches in ein anderes eingebunden ist einfach mit removeAll und validate verändern oder muss ich das Hauptpanel ebenfalls irgendwie modifizieren?

 

[SlaterB]

ein Panel kann geleert werden, ja,
dazu muss (normalerweise) nicht noch irgendein anderes Panel geleer werden

 

[Igäl]

Das Panel das ich verändere wurde nur im Konstruktor der Hauptklasse einmal ins Fenster reingesetzt. Kann es sein, dass er nicht mehr weiss, was verändert wurde, wenn ich das Panel nun in der Klasse C verändere?

 

[SlaterB]

kann sein, kann nicht sein,

ich weiß allerdings inzwischen nicht mehr wovon du redest,
ob jemand etwas weiß oder vergessen hat ist keine sinnvolle Diskussion in der Programmierung

 

[Igäl]

Ich weiss glaub worans liegt.

Jedes Mal wenn ich die Funktion zum Überschreiben des alten Panelinhaltes aufrufe, wird ein neues Panel generiert (weil ich halt die Klasse lade und da automatisch ein neues Panel gemacht wird). Das heisst, dass es vermutlich schon funktionieren würde... das neue Panel hat bestimmt das neu generierte Label darin. Nur wird es nie ins Haupt-Panel geladen und daher auch nie angezeigt... Mal sehen wie ich diesen Missstand behebe.

 

[Igäl]

Daran lag es. Das Panel wurde jedes Mal überschrieben und das neue Panel nicht angezeigt. Hm.. ich scheine doch was zu lernen. Allerdings langsam -.- Danke dir Slater

Edit: Allerdings weiss ich noch immer nicht warum sich die Hintergrundfarbe nicht ändert ^^ ist aber erstmals Zweitrangig.

 

[Leroy42]

Hm.. ich scheine doch was zu lernen. Allerdings langsam -.- Danke dir Slater

Na bravo @SlaterB!

Nur wegen dir, lernt Igäl jetzt nur noch langsam. :shock:

Schäm dich! ( 😎 )

 

[SlaterB]

kannst mich Slater nennen