Filesystem in JTree anzeigen

[Guest]

Hallo,

ich würde gerne das Filesystem in einem Jtree ausgeben lassen. Habe hierfür folgende rekursive Funktion geschrieben, die auch funktioniert, wenn man das Auflisten bei einem Ordner anfangen lässt, der nur wenige Unterordner hat.

Und hier ist auch das Problem: Starte ich beispielsweise bei / , dann kommt das Ding nie zu einem Ende.

Kann man das irgendwie optimieren?

mfg

import java.io.File;
import java.util.ArrayList;
import java.util.List;

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

/**
 * @author lennartkoster
 *
 */
public class FolderTree extends JPanel {
	private static final long serialVersionUID = 3978490439743163001L;

	public FolderTree() {
		
		// Neuer JTree
		
                // Geht
                JTree tree = new JTree(AddNodes(null, new File("/Volumes/Neue HDD/Users/lennartkoster/")));
		
                // Geht nicht
               // JTree tree = new JTree(AddNodes(null, new File("/")));

		// JTree in Scrollpane legen
		JScrollPane scrollpane = new JScrollPane(tree);
		
		// Scrollpane hinzufügen
		add(scrollpane);
		
	}
	
	// Rekursives hinzufügen von Foldern
	private DefaultMutableTreeNode AddNodes(DefaultMutableTreeNode curParent,
	      File dir) {

		String curPath = dir.getPath();
		DefaultMutableTreeNode curDir = new DefaultMutableTreeNode(curPath);

		if (curParent != null) {
			curParent.add(curDir);
		}

		List<String> allElements = new ArrayList<String>();
		String[] tmp = dir.list();

		if (tmp != null && tmp.length > 0) {
			for (String s : tmp) {
				allElements.add(s);
			}
		}

		java.util.Collections.sort(allElements);

		File f;

		for (String s : allElements) {

			String element = s;
			String elemPath = new String();

			if (curPath.equals(".")) {
				elemPath = s;
			} else {
				elemPath = curPath + File.separator + element;
			}

			f = new File(elemPath);

			if (f.isDirectory()) {
				//Rekursiver Aufruf
				AddNodes(curDir, f);
			}

		}
		return curDir;
	}
}

Zum anzeigen, ne popelige GUI dazu:

import javax.swing.JFrame;

/**
 * @author lennartkoster
 *
 */
public class FileSystemGUI extends JFrame {
	
	private static final long serialVersionUID = 6872264293765666266L;
	FolderTree tree = new FolderTree();
	
	public FileSystemGUI() {
		ShowGUI();
	}
	
	public static void main(String[] args) {
		javax.swing.SwingUtilities.invokeLater(new Runnable() {
			public void run() {
				new FileSystemGUI();
			}
		});
	}
	
	private void ShowGUI() {
		add(tree);
		pack();
		setVisible(true);
	}
	
}

Kurze Zusammenfassung: Wie kann das Ding optimiert werden?
Habe ich evtl. einen Denkfehler drin, der eine Endlosschleife verursacht?

mfg
lennart

 

[Guest]

import java.io.File;
import java.util.ArrayList;
import java.util.List;

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

/**
* @author lennartkoster
*
*/
public class FolderTree extends JPanel {
   private static final long serialVersionUID = 3978490439743163001L;

   public FolderTree() {
      
      // Neuer JTree
      
                // Geht
                JTree tree = new JTree(AddNodes(null, new File("D:\\")));
      
                // Geht nicht
               // JTree tree = new JTree(AddNodes(null, new File("/")));

      // JTree in Scrollpane legen
      JScrollPane scrollpane = new JScrollPane(tree);
      
      // Scrollpane hinzufügen
      add(scrollpane);
      
   }
   
   // Rekursives hinzufügen von Foldern
   private DefaultMutableTreeNode AddNodes(DefaultMutableTreeNode curParent,
         File dir) {

      String curPath = dir.getPath();
      DefaultMutableTreeNode curDir = new DefaultMutableTreeNode(curPath);

      if (curParent != null) {
         curParent.add(curDir);
      }

      
      File[] files = dir.listFiles();
      
      if(files != null){
          for(File z : files){
        	  if(z.isDirectory()){
        		  AddNodes(curDir, z);
        	  }
          }  
      } 
      return curDir;
   }
}

 

[Gast]

Danke schön.

Geht allerdings immernoch nicht.. ich vermute mal, dass das an OS X liegt.. evtl. die Syslinks.. mal schauen..

 

[Gast]

Noch kürzer:

private DefaultMutableTreeNode addNodes(DefaultMutableTreeNode curParent, 
        File dir) { 

     String curPath = dir.getPath(); 
     DefaultMutableTreeNode curDir = new DefaultMutableTreeNode(curPath); 

     if (curParent != null) { 
        curParent.add(curDir); 
     } 
     
     for (File f : dir.listFiles()) {
    	 if (f != null && f.isDirectory()) {
    		 addNodes(curDir, f);
    	 }
     }
     
     return curDir; 
  }

Aber danke für "listFiles()".

 

[cheezburgla]

Falls es jemanden interessieren sollte, oder jemand anders per Suche mal auf diesen Thread stoßen sollte:

Hab das Ganze jetzt nicht mehr rekursiv gelöst, sondern füge einer Node nur dann die Unter-ordner hinzu wenn man drauf klickt.

Spart einiges an Zeit, da man ja eh nie das gesamte FS aufeinmal anschauen kann.

/**
 * @Name newFolderTree.java
 * @Date 23.01.2009
 * @Author lennartkoster
 */
package de.cheez.components;

import java.io.File;

import javax.swing.GroupLayout;
import javax.swing.JPanel;
import javax.swing.JScrollPane;
import javax.swing.JTree;
import javax.swing.event.TreeSelectionEvent;
import javax.swing.event.TreeSelectionListener;
import javax.swing.tree.DefaultMutableTreeNode;

/**
 * @author lennartkoster
 * 
 */
public class NewFolderTree extends JPanel {

	private static final long serialVersionUID = -3635401151527581532L;
	private final JTree tree;
	private final JScrollPane scrollPane;
	
	public NewFolderTree() {
		DefaultMutableTreeNode node = new DefaultMutableTreeNode("/");
		addSubDirs(node);

		tree = new JTree(node);

		tree.addTreeSelectionListener(new TreeSelectionListener() {
			@Override
			public void valueChanged(TreeSelectionEvent arg0) {
				DefaultMutableTreeNode selNode = (DefaultMutableTreeNode) arg0
				      .getPath().getLastPathComponent();
				addSubDirs(selNode);
				if (!tree.isExpanded(arg0.getPath()))
					tree.expandPath(arg0.getPath());
			}
		});
		
		scrollPane = new JScrollPane(tree);
		
		GroupLayout layout = new GroupLayout(this);
		setLayout(layout);

		layout.setAutoCreateContainerGaps(true);
		layout.setAutoCreateGaps(true);

		layout.setVerticalGroup(layout.createSequentialGroup()
					.addComponent(scrollPane)
		);
		layout.setHorizontalGroup(layout.createSequentialGroup()
					.addComponent(scrollPane)
		);

	}

	private void addSubDirs(DefaultMutableTreeNode addToThisNode) {

		String curDir = new String(addToThisNode.toString());
		File file = new File(curDir);

		// Overriding isLeaf() to always return false
		class FolderNode extends DefaultMutableTreeNode {
			private static final long serialVersionUID = 1L;

			public FolderNode(String s) {
				super(s);
			}

			@Override
			public boolean isLeaf() {
				return false;
			}
		}

		for (File f : file.listFiles()) {
			if (f != null && !f.getPath().equalsIgnoreCase(curDir)
			      && f.isDirectory() == true) {
				addToThisNode.add(new FolderNode(f.getPath()));
			}
		}

	}

}

Falls jemand kosntruktive Kritik zu dem Ganzen hat, gerne her damit!
Bevors böse wird, aber bedenken, dass ich mit Java vor knapp 6 Wochen angefangen habe und das auch nur nebenher

Deswegen: Gerne her mit konstruktiver Kritik!

lg