Swing [JTree] Selektierte Nodes nummerieren

[lemming]

hallo zusammen,
hab schon viel rumgesucht und auch schon einige sachen ausprobiert. jetzt wollt ich euch mal fragen ob ihr vielleicht auf anhieb wüsstet, wie man folgendes realisieren könnte:

mein jtree hat zum beispiel 3 nodes in form von jpanels+jlabels mit einer seiten-vorschau. selektierte seiten (nodes) erhalten einen farbigen rahmen in form eines TitledBorders mit einer zahl, welche die reihenfolge der selektion anzeigt:

TreeCellRenderer

			// Show a titled border - count selection and display it on each selected.
			if (XYTreeCellRenderer.this.selected) {
				panel.setBorder(BorderFactory.createTitledBorder(highlightBorder, String.valueOf(treeHandle.getSelectionCount()), TitledBorder.RIGHT, TitledBorder.DEFAULT_POSITION, new Font("Arial", Font.BOLD, 16), Color.BLACK));
			} else {
				panel.setBorder(BorderFactory.createLineBorder(Color.BLACK, 1));
			}

das mit dem roten rahmen funktioniert natürlich. probleme hab ich nur mit der nummerierung der reihenfolge bei einer selektion. der obige text wird innerhalb einer paint() methode ausgeführt, welche sich innerhalb des panels befindet, welches für die darstellung der seitenvorschau zuständig ist.

der fehler zeigt sich dadurch, dass wenn ich zum beispiel die ersten drei seiten selektiere, die erste eine "1" bekommt - die zweite eine "3" und die erste zur "2" wird. siehe screenshot...

wie würdet ihr ansetzen? würd mich über tipps freuen!

gruß

 

[Michael...]

Entweder des TreeNode Objekt merkt sich selbst die Nummer oder Du hälst eine Liste vor mit der Du Nummer und Objekt mappen kannst.
So wie Du das machst, kann es sein, dass aufgrund irgendeines repaints bei alle selektierten Knoten die selbe Nummer (nämlich die Anzahl der selektierten Knoten) angezeigt wird.

der obige text wird innerhalb einer paint() methode ausgeführt

das hört sich gefährlich an. Du redest hier hoffenlich nich von einer überschriebenen paint einer Komponente?

 

[lemming]

hey michael,
danke für deine antwort. ich hatte das schon implementiert, dass sich jedes treeNode objekt beim selektieren eine variable namens "counter" mit dem wert aus treeHandle.getSelectionCount() merkt.

hat auch nichts geholfen, ich denke aber dass dein hinweis mit der paint() methode in die richtige richtung geht! das ist unschön aber wo wär dann der richtige platz innerhalb eines cellrenderes? zum beispiel um selektierten nodes einen border zu geben? wichtig hierbei, wo überhaupt hatt man zugriff auf die aktuell bzw. einzelnen nodes? in meinem fall werden ja immer gleich mehrere auf einmal aktualisiert ohne dass ich das steuern kann.

grüße

 

[Michael...]

aber wo wär dann der richtige platz innerhalb eines cellrenderes? zum beispiel um selektierten nodes einen border zu geben?

Das Setzen der Border sollte innerhalb der getTreeCellRendererComponent(...) erfolgen,
das Erfassen der Selektionsreihenfolge über einen TreeSelectionListener

 

[lemming]

für mich scheint die thematik doch komplizierter zu sein als ich anfangs angenommen hatte. selbst wenn ich im TreeSelectionListener die zahl in form einer variable abspeichere und in der getTreeCellRendererComponent(...) versuche zu setzen, gehts nicht. sobald nodes wieder deselektiert werden, läuft es aus dem ruder...

danke soweit mal, werd das vorerst als nice-to-have feature zurückstellen. sollte ich noch ne lösung finden, werd ich das natürlich nachreichen.

 

[Michael...]

Nicht gerade schön, aber es zeigt wie man da möglicher Weise vorgehen kann.
Hatte erst gedacht, man könnte das TreeSelectionModel nutzen, aber das Model scheint beim deselektieren einzelner Knoten die Reihenfolge zu ändern.
Hab's jetzt mal so gelöst, dass ich die selektierten Knoten entsprechend ihrer Selektionsreihenfolgen in eine Liste gesteckt habe, den Listenindex verwende für den Titel der Border.

import java.awt.Color;
import java.awt.Component;
import java.awt.Font;
import java.util.ArrayList;

import javax.swing.BorderFactory;
import javax.swing.JFrame;
import javax.swing.JLabel;
import javax.swing.JScrollPane;
import javax.swing.JTree;
import javax.swing.border.Border;
import javax.swing.border.TitledBorder;
import javax.swing.event.TreeSelectionEvent;
import javax.swing.event.TreeSelectionListener;
import javax.swing.tree.DefaultMutableTreeNode;
import javax.swing.tree.TreeCellRenderer;
import javax.swing.tree.TreePath;

public class TreeSelectionCountDemo extends JFrame {
	
	private JTree tree;
	private ArrayList<DefaultMutableTreeNode> selectionList;
	
	public TreeSelectionCountDemo() {
		selectionList = new ArrayList<DefaultMutableTreeNode>();
		
		tree = new JTree();
		tree.getSelectionModel().addTreeSelectionListener(new TreeSelectionListener() {
			public void valueChanged(TreeSelectionEvent e) {
				TreePath[] paths = e.getPaths();
				for (TreePath path : paths) {
				DefaultMutableTreeNode node = (DefaultMutableTreeNode) path.getLastPathComponent();
					if (node.isLeaf()) {
						if (e.isAddedPath())
							selectionList.add(node);
						else
							selectionList.remove(node);
					}
				}
				actualizeSelectionList();
			}
		});
		
		tree.setCellRenderer(new BorderCellRenderer());
		this.getContentPane().add(new JScrollPane(tree));
	}
	
	private void actualizeSelectionList() {
		TreePath[] paths = tree.getSelectionModel().getSelectionPaths();
		if (paths==null) 
			selectionList.clear();
		else {
			ArrayList<DefaultMutableTreeNode> compareList = new ArrayList<DefaultMutableTreeNode>();
			for(TreePath path : paths) 
				compareList.add((DefaultMutableTreeNode)path.getLastPathComponent());
			
			for (int i=selectionList.size()-1; i>=0; i--) {
				DefaultMutableTreeNode node = selectionList.get(i);
				if (compareList.indexOf(node)<0)
					selectionList.remove(node);
			}
		}
		tree.repaint();
	}
	
	class BorderCellRenderer extends JLabel implements TreeCellRenderer {
		private TitledBorder titledBorder;
		private JLabel dummyNodeLabel, defaultLabel;
		
		public BorderCellRenderer() {
			Border border = BorderFactory.createLineBorder(Color.RED, 2);
			titledBorder = BorderFactory.createTitledBorder(border, "XXX", TitledBorder.RIGHT, TitledBorder.DEFAULT_POSITION, tree.getFont().deriveFont(Font.BOLD), Color.BLACK);
			defaultLabel = new JLabel();
			dummyNodeLabel = new JLabel();
			dummyNodeLabel.setBorder(titledBorder);
		}
		
		public Component getTreeCellRendererComponent(JTree tree, Object value, boolean selected, boolean expanded, boolean leaf, int row, boolean hasFocus) {
			DefaultMutableTreeNode node = (DefaultMutableTreeNode)value;
			String labelText = node.getUserObject().toString();
    		if (!leaf) {
    			defaultLabel.setText(labelText);
    			return defaultLabel;
    		}
    		
    		this.setText(labelText);
    		
    		if (selected) {
    			titledBorder.setTitle(String.valueOf(selectionList.indexOf(node)+1));
    	    	this.setBorder(titledBorder);
    	    	return this;
    		}
    		
    		this.setBorder(null);
    		this.dummyNodeLabel.setText(labelText);
    		this.setPreferredSize(this.dummyNodeLabel.getPreferredSize());
    		
    		return this;
		}
	}
	
	public static void main(String[] args) {
		JFrame frame = new TreeSelectionCountDemo();
		frame.setBounds(0, 0, 500, 300);
		frame.setLocationRelativeTo(null);
		frame.setDefaultCloseOperation(JFrame.EXIT_ON_CLOSE);
		frame.setVisible(true);
	}
}

Das entscheidende ist das was im TreeSelectionListener passiert (Zeile 28 bis 64) bin mir nicht sicher ob es in allen möglichen Konstellationen so sauber läuft.

 

[lemming]

Hallo Michael,
sorry für die sehr verspätete Rückmeldung. Vielen Dank nochmal für die Mühe! Leider machen manche Konstellationen weiterhin Probleme, speziell beim deselektieren und erneuten selektieren. Hab die Anzeige der Zahlen jetzt erstmal begraben..
Aber ich denke für andere die siche evtl. mit einer ähnliche Thematik beschäftigen, ist der Thread trotzdem hilfreich!

Grüße