Swing JTree + JScrollPane, refresh Problem

[RELAXccc]

Hallo Forum,
ich habe einen "Datei Explorer" aus JScrollPane & JTree Gebastelt.
Allerdings kann man nicht scrollen wenn ich den JTree im nachnein verändere(verlängere). ???:L
@EDIT: Also gibt es mehr TreeNode's aber die ScrollBar erscheint nicht -> man sieht nicht alle Einträge.
Mein Code:

//Implementierung:
JTree tree;
	JScrollPane scrollTree;

DefaultTreeModel treeModel;
	DefaultMutableTreeNode dir;
//Erstes erstellen:
tree = new JTree(treeModel);
		scrollTree = new JScrollPane(tree);
//methode für Änderungen während Laufzeit:
public void setTree(String dir, String[] files) {
		this.dir.setUserObject(dir);
		this.dir.removeAllChildren();

		DefaultMutableTreeNode file = null;
		
		for (int i = 0; i < files.length; i++) {

			String temp = "";

			if (files[i].startsWith("<dir<")) {

				temp = files[i].split("<dir<")[1];

				file = new DefaultMutableTreeNode(temp);
				file.add(new DefaultMutableTreeNode("..."));
				this.dir.add(file);

			} else if (files[i].startsWith("<file<")) {
				temp = files[i].split("<file<")[1];
				file = new DefaultMutableTreeNode(temp);
				this.dir.add(file);
			}

		}

		treeModel.reload();

	}

Habe im Forum oder sonst wo keine Hilfe gefunden,
darum währe es cool wen jemand mein Problem lösen könnte.
Meine Vermutung:

"Nur"

treeModel.reload();

ist falsch.

Danke im Voraus.
Gruß RELAXccc.


@EDIT: Video zum Problem:
Stream:
JTree Problem. www.java-forum.org
Downloaden(falls Stream nicht mehr verfügbar):
JtreeProblem.mp4 (349,09 KB) - uploaded.to

 

[Tobse]

versuch mal repaint() auf dem Frame oder dem JScrollPane.

 

[hdi]

Die Frage ist was treeModel.reload() macht? Sofern du ein DefaultTreeModel verwendest, sollte nach Änderung der Daten für den Tree eine der fire()-Methoden aus dieser Klasse aufgerufen werden. Diese führen zu einem Repaint-Event, wie vom Vorredner angesprochen. Ein repaint() per Hand würde ich nicht empfehlen, da das meines Wissens nach zu einem vollständigen Neuzeichnen des Trees führt, was in dem meisten Fällen wohl Overkill ist und insofern unperformant.

edit ach sry das ist ja eine Methode aus der API. Hab grad keine Zeit mehr Tür hat grad geklingelt Melde mich später nochmal wenn dir bis dahin inemand hilft

 

[RELAXccc]

1.Habe

treeModel.reload();
scrollTree.repaint();
this.repaint();

versucht, aber funktioniert immer noch nicht.

2.Ich Habe ein kleines 30-Sec. Video gemacht:
Stream:
JTree Problem. www.java-forum.org
Downloaden(falls Stream nicht mehr verfügbar):
JtreeProblem.mp4 (349,09 KB) - uploaded.to

Hoffe es hilft.
Gruß RELAX.

 

[hdi]

reload

weist so wie ich die API verstehe nur auf eine Änderung einer Node hin, d.h. bezüglich ihres Inhalts. Und nicht, dass eine neue Node dazu gekommen ist (oder eine entfernt wurde.)

Was du brauchst ist wohl

treeModel.fireTreeNodesInserted

Die beiden repaint()-Zeilen lösch erstmal wieder raus. Sollte nicht nötig sein.

 

[RELAXccc]

hdi,
ich kann

protected  void fireTreeNodesInserted

nicht aufrufen.

 

[hdi]

Ach Gott, warum sind die protected.. Macht auch nix, dann über den Umweg:

treeModel.nodesWereInserted

Warum sie die fire-Methoden protected machen und dann redundante public Methoden schreiben die an die fire-Methode delegieren erschließt sich mir nicht ganz, aber egal.

PS: Schau dir doch einfach mal die Methodenliste von DefaultTreeModel an. Ich mach grad nix anderes... Du musst dir merken (das gilt auch für JList, JTable, ...): Wann immer du die Daten änderst musst du die GUI über diese Änderung benachrichtigen damit alles korrekt angezeigt wird. Normalerweise über die fire-Methode der DefaultModel-Klasse. In diesem Fall heißen die public Methoden halt anders.

 

[RELAXccc]

Ja sry, hab ich kurz nach meinem post auch bemerkt ,
aber leider versteh ich die Methode nicht ganz.

Sry aber ich hab keine Ahnung wie ich die aufrufen soll.
(TreeNode node, int[] childIndices)

 

[hdi]

Invoke this method after you've inserted some TreeNodes into node. childIndices should be the index of the new elements and must be sorted in ascending order.

Der erste Parameter ist also die Parent-Node. In deinem Fall wohl der Ordner. Das int-Array muss die Indizies der neu eingefügten Child-Nodes (die Dateien in deinem Fall) enthalten.

Angenommen du hast in deiner Node "directory1" im Moment 4 Einträge (Indizes 0-3), und du fügst 3 neue (ans Ende) hinzu:

nodesWereInserted(directory1, new int[]{4,5,6});

So wie ich das sehe löscht du jedesmal alle Einträge und fügst dann nochmal komplett neu ein?!
In dem Fall gehen die Indizies über die gesamte Menge an Einträgen.

Wenn man sich die Methodenliste weiter anschaut stellt man fest dass es übrigens noch einfacher geht:

public void insertNodeInto(MutableTreeNode newChild,
MutableTreeNode parent,
int index)

Invoked this to insert newChild at location index in parents children. This will then message nodesWereInserted to create the appropriate event. This is the preferred way to add children as it will create the appropriate event.

Also: Statt deinem this.dir.add() nimm einfach inserNodeInto(...) - fertig!

Ein gut gemeinter Ratschlag: Schau dir die Klassen mit ihren Methoden erstmal sorgfältig an, bevor du damit arbeitest. Hättest du dir die Methodenliste von DefaultTreeModel mal komplett angesehen (d.h. auch die Texte zu den Methoden lesen und verstehen), dann wäre dir aufgefallen dass die add()-Methode wohl nicht der optimale Weg ist. Steht ja schon in der Beschreibung, dass obige Methode der Weg ist den man wählen sollte

(Ich hab vor ewiger Zeit das letzte mal mit Trees gearbeitet, deswegen hatte ich die richtige Methode da jetzt leider auch nicht im Kopf)

 

[hdi]

Noch ein Nachtrag:

Diese

insertNodeInto

-Methode ist zwar der simpelste Weg eine Node zu adden - es kann sich aber negativ auf die Performance auswirken wenn du mit dieser Methode mal eben 15000 Nodes addest. Weil eben bei jedem mal eine größere Benachrichtungskette ausgelöst wird die zu einem Repaint der GUI führt.

Wenn du es schön coden willst, dann bleib bei deiner add() und führ am Ende, wenn du alle Einträge eingefügt hast, einmalig die

nodesWereInserted

-Methode aus.

 

[hdi]

Weil du mir eine PN geschickt hast.. Diskutieren wir hier weiter, dann haben andere auch etwas davon.

Ich bin mir grad noch immer nicht ganz im Klaren was im Moment das Problem ist? Hast du denn das, was ich in meinem letzten Post geschrieben habe mal versucht? Einfach statt dem treeModel.reload() ein entsprechendes treeModel.nodesWereInserted.

 

[RELAXccc]

Ja habe ich versucht, aber es hat nichts davon funktioniert.
Dann hab ich nochmal versucht von Anfang an mehrere Nodes einzufügen,
und da erschien komischerweise auch keine Scrollbar mehr.
Daher merkte ich das etwas an meinem Code nicht stimmen muss.
Da ich den aber nicht ganz Öffentlich posten wollte hab ich dir ne PN mit der GUI Klasse geschrieben.

 

[hdi]

Okay also zu deiner GUI-Klasse:

1. Das Look & Feel solltest du nicht im Konstruktor deines Frames setzen. Das sollte vor der Erzeugung jeglicher GUI-Komponenten geschehen. Am besten 1.Zeile deiner main-Methode. (auch wenn es jetzt funktioniert, ich sag's dir nur generell).

2. setResizable(true) ist die Default-Einstellung. Das kannst du dir also sparen.

3. Beim Start der GUI sollte der Tree ordnungsgemäß angezeigt werden. Ist doch der Fall, oder?

4. in der setTree() Methode am Ende das treeModel.reload() rausnehmen, statt dessen:

treeModel.nodeStructureChanged(this.dir);

Damit sollte der Tree nach Aufruf der setTree() Methode den neuen korrekten Zustand anzeigen. Und die Sache mit der Scrollbar sollte ebenso funktionierne.

 

[RELAXccc]

das Problem ist das am Anfang bei

private void setDefaultTree() {
		String[] files = { "Darteien.exe", "Texte.txt", "Ordner [dir]",
				"WeitereDartei.ccc",".",".",".",".",".",".",".",".",".",".",".",".",".",".",".",".",".",".",".",".",".",".","." };
		dir = new DefaultMutableTreeNode("Ordner");

Auch keine Scrollbar angezeigt wird

 

[hdi]

 tree.setPreferredSize(new Dimension(300, 0));

löschen, statt dessen:

scrollTree.setPreferredSize(new Dimension(300, 500));

 

[RELAXccc]

Danke! Jetzt Funktionierst es.

 

[hdi]

Okay. Die nächste Aufgabe (im Hinblick auf besseres Design) wäre es dann, den eigentlichen Datenbestand (Dateien und Ordner-Information) in eine eigene Datenstruktur auszulagern, und nicht zum unmittelbaren Bestandteil deines Trees zu machen. Dazu solltest du das DefaultTreeModel über Bord werfen, und statt dessen ein eigenes TreeModel zu implementieren (extends DefaultTreeModel), welches auf deinen Datenbestand zugreift und dieses lediglich auf den Tree mappt. Dann hast du in deiner GUI-Klasse auch nur noch eine Variable für den tree, und nicht (redundanterweise) auch noch zusätzliche Referenzen auf das Model und den Root-Knoten. Schau dir mal das Interface TreeModel an.

 

[RELAXccc]

Danke für den Tipp.