JTree, Selection via event

[Dragonfire]

Hihu,
ich hab ein Problem.
Hab eine JTree gebastelt,
möchte folgende Funktion implementieren:

Sobald man einen Knoten anklickt,
sollen alle Blätter darunter ausgewählt (selektiert) werden.

Notiz:
- Den cast-operator wusste ich nicht zu vermeiden.
- Die Variable t ist mein JTree
- Ich entferne die Markierung zum Schluss,
damit ich den Knoten erneut anklicken kann.


Mein Ansatz:

t.addTreeSelectionListener(new TreeSelectionListener()
        {
            public void valueChanged(TreeSelectionEvent event) 
            {
                if(event.getNewLeadSelectionPath().getLastPathComponent() instanceof DefaultMutableTreeNode)
                {
                    TreePath tmpP = event.getNewLeadSelectionPath();
                    DefaultMutableTreeNode tmpN = (DefaultMutableTreeNode) tmpP.getLastPathComponent();
                    
                    //wenn es ein Knoten ist
                    if(!tmpN.isLeaf())
                    {
                           //und gerade markiert wurde
                           boolean select = t.getSelectionModel().isPathSelected(tmpP);
                           if(select)
                           {
                               //markiere Rest
                               selectTree(tmpN); 
                               t.removeSelectionPath(tmpP);                               
                           }
                    }
                }
            }
        });

die Methode selectTree(tmpN) soll den Rest des Baums selektieren:

private void selectTree(DefaultMutableTreeNode node)
    {
        DefaultMutableTreeNode tmpNode = null;
        //gehe alle Kinder durch
        for(int i = 0; i < node.getChildCount(); i++)
        {
            tmpNode = (DefaultMutableTreeNode) node.getChildAt(i);
            System.out.println(tmpNode);
            //auswählen
            this.t.addSelectionPath( new TreePath(tmpNode.getPath()) );
        } 
    }

Wenn ein Knoten wieder selektiert würde,
dann soll erneut der Event-Listener anschlagen.

Manchmal geht es,
ein anderer mal kommt dann:

Exception in thread "AWT-EventQueue-0" java.lang.StackOverflowError
	at sun.nio.cs.SingleByteEncoder.encodeArrayLoop(SingleByteEncoder.java:91)
	at sun.nio.cs.SingleByteEncoder.encodeLoop(SingleByteEncoder.java:130)
	at java.nio.charset.CharsetEncoder.encode(CharsetEncoder.java:544)
	at sun.nio.cs.StreamEncoder.implWrite(StreamEncoder.java:252)
	at sun.nio.cs.StreamEncoder.write(StreamEncoder.java:106)
	at java.io.OutputStreamWriter.write(OutputStreamWriter.java:190)
	at java.io.BufferedWriter.flushBuffer(BufferedWriter.java:111)
	at java.io.PrintStream.write(PrintStream.java:476)
	at java.io.PrintStream.print(PrintStream.java:619)
	at java.io.PrintStream.println(PrintStream.java:773)
	at SelectPanel.selectTree(SelectPanel.java:179)
	at SelectPanel.access$100(SelectPanel.java:20)
	at SelectPanel$1.valueChanged(SelectPanel.java:89)
	at javax.swing.JTree.fireValueChanged(JTree.java:2823)
	at javax.swing.JTree$TreeSelectionRedirector.valueChanged(JTree.java:3194)
	at javax.swing.tree.DefaultTreeSelectionModel.fireValueChanged(DefaultTreeSelectionModel.java:629)
	at javax.swing.tree.DefaultTreeSelectionModel.notifyPathChange(DefaultTreeSelectionModel.java:1078)
	at javax.swing.tree.DefaultTreeSelectionModel.addSelectionPaths(DefaultTreeSelectionModel.java:406)
	at javax.swing.tree.DefaultTreeSelectionModel.addSelectionPath(DefaultTreeSelectionModel.java:303)
	at javax.swing.JTree.addSelectionPath(JTree.java:1701)
	at SelectPanel.selectTree(SelectPanel.java:181)
	at SelectPanel.access$100(SelectPanel.java:20)
	at SelectPanel$1.valueChanged(SelectPanel.java:89)
	at javax.swing.JTree.fireValueChanged(JTree.java:2823)
	at javax.swing.JTree$TreeSelectionRedirector.valueChanged(JTree.java:3194)
	at javax.swing.tree.DefaultTreeSelectionModel.fireValueChanged(DefaultTreeSelectionModel.java:629)
	at javax.swing.tree.DefaultTreeSelectionModel.notifyPathChange(DefaultTreeSelectionModel.java:1078)
	at javax.swing.tree.DefaultTreeSelectionModel.removeSelectionPaths(DefaultTreeSelectionModel.java:497)
	at javax.swing.tree.DefaultTreeSelectionModel.removeSelectionPath(DefaultTreeSelectionModel.java:427)
	at javax.swing.JTree.removeSelectionPath(JTree.java:2510)
	at SelectPanel$1.valueChanged(SelectPanel.java:90)
	at javax.swing.JTree.fireValueChanged(JTree.java:2823)
	at javax.swing.JTree$TreeSelectionRedirector.valueChanged(JTree.java:3194)
	at javax.swing.tree.DefaultTreeSelectionModel.fireValueChanged(DefaultTreeSelectionModel.java:629)
	at javax.swing.tree.DefaultTreeSelectionModel.notifyPathChange(DefaultTreeSelectionModel.java:1078)
	at javax.swing.tree.DefaultTreeSelectionModel.addSelectionPaths(DefaultTreeSelectionModel.java:406)
	at javax.swing.tree.DefaultTreeSelectionModel.addSelectionPath(DefaultTreeSelectionModel.java:303)
	at javax.swing.JTree.addSelectionPath(JTree.java:1701)
	at SelectPanel.selectTree(SelectPanel.java:181)
	at SelectPanel.access$100(SelectPanel.java:20)
	at SelectPanel$1.valueChanged(SelectPanel.java:89)
	at javax.swing.JTree.fireValueChanged(JTree.java:2823)	
        [...]

GEKÜRZTE

Warum kommt gerade ein "StackOverflow" ?

Ich bin am Verzweifeln,
gibt es vielleicht einen besseren Ansatz?

 

[klein-odd]

in dem Listing gibt es viele Informaionen,
die mit Deinem Programm wenig direkt zu tun haben.

Vielleicht ist es besser, dass Du in der ganzen Fehlermeldung die Zeile findest,
die eine Zeile im Code Deines Programms direkt anspricht.

In der Zeile ist die Klasse genannt, in der der Fehler aufgetreten ist.

Welche Zeile ist das ?

 

[Ebenius]

Das ist eigentlich ganz einfach:

1. Du hörst auf SelectionEvents
2. Du selektierst als Reaktion auf Selektionsänderung
3. Du erhältst wieder einen Event, weil Du selbst die Selektion geändert hast
4. goto #2

Je nachdem wie viele Kinder der Knoten hat, läuft natürlich der Stack über. Du könntest Deinen Code dahingehend ändern:

private handlingSelectionEvent = false;
private void selectTree(DefaultMutableTreeNode node) {
  if(handlingSelectionEvent) {
    /* ignore selection change caused by this method */
    return;
  }

  try {
    handlingSelectionEvent = true;
    DefaultMutableTreeNode tmpNode = null;
    //gehe alle Kinder durch
    for(int i = 0; i < node.getChildCount(); i++) {
      tmpNode = (DefaultMutableTreeNode) node.getChildAt(i);
      System.out.println(tmpNode);
      //auswählen
      this.t.addSelectionPath( new TreePath(tmpNode.getPath()) );
    }
  } finally {
    handlingSelectionEvent = false;
  }
}

Ebenius

 

[Dragonfire]

DANKE,
es geht !!!

hier der Code:

private void selectTree(DefaultMutableTreeNode node) 
    {
        if(handlingSelectionEvent) 
        {
            // ignore selection change caused by this method
            return;
        }

        try 
        {
            handlingSelectionEvent = true;
            DefaultMutableTreeNode tmpNode = null;
            //gehe alle Kinder durch
            for(int i = 0; i < node.getChildCount(); i++) 
            {
                tmpNode = (DefaultMutableTreeNode) node.getChildAt(i);
                System.out.println(tmpNode);
                //auswählen
                this.t.addSelectionPath( new TreePath(tmpNode.getPath()) );
                //wenn Knoten weiter untersuchen
                if(!tmpNode.isLeaf())
                {
                    handlingSelectionEvent = false;
                    selectTree(tmpNode);
                }
            }
        }
        finally 
        {
            handlingSelectionEvent = false;
        }
    }

Was genau bewirkt das return?

 

[Ebenius]

Was genau bewirkt das return?

Ich dachte, das hätte ich oben schon gut beschrieben.

Dein Problem war, dass Du in einem Event-Handler der auf SelectionEvents reagiert die Selektion geändert hast,
· was wieder SelectionEvents erzeugt hat,
· · die wieder dazu führten, dass Du die Selektion ändertest,
· · · was wieder SelectionEvents erzeugt hat,
· · · · die wieder dazu führten, dass Du die Selektion ändertest,
· · · · · was wieder SelectionEvents erzeugt hat,
· · · · · · die wieder dazu führten, dass Du die Selektion ändertest,
· · · · · · · was wieder SelectionEvents erzeugt hat,
· · · · · · · · die wieder dazu führten, dass Du die Selektion ändertest,
· · · · · · · · · was wieder SelectionEvents erzeugt hat,
· · · · · · · · · · die wieder dazu führten, dass Du die Selektion ändertest,
· · · · · · · · · · · was wieder SelectionEvents erzeugt hat,
· · · · · · · · · · · · die wieder dazu führten, dass Du die Selektion ändertest,
· · · · · · · · · · · · · was wieder SelectionEvents erzeugt hat,
· · · · · · · · · · · · · · ...
· · · · · · · · · · · · · · · · · · · · · · · · · · · · → Stack Overflow

Der if-Block mit return verlässt nun die Methode, wenn die Ursache der Selektionsänderung aus der selben Methode kommt, bevor es wieder eine Änderung gibt.

Um Rekursion zu verstehen muss man Rekursion verstanden haben! ;-)

Ebenius

 

[Dragonfire]

Rekursion war noch nie mein Lieblingsthema ...
Bin schon am Acht-Damen Problem (mit Backtracking) gescheitert ...

/**
komisch ist nur,
das es ohne return nicht geht ...
also so ...
sonst hätte ich das ja schon oben im event abgefangen :-(
*/

Geht jetzt auch ohne return:

if(!handlingSelectionEvent) 
{

        try 
        {
            handlingSelectionEvent = true;
            DefaultMutableTreeNode tmpNode = null;
            //gehe alle Kinder durch
            for(int i = 0; i < node.getChildCount(); i++) 
            {
                tmpNode = (DefaultMutableTreeNode) node.getChildAt(i);
                System.out.println("'" + tmpNode + "' selected.");
                //auswählen
                this.t.addSelectionPath( new TreePath(tmpNode.getPath()) );
                //wenn Knoten weiter untersuchen
                if(!tmpNode.isLeaf())
                {
                    handlingSelectionEvent = false;
                    selectTree(tmpNode);
                }
            }
        }
        finally 
        {
            handlingSelectionEvent = false;
        }
}

update

 

[Ebenius]

Na Du Nasenbär, willst Du nicht ein klitzekleines Ausrufezeichen in Zeile 2 Spalte 4 einfügen, hm? ;-)

 

[Dragonfire]

Das kommt davon wenn man das nebenbei macht ..
beim erneuten durchlesen ist mir das nochmal aufgefallen ^^
Warum Nasenbär?

break, oder return mitten in der Method mag ich nicht so gerne ..
Kann man irgendwie den cast-operator im TreeSelectionListener vermeiden?