Ordnerstrucktur rekursiv auslesen

[Tobi88]

Moin zusammen,

ich möchte gerne alle Ordner rekursiv lesen. Dafür habe ich folgenen Code geschrieben

 	public static void listTree( File path )
	{
			for( File file : path.listFiles() )
			{
				if( file.isDirectory() )
				{
					System.out.println( file );
					listTree( file );
				}
			}
	}

Eclipse gibt mir immer an unterschiedlichen Stellen eine java.lang.NullPointerException aus. Ich weiß beim besten Willen nicht was ich mache soll.

Könnt ihr mir weiter helfen?

Danke im vorraus

 

[SlaterB]

entweder Fehlermeldungen lesen lernen, z.B. dürfte die genaue Zeilennummer drinstehen,

oder selber überall prüfen, z.B.
if (path == null) {
throw new IllegalArgumentException("diese Methode macht nix gutes wenn Argument path=null übergeben wird, böse böse");
}

 

[Guest]

	public static void listTree( File path )
	{

		if (path.listFiles() == null) {
			throw new IllegalArgumentException("diese Methode macht nix gutes wenn Argument path=null übergeben wird, böse böse");
		} else {
			for( File file : path.listFiles() )
			{
				if( file.isDirectory() )
				{
					System.out.println( file );
					listTree( file );
				}
			}
		}
	}

ich habe das jetzt so umgeändert, aber wieso kommt die exception immer an einer anderen stelle?

D:\Markt\cash\10_cash_preisException in thread "main" java.lang.IllegalArgumentException: diese Methode macht nix gutes wenn Argument path=null übergeben wird, böse böse
at ListTree.listTree(ListTree.java:15)
at ListTree.listTree(ListTree.java:23)
at Demo.main(Demo.java:9)
spiegel

Das wirft er mir jetzt aus.
Der Ordner heißt eigendlicht
"D:\Markt\cash"
hier sollte "D:\Markt\cash\10_cash_preisspiegel stehen"
"D:\Markt\cash\10_cash_preisspiegel\2008_cash_preisspiegel"

Wenn ich es erneut starte funktioniert die stelle.

Ausgabe:
D:\Markt\cash
D:\Markt\cash\10_cash_preisspiegel
D:\Markt\cash\10_cash_preisspiegel\2008_cash_preisspiegel

Aber der Fehler ist jetzt hier:
D:\java\Threads und nebenläufige Programmierung\bin
D:\java\Threads Exception in thread "main" java.lang.IllegalArgumentException: diese Methode macht nix gutes wenn Argument path=null übergeben wird, böse böse
at ListTree.listTree(ListTree.java:15)
at ListTree.listTree(ListTree.java:22)
at Demo.main(Demo.java:9)
und nebenläufige Programmierung\src


Die Exeption wird doch nur dann ausgeworfen, wenn die Datei nicht existiert... Aber wie kommt es dazu, dass mir ein falscher pfad geben wird?

 

[SlaterB]

Die Fehlermeldung wird im ErrorStream geschrieben, deine Ausgaben in den normalen Output-Stream,

diese sind nebenläufig und werden beliebig miteinander gemischt +- paar Millisekunden,
aus der Reihenfolge der Ausgabe kannst du also wenig bis nix ablesen,

du kannst natürlich

if (path.listFiles() == null) {
System.out.println( "an dieser Stelle der Fehler" );
throw new IllegalArgumentException("path=null");
}
schreiben, dann dürfte die normale Ausgabe 'an dieser Stelle der Fehler' im Vergleich zu den anderen Ausgaben immer an der gleichen Stelle stehen

 

[tobi88]

ahhh ich glaube ich habe es. danke

es liegt an "D:\System Volume Information"

Ich werde mal weiter fummeln

danke

 

[andre111]

wird daran liegen, gleiches problem hatte ich auch mal

 

[tobi88]

Ich möchte jetzt gerne noch, dass die Dateien, die ich nicht lesen kann, überspringen werden. Der Lauf durch die Ordner soll bis zum Schluss durchlaufen werden. ich habe schon sämliche dinge probiere und gegoogelt, aber nichts gefunden
hoffe, dass ihr mir da noch weiter helfen könnt.

 

[SlaterB]

if (path.listFiles() == null) {
// kann eigentlich nur ein spezielles Verzeichnis sein, fertig an dieser Stelle
return;
}

allerdings solltest du nicht path.listFiles() ständig doppelt aufrufen, speichere das in einer Variablen
und verwende die Variable sowohl für den Test als auch die Schleife danach

 

[tobi88]

funktioniert wunderbar

was macht genau das return?
wird da die methode wieder erneut aufgerufen?

 

[SlaterB]

ein erneuter Aufruf brächte die gleiche Situation,
wieder wäre das Array null, wieder ein return usw..
also wenig hilfreich,

ein return beendet logischerweise die aktuelle Methode,