Swing Problem beim Neuerstellen einer Node

[Albertus]

Hi

Ich habe heuete etwas mit Java rumgespielt und versucht, eine ein- und ausblendbare Node zu machen. Ich bin mir aber nicht sicher wenn es um Objekte in Java geht, da ich aus dem C Bereich komme (Immer etwas unklar, wann die Referenz wirklich verloren ist und wann nur die Referenz zur Referenz des Objektes gelöscht wird).

Also ich habe einen Button, der eine Node immer ein oder ausblendet. Blendet man sie ein und aus, kann man die Nodes aber nicht mehr einblenden. Ich bin ehrlich gesagt etwas überfordert, da ich dachte, man könne die Rootnode mit dem Namen root einfach removen und das wars gewesen - was aber anscheinend nicht der Fall ist (oder wie gesagt eventuell mein Fehler ist).

Das Ganze hab ich verkürzt, der Gedanke ist der, als Parameter ein JPanel mitzugeben, welches dann in einer Subklasse bearbeitet wird.

import java.awt.BorderLayout;
import java.awt.Dimension;
import java.awt.event.ActionEvent;
import java.awt.event.ActionListener;

import javax.swing.JButton;
import javax.swing.JFrame;
import javax.swing.JPanel;


public class GUI extends JFrame implements ActionListener
{
	public static void main(String[] args)
	{
		new GUI();
	}

	private JPanel panel = new JPanel();
	private JButton button = new JButton("Reload");
	private int val = 0;
	
	// Nodeclass
	private NodeSwitcher NodeHandler = new NodeSwitcher();
	
	public GUI()
	{
		// Setzen eines Layouts
		setLayout(new BorderLayout());
		add(panel, BorderLayout.CENTER);
		add(button, BorderLayout.NORTH);
		button.addActionListener(this);
		setMinimumSize(new Dimension(150, 150));
		setVisible(true);
	}
	@Override
	public void actionPerformed(ActionEvent e)
	{
		if(e.getSource() == button)
		{
			if(val == 0) // Erstellen
			{
				NodeHandler.createNodes(panel);
				val = 1;
			}
			else // Löschen
			{
				NodeHandler.destroyNodes(panel);
				val = 0;
			}
		}
	}
}

import java.awt.BorderLayout;

import javax.swing.JPanel;
import javax.swing.JScrollPane;
import javax.swing.JTree;
import javax.swing.tree.DefaultMutableTreeNode;
import javax.swing.tree.DefaultTreeModel;

public class NodeSwitcher
{
	
	private DefaultMutableTreeNode root;
	private JTree guiTree;
	
	public void createNodes(JPanel v_panel)
	{
		//  Rootnode mit 5 Childs erstellen
		System.out.println("Create");
		root = new DefaultMutableTreeNode("Rootnode");
		guiTree = new JTree(root);
		for(int i = 0; i < 5; i++)
		{
			root.add(new DefaultMutableTreeNode(i));
		}
		v_panel.setLayout(new BorderLayout());
		JScrollPane scrollpane = new JScrollPane();
		scrollpane.getViewport().add(guiTree);
		v_panel.add(BorderLayout.CENTER, scrollpane);
		v_panel.setVisible(true);
	}
	public void destroyNodes(JPanel v_panel)
	{
		// Aktuelles Rootnode samt Childs darunter löschen, um es beim erneuten Drücken des Buttons wiederherzustellen
		System.out.println("Destroy");
		DefaultTreeModel model = (DefaultTreeModel)guiTree.getModel();
		model.setRoot(null);
	}
}

Sieht jemand einen Fehler ? Entfernen kann er sie, aber nicht mehr hinzufügen (Oder sie sind irgendwie nicht sichtbar)

Gruss

 

[bERt0r]

Was willst du denn genau machen. Dein createNodes Code erstellt einen JTree, deine destroyNodes Funktion setzt die Root auf null.
Wenn du nur die Nodes löschen/neu erstellen willst, brauchst du nicht jedes mal einen neuen JTree anlegen. Da reichts wenn du dir deine Nodes zusammenbaust und dann setRoot - so wie im destroyNodes - mit der neuen root Node aufrufst.

 

[Albertus]

Ich möchte in den 5 Childs Sachen speichern. Wird ein Program gestartet kann man Notizen laden (Die Informationen landen dann in den 5 Childs), beim Schliessen der Notiz werden die Childs wieder geleert..beim Laden dann wieder befüllt und so weiter.

 

[bERt0r]

Eben, dafür brauchst du nicht jedes mal einen neuen JTree erzeugen. Nichtmal eine eigene Klasse Nodeswitcher brauchst du dafür, du machst das viel zu kompliziert:

import java.awt.BorderLayout;
import java.awt.EventQueue;
import java.awt.event.ActionEvent;
import java.awt.event.ActionListener;

import javax.swing.JFrame;
import javax.swing.JPanel;
import javax.swing.JScrollPane;
import javax.swing.JToggleButton;
import javax.swing.JTree;
import javax.swing.border.EmptyBorder;
import javax.swing.tree.DefaultMutableTreeNode;
import javax.swing.tree.DefaultTreeModel;

public class NodeDeleteDemo extends JFrame
{
	
	private JPanel contentPane;
	private JScrollPane scrollPane;
	private JTree tree;
	private JToggleButton tglbtnIWantNodes;
	private DefaultTreeModel model;
	
	/**
	 * Launch the application.
	 */
	public static void main(String[] args)
	{
		EventQueue.invokeLater(new Runnable()
			{
				public void run()
				{
					try
					{
						NodeDeleteDemo frame = new NodeDeleteDemo();
						frame.setVisible(true);
					} catch (Exception e)
					{
						e.printStackTrace();
					}
				}
			});
	}
	
	/**
	 * Create the frame.
	 */
	public NodeDeleteDemo()
	{
		setDefaultCloseOperation(JFrame.EXIT_ON_CLOSE);
		setBounds(100, 100, 450, 300);
		contentPane = new JPanel();
		contentPane.setBorder(new EmptyBorder(5, 5, 5, 5));
		contentPane.setLayout(new BorderLayout(0, 0));
		setContentPane(contentPane);
		
		scrollPane = new JScrollPane();
		contentPane.add(scrollPane, BorderLayout.CENTER);
		
		model = new DefaultTreeModel(null);
		tree = new JTree(model);
		scrollPane.setViewportView(tree);
		
		tglbtnIWantNodes = new JToggleButton("I want Nodes!");
		tglbtnIWantNodes.addActionListener(new ActionListener()
			{
				public void actionPerformed(ActionEvent e)
				{
					if (tglbtnIWantNodes.isSelected())
					{
						DefaultMutableTreeNode root = new DefaultMutableTreeNode("Root");
						for (int i = 0; i < 5; i++)
						{
							root.add(new DefaultMutableTreeNode(i));
						}
						model.setRoot(root);
					}
					else
					{
						model.setRoot(null);
					}
				}
			});
		contentPane.add(tglbtnIWantNodes, BorderLayout.SOUTH);
	}
	
}

 

[Albertus]

Okay, vielen Dank. Jetzt ist mir auch klar, warum ich immer mehr und mehr Tree's auf dem Layout erschienen. Ich hab das Ganze darum in eine Klasse ausgelagert, da ich noch zusätzliche Methoden hinzufügen werden (Es war schlichtswegs eine kleine, "lauffähige" Version zum zeigen des Problemes).