Darstellungsproblem des eigenen TreeModel bei Veränderungen

[dwn8]

Hallo,

das TreeModel wird richtig dargestellt, wenn der JTree initialisiert wird. Jedoch funktioniert es nicht Blätter zu löschen oder zu adden. Ich schicke ein TreeModelEvent los, jedoch passiert nichts und ich komm dem Fehler einfach nicht auf die Spur. So hier erstmal der Code.

public class AccountTreeModel extends AbstractTreeModel implements Serializable {

	private static final long serialVersionUID = -2517899978776523557L;

	List<Account> accountList;
	
	public AccountTreeModel() {
		accountList = new ArrayList<Account>();
	}
	
	public AccountTreeModel(Map<String, Account> accountData) {
		this();
		setData(accountData);
	}
	
	public void setData(Map<String, Account> accountData) {
		for ( Map.Entry<String, Account> e : accountData.entrySet() ) {
			accountList.add(e.getValue());
		}
	}

	@Override
	public Object getChild(Object parent, int index) {
		return accountList.get(index);
	}

	@Override
	public int getChildCount(Object parent) {
		return accountList.size();
	}

	@Override
	public int getIndexOfChild(Object parent, Object child) {
		int result = -1;
		if (child instanceof Account) {
			Account account = (Account) child;
			for (int i = 0; i < accountList.size(); i++) {
				if (accountList.get(i).equals(account)) {
					result = i;
					break;
				}
			}
		}
		return result;
	}

	@Override
	public Object getRoot() {
		return accountList;
	}

	@Override
	public boolean isLeaf(Object node) {
		return (node instanceof Account);
	}

	@Override
	public void valueForPathChanged(TreePath path, Object newValue) {
	}
	
	public void add(Account account) {
		accountList.add(account);
		
		Account[] accountToAdd = {account};
		TreePath path = new TreePath(getRoot());
		int[] addedAccountIndex = {getIndexOfChild(null,account)};
		System.out.println(getIndexOfChild(null,account));
		TreeModelEvent event = new TreeModelEvent(this, path, addedAccountIndex, accountToAdd);
		fireTreeNodesInserted(event);
	}
	
	public void remove(Account account) {
		Account[] accountToRemove = {account};
		TreePath path = new TreePath(getRoot());
		int[] addedAccountIndex = {getIndexOfChild(null, account)};
		accountList.remove(account);
		
		TreeModelEvent event = new TreeModelEvent(this, path, addedAccountIndex, accountToRemove);
		fireTreeNodesRemoved(event);
	}

}

public abstract class AbstractTreeModel extends TreeModelSupport implements TreeModel {
}

public class TreeModelSupport {
	
   private List<TreeModelListener> list = new ArrayList<TreeModelListener>();

   public void addTreeModelListener( TreeModelListener listener ) {
      if ( listener != null && !list.contains( listener ) ) {
         list.add( listener );
      }
   }

   public void removeTreeModelListener( TreeModelListener listener ) {
      if ( listener != null ) {
         list.remove( listener );
      }
   }

   public void fireTreeNodesChanged( TreeModelEvent e ) {
      Enumeration<TreeModelListener> listeners = Collections.enumeration(list);
      while ( listeners.hasMoreElements() ) {
         TreeModelListener listener = (TreeModelListener)listeners.nextElement();
         listener.treeNodesChanged( e );
      }
   }

   public void fireTreeNodesInserted( TreeModelEvent e ) {
      Enumeration<TreeModelListener> listeners = Collections.enumeration(list);
      while ( listeners.hasMoreElements() ) {
         TreeModelListener listener = (TreeModelListener)listeners.nextElement();
         listener.treeNodesInserted( e );
      }
   }

   public void fireTreeNodesRemoved( TreeModelEvent e ) {
      Enumeration<TreeModelListener> listeners = Collections.enumeration(list);
      while ( listeners.hasMoreElements() ) {
         TreeModelListener listener = (TreeModelListener)listeners.nextElement();
         listener.treeNodesRemoved( e );
      }
   }

   public void fireTreeStructureChanged( TreeModelEvent e ) {
      Enumeration<TreeModelListener> listeners = Collections.enumeration(list);
      while ( listeners.hasMoreElements() ) {
         TreeModelListener listener = (TreeModelListener)listeners.nextElement();
         listener.treeStructureChanged( e );
      }
   }
}

Gruß
dwn8

 

[MiDniGG]

Sorry bin zu faul was zu erklären ^^ Und zu ändern deswegen 1:1 abkopiert vllt hilfts ja

Beispiel am Löschen:

try
			{
				if(tree.getSelectionPath() != null)
				{
					int mDel = JOptionPane.showConfirmDialog(this, "Wollen Sie \"" + selectedNode + "\" wirklich löschen?", "Sind Sie sicher?", JOptionPane.YES_NO_OPTION);
					if(mDel == JOptionPane.YES_OPTION)
					{
						selectedNode = (DefaultMutableTreeNode) tree.getLastSelectedPathComponent();
						selectedNode.removeFromParent();
					}
					else
					{
						
					}
				}
			}
			catch(NullPointerException e)
			{
				e.printStackTrace();
			}

DefaultMutableTreeNode selectedNode = null;
.....
selectedNode = (DefaultMutableTreeNode) tree.getLastSelectedPathComponent();

 

[dwn8]

Ich benutze jedoch ein eigenes TreeModel und meine Nodes sind vom Typ Account.
Das Aktualisieren des JTree funktioniert mittels updateUI(). Soweit ich jedoch verstanden hab, sollte man die Aktualisierung über die Listener machen. Und genau dies funktioniert bei mir aus irgend einem Grund nicht.

 

[dwn8]

OK hab es jetzt damit probiert:

fireTreeStructureChanged(new TreeModelEvent(this, new TreePath(getRoot()), null, null));

Und es hat geklappt.
Die Methoden

fireTreeNodesRemoved(event) 
fireTreeNodesInserted(event)

klappen jedoch noch immer nicht.

 

[André Uhres]

Versuch's mal so:

...
    private DefaultMutableTreeNode root = new DefaultMutableTreeNode("Accounts");
...
    @Override
    public Object getRoot() {
        return root;
    }
...

 

[dwn8]

Unglaublich es klappt. Darauf wäre ich nie gekommen.
Das heißt wohl, dass nicht jedes Objekt als root akzeptiert wird, wenn es um die Aktualisierung geht.

 

[André Uhres]

..Das heißt wohl, dass nicht jedes Objekt als root akzeptiert wird, wenn es um die Aktualisierung geht.

Naja, es müsste ja wohl zumindest ein TreeNode sein, nicht wahr?

 

[dwn8]

Ja das hat was