Fragen zur Vererbung und Update

[JasonV]

Moin

Also, ich habe ein kleines Problem. Mein Editor soll nach erfolgreichem Speichern von einer neuen Datei in ein Vorhandes Verzeichnis die ProjectView aktualisieren - da ja nun eine Datei mehr drin ist. Meine Fragen folgen dann nach dem Code.

Ein Ausschnitt aus meinem Code:

public class Gui extends JFrame implements ActionListener {
	private ProjectView projectView = new ProjectView();

	private void ClientGuiAufbau() {
                [...]
		JSplitPane SplitView = new JSplitPane(JSplitPane.HORIZONTAL_SPLIT, projectView.getPV(), 
                                                        tEditor.getTabbedEditor());
                [...]
        }

	public Gui() {
		ClientFrame = new JFrame();
                [...]
		ClientGuiAufbau();
                [...]
       }

	class ProjectView extends JTree implements MouseListener {
		private JTree pView;
		private DefaultMutableTreeNode root;
		
		public JTree getPV() {
			return pView;
		}

		public void update() {
			projects = getProjectsByName();	
			root = new DefaultMutableTreeNode("Projekte");
			for(int i=0; i<projects.length; i++) {
				addNodes(projects[i][0],projects[i][1], root);
			}
			pView = new JTree(root);
			pView.setRootVisible(false);
			pView.putClientProperty("JTree.lineStyle", "Angled");
			pView.addMouseListener(this);			
		}

		public ProjectView() {
			projects = getProjectsByName();	
			root = new DefaultMutableTreeNode("Projekte");
			for(int i=0; i<projects.length; i++) {
				addNodes(projects[i][0],projects[i][1], root);
			}
			pView = new JTree(root);
			pView.setRootVisible(false);
			pView.putClientProperty("JTree.lineStyle", "Angled");
			pView.addMouseListener(this);
		}
        }
}

Ich verwende Sub-Klassen, um den Aufbau der Gui etwas übersichtlicher zu gestalten.

Nun meine Fragen:
1) Kann ich ohne die Methode getPV() auskommen, so wie es ja eigentlich auch die eigentlichen Klassen tun?

2) Wenn ich nun eine neue Datei in einem Projekt speichere, soll diese auch im "projectView" angezeigt werden. Dazu habe ich bisher eine update()-Methode implemetiert, die im Prinzip das gleiche machen soll wie der Konstruktor: anlegen der Einträge (und ggf. setzen der Eigenschaften). Aber ein repaint() bringt hier leider keinen Erfolg. Mir fehlt da irgendwie der "Sprung" von meiner projectView zum eigentlich JTree.

3) Eine generelle Frage: gibt es einen schöneren (bzw. saubereren) Weg, das zu implementieren, als ich das momentan mache? Vor allem das Ansprechen von ausserhalb der Sub-Klasse scheint mir etwas dumm gelöst (also von mir): projectView.pView.add[..] - was also vergleichbar mit einer eleganten Lösung auf Frage 1 wäre.


Hoffe auf Hilfe.

Beste Grüße,
JasonV

 

[Wildcard]

Du brauchst eigene Listener.
Die View ist ein Listener auf dem Model
Im Model wird ein neues Element eingefügt (Datei) -> Model benachrichtigt View über die Listenerschnittstelle.

 

[JasonV]

Du brauchst eigene Listener.
Die View ist ein Listener auf dem Model
Im Model wird ein neues Element eingefügt (Datei) -> Model benachrichtigt View über die Listenerschnittstelle.

Hm. Was für einen Listener und an welcher Stelle? Ein Listener für "projectView"?

 

[Wildcard]

Du musst doch irgendwo beispielsweise die Datein verwalten.
An dieser Stelle weißt du welche Dateien du hast und nur dort dürfen dann auch neue Dateien angelegt werden.
Jede Komponente deiner Anwendung die sich dafür Änderungen an den Dateien interessiert implementiert dann beispielsweise IDateiMangerListener mit fileAdded, fileRemoved, fileChanged Methoden.
Wenn du jetzt eine Datei änderst machst du das über denn 'Manager'.
Der ruft anschließend fireFileAdded auf. In dieser Methode informierst du dann alle Listener die sich vorher auf dem Manager registriert haben über die Änderung.
Diese können dann auf die Events reagieren (Im Falle deiner View würde das dann heißen das die neue Datei in den Baum aufgenommen wird).
Schau dir einfach mal an wie Swing funktioniert. Da hast du überall Listener...
Das ganze nennt sich Observer-Pattern (siehe FAQ oder Wiki)

 

[JasonV]

Du musst doch irgendwo beispielsweise die Datein verwalten.

Hmmm... hatte das bisher so gelöst, dass Änderungen (speichern, schliessen) in der jeweiligen AbstractAction implementiert sind. Mehr als Speichern, Schliessen kommt ja eigentlich nicht vor.

Wenn ich Dich also richtig verstehe:
- In der GuiKomponente (also z.B. ProjectView) füge ich die Methode update(Observable o, Object update) hinzu und sage ihr hier, lade die Projekt-Daten neu + zeige sie dann an
- ich sollte eine Klasse basteln, die sich explizit um die Verwaltung der Daten kümmert - sprich das Löschen, Speichern etc. übernimmt.
- der Klasse drücke ich ein Observable auf
- eine Methode saveFile führt den Speicher-Code aus und macht ein notifyObservers()
- drücke ich save --> Ausführen der Action, in der ich ein neues Objekt der Klasse "Datei" anlege, darauf registriere ich den Observer über "neueDatei.addObserver(MeineGuiKomponente)" und führe dann die Methode "saveFile" der Klasse Datei aus
(==> also ist die GuiKomponente der Observer und meine Klasse fürs Managen der Daten eine Observable?)

So hab ich das jetzt verstanden (anhand Deiner Aussage + der FAQ zu den Observer-Patterns).

 

[Wildcard]

Richtig. Die Details (zB wie granular man das ganze Aufteilt usw.) hängen von der Struktur deines Projekts ab, aber das ist das Prinzip. Du musst auch nicht die Java Obersable/Observer Interfaces verwenden, sondern kannst da was eigenes nehmen.
Nur auf die grob-Struktur kommt's an.
Pass aber auf das es zu jeder addListener Methode auch ein removeListener Methode gibt, damit du keine Speicherleichen bekommst (alternativ kann man zB auch WeakReferences verwenden, aber das ist vielleicht zu viel auf einmal...)

 

[JasonV]

Danke, werde mich mal dransetzen und das probieren. Hört sich von der Idee her ganz gut an. Das sollte Frage 2 beantworten.


Bleiben nur noch die Fragen 1+3.

 

[Wildcard]

Frage 3 hab ich nicht verstanden.
Frage 1 auch nicht so richtig. Da hat's bei mir ehrlich gesagt schon aufgehört als ich gesehen habe das ProjectView ein JTree ist, der einen JTree hat :autsch: :wink:

 

[JasonV]

Frage 3 hab ich nicht verstanden.
Frage 1 auch nicht so richtig. Da hat's bei mir ehrlich gesagt schon aufgehört als ich gesehen habe das ProjectView ein JTree ist, der einen JTree hat :autsch: :wink:

Genau das ist mein Problem. Wie löst man es eleganter? Das hab ich irgendwie nicht so ganz kapiert

Im Haupt-Teil der Gui lege ich ein ProjectView an, welches eigentlich ein JTree ist. Aber ich komme an den JTree irgendwie nicht ran, ausser ich setze nach dem Initialisieren mein ProjectView auf den JTree, den ich erzeuge.

Verwirrend, ich weiss. Eine schönere Lösung würde mich auch glücklicher machen (und Frage 3 evt. direkt mitbeantworten).

 

[Wildcard]

Du musst dich entscheiden:
Entweder du bist (erbst von/erweiterst) ein JTree, oder du instanzierst einen JTree und verwendet ihn.
Und bevor du dich fragen kannst wie du aus anderen Objekten auf den Tree zugreifen kannst, musst du dich fragen warum(oder ob) du darauf zugreifen musst.

 

[JasonV]

Nun, eigentlich wollte ich ihn instanzieren - der Übersichtlichkeit halber (und weil ich die Methoden wie z.B. hier getProjects nur innerhalb der Instanz benötige) habe ich das aber als Klasse gemacht, die von JTree erbt. So kann ich ja um eigene Methoden erweitern und habe nicht gleich alle Methoden in der Haupt-Gui-Klasse.

 

[Wildcard]

Du hast beides gemischt. Das darfst du in keinem Fall.
Entweder, oder.

 

[JasonV]

*edit*