ein JTree soll einen 2. JTree steuern

[hussi]

Hallo zusammen,

Mein Programm sieht bis jetzt so aus: auf meinem Frame sind 2 Trees nun will ich in dem linken Tree auf einen Knoten klicken (der dann auf geht) und dann soll im rechten Tree der Knoten (auf der selben ebene mit dem selben Namen) ebenfalls aufgehen. In den Tree sind die selben Daten enthalten, z.B.: ich öffne in dem linken einen Ordner und im rechten dann eine Kopie davon.

Mein Versuch war bis jetzt so das ich im linkenTree meinen ExpansionListener setze und von die Methode des rechten Tree verwende, aber im rechten Baum rührt sich nix:

class MyExpansionListener implements TreeExpansionListener{
 
    public void treeCollapsed(TreeExpansionEvent arg0) {
        final MyNode node = (MyNode) arg0.getPath().getLastPathComponent();
        TreePath  treepath= arg0.getPath(); 
        jTreeRechts.collapsePath(treepath);
    } 

    public void treeExpanded(TreeExpansionEvent arg0)
    {
        final MyNode node = (MyNode) arg0.getPath().getLastPathComponent();
        TreePath  treepath= arg0.getPath(); 
        jTreeRechts.expandPath(treepath);
      
    }
    
  }

Kann mir da wer weiter helfen?

Grüße hussi

 

[AlArenal]

Arbeiten beide Trees mit demselben Model, oder nur mit dengleichen Daten?

 

[hussi]

beide haben das selbe Model

      jTreeLinks = new JTree();
      jTreeLinks.setCellRenderer(treeRenderer);
      jTreeLinks.addTreeExpansionListener(new DifferExpansionListener());
      jTreeLinks.setModel(new MyTreeModel(rootLinks));
      jScrollPane2.setViewportView(jTreeLinks);

      jTreeRechts = new JTree();
      jTreeRechts.setCellRenderer(treeRenderer);
      jTreeRechts.setModel(new MyTreeModel(rootRechts));
      jScrollPane1.setViewportView(jTreeRechts);

 

[AlArenal]

Falsch!
Deinem Code nach zu urteilen ist weder das Model gleich (du erzeugst jeweils eine neue Instanz), noch die darin befindlichen Daten (du benutzt ja wohl bereits unterscheidliche Root-Nodes).

Demenetsprechend kann es nicht über einen TreePath funktionieren, weil im Model des rechten Trees ganz andere Instanzen verwendet werden als im linken. Das sollte einem aber wie Schuppen aus den Haaren fallen, wenn man weiß wie ein TreePath arbeitet - und das wiederum sollte man wissen, wenn man schon damit arbeitet

Du kannst bestenfalls versuchen wir von hand einen TreePath-Ersatz zu stricken, der seinen Pfad nicht mit Object-Instanzen, sondern mit Indices bildet. Und in dem Fall heißt es: Selbst ist der Programmierer!

 

[hussi]

Ja, hast recht! Habs auch grad bemerkt das ich in einem Path
die Objekte bekomm ich dachte ich bekomm einfach ne
Stringliste mti den Namen der Nodes.

Das sollte einem aber wie Schuppen aus den Haaren fallen,
wenn man weiß wie ein TreePath arbeitet - und das wiederum sollte
man wissen, wenn man schon damit arbeitet

genau wenn man es weiß!!! ;-)

Aber damit geht es ohne es selber zu programmieren.

 class MyExpansionListener implements TreeExpansionListener{
  
    
    public void treeCollapsed(TreeExpansionEvent arg0) {
        final MyNode node = (MyNode) arg0.getPath().getLastPathComponent();
        jTreeRechts.collapseRow(jTreeLinks.getRowForPath(arg0.getPath()));
      }
  
    public void treeExpanded(TreeExpansionEvent arg0)
    {
      final MyNode node = (MyNode) arg0.getPath().getLastPathComponent();
      jTreeRechts.expandRow(jTreeLinks.getRowForPath(arg0.getPath())); 
    }
    
  }

[/quote]

 

[AlArenal]

Damit gehts aber auch nur, wenn der 2. Tree wirklich eine exakte Kopie ist. Wenn der durch den User geändert werden kann und seine Änderungen nicht an den linken propagiert, isses schon wieder essig.

 

[hussi]

Also das er nur die auf macht die auch gleich sind verhindere ich jetzt hier mit und meine Notes sind sortiert:

public void treeExpanded(TreeExpansionEvent arg0)
    {
      final MyNode nodeLinks = (MyNode) arg0.getPath().getLastPathComponent();
     
      final MyNode nodeRechts = (MyNode)jTreeRechts.getPathForRow(jTreeLinks.getRowForPath(arg0.getPath())).
            getLastPathComponent();
      if(nodeLinks.toString()==nodeRechts.toString()) {
        jTreeRechts.expandRow(jTreeLinks.getRowForPath(arg0.getPath()));
      }
    }

Siehst du noch gravierende Probleme? Bis jetzt bin ich damit ganz zufrienden. [/code]

 

[AlArenal]

Ich kenne den konkreten Anwendungsfall nicht und abgesehen davon, dass dein Code mir in den AUgen weh tut, ist die Hauptsache ja zunächst, dass es überhaupt funktioniert.

 

[hussi]

Is der Code echt so schlimm?

Kannst mir ein kleines Bespiel geben, wie du ihn schreiben würdest.
Will ja nicht das du dir die Augen kaputt machst. Und lernen will ich ja auch was.

 

[AlArenal]

1. Na wenigstens benutzt du fleißig final. Das ist schonmal ein gutes Stilelement, dass man viel zu selten - eigentlich fast nie - zu sehen bekommt.

2. Strings vergleicht man aber nicht mit ==, sondern mit #equals .

3. Bandwurmzeilen mit lauter vielen Punkten sind zu vermeiden.

4. Du könntest dich mal entscheiden, ob du ENTWEDER englische ODER Deutsche Bezeichner und Klassennamen verwendest.

5. Guter Stil ist es auch Parameter als final zu deklarieren. Bei der Gelegenheit kannst du deinen Event auch mal gleich umbenennen. 'e' oder 'evt' oder 'event' böten sich an.

6. Wenn du jTreeLinks.getRowForPath(e.getPath()) in einer lokalen Variable speicherst, kannst du den Code deutlich aufräumen und musst die Methode nur einmal, statt wie jetzt zweimal, aufrufen.

7. Ist der Vergleich zweier Nodes über #toString wirklich narrensicher? Ich vermute, dass #toString hier den angezeigten Text des Nodes liefert und wenn zwei Sachen den gleichen texte haben, müssen sie nicht automatisch dasgleiche bedeuten...

8. Liefert #getRowForPath -1, weil der Node hidden oder nicht vorhanden ist (in dem Fall könnte es auch ne NPE geben, steht aber nicht in der Doku), dann liefert #getPathForRow(-1) null und damit liefert null#getLastPathComponent() eine NPE ....

 

[AlArenal]

10. "MyNode" ist auch reichlich nichtssagend

 

[AlArenal]

9. vergessen