Laufwerkinhalt in ArrayList schreiben

[sdt105]

Hi, bitte schreibt mir mal ein komplettes Programm ... is natürlich nur Spaß. Ich hab folgendes Problem und auch noch keine Lösung im Forum oder per Google gefunden:

Ich möchte die Pfadnamen aller Dateien eines Laufwerks in eine ArrayList schreiben... Das hab ich jetzt soweit auch hinbekommen... Allerdings kann ich nur Verzeichnisse durchsuchen. Bei Laufwerksangaben bekomme ich eine NullPointerException. Ich weiß auch, dass das daran liegt, dass ich mit dem Programm nur überprüfe, ob das aktuelle Objekt ein Verzeichnis oder eine Datei ist und das dann je nach dem in den Stack oder in die ArrayList kommt.

Ich weiß nicht, wie ich das machen soll, wenn mein Startverzeichnis ein Laufwerk ist...

public static List<File> find( String start ) 
	{ 
		List<File> files = new ArrayList<File>( 65536 ); 
		Stack<File> dirs = new Stack<File>(); 
		File startdir = new File( start ); 
				
		if ( startdir.isDirectory() ) dirs.push( startdir ); 
		while ( dirs.size() > 0 ) 
			for ( File file : dirs.pop().listFiles() ) 
				if ( file.isDirectory() ) 
				{
				 dirs.push( file );  
				}
				else
				{
				 files.add( file );
				}
		
		return files; 
	}

 

[The_S]

Also ich bekomme da keine NullPointerException ???:L

 

[sdt105]

import java.io.*; 
import java.util.*;  

public class Test 
{ 

	public static List<File> find( String start ) 
	{ 
		List<File> files = new ArrayList<File>( 65536 ); 
		Stack<File> dirs = new Stack<File>(); 
		File startdir = new File( start ); 
			
		if ( startdir.isDirectory() ) dirs.push( startdir ); 
		while ( dirs.size() > 0 ) 
			for ( File file : dirs.pop().listFiles() ) 
				if ( file.isDirectory() ) 
				{
				 dirs.push( file );  
				}
				else
				{
				 files.add( file );
				}
		
		return files; 
	} 
	
	
	
	public static void main(String[] args) throws IOException
	{
	    
		String path = "D:/";
		System.out.println( "Looking in path: " + path ); 
		List<File> files = Test.find( path );

                                for ( File f : files ) 
		{
	
			System.out.println( f.getAbsolutePath());
			
		
		}				
		
	}
	
}

das ist das komplette Prog...

 

[The_S]

Bau noch ne Überprüfung ein, ob es sich bei deinem File im Stack wirklich um ein Directory handelt. Ansonsten liefert listFiles nämlich null zurück (daher die Exception). Das hat aber rein garnichts mit Laufwerksbuchstaben zu tun.

 

[SlaterB]

naja, wenns kein Verzeichnis ist, dann kommst auch gar nicht in den Stack, die Überprüfung gibts schon

besondere Verzeichnisse ohne Zugriff sind aber ein Problem,
bei mir war bei C:\Windows\Temp Schluss

 

[sdt105]

@ slater: und wie hast du das Laufwerk-Problem gelöst?

 

[sdt105]

@ slater: und wie hast du das Laufwerk-Problem gelöst?

 

[The_S]

naja, wenns kein Verzeichnis ist, dann kommst auch gar nicht in den Stack, die Überprüfung gibts schon

Wo???

@sdt105

also ich habs mit der Abfrage gelöst, aber das will ja niemand wissen

 

[sdt105]

naja, wenns kein Verzeichnis ist, dann kommst auch gar nicht in den Stack, die Überprüfung gibts schon

Wo???

@sdt105

also ich habs mit der Abfrage gelöst, aber das will ja niemand wissen

die klappt bei mir aber leider nicht ... dachte eigentlich auch, dass die Directory-Überprüfung im Stack schon automatisch mit

if ( startdir.isDirectory() ) dirs.push( startdir );

gelöst ist...

 

[SlaterB]

> Wo???


->

> if ( startdir.isDirectory() ) dirs.push( startdir );
+
> if ( file.isDirectory() )
> {
> dirs.push( file );

----------

ein Laufwerkproblem habe ich nicht,
nur kein D-Laufwerk (DVD), deshalb C getestet

 

[The_S]

[edit] Schwachsinn, mein Fehler ich nehm alles zurück und behaupt es gegenteil Ich bin aber dennoch irritiert und werde wohl morgen nochmal drüber nachdenken, wenn ihr bis dahin keine Lösung habt

[edit2] Kann es sein, dass du auf einen Ordner zugreifst, auf welchen du keine Berechtigung hast!? Bzw. bi mir ziemlich sicher, dass es daran liegt. Also evtl. noch ne Überprüfung einbauen, ob der Rückgabewert von f.listFiles != null ist.

 

[sdt105]

so Freunde... ich hab dann doch ne Lösung gefunden... wär nett, wenn ihr euch das mal anschaut und mir vielleicht irgendwelche Tipps geben könnt, was man noch verbessern könnte...

edit: abgesehen von der Benennung der Objekte...

public static List<File> find( String start ) 
   { 
	  File filealt = new File(start);
	  File[] filelist = filealt.listFiles();
	  
      List<File> files = new ArrayList<File>( 65536 ); 
      Stack<File> dirs = new Stack<File>(); 
      
      for (File file3 : filelist) 
		{
          
    	  if ( file3.isDirectory() ) 
    		  {
    		  dirs.push( file3 );
    		  }

    	  else 
    	  		{
    		  	files.add( file3 );
    	  		}
          try
          {
    	   while ( dirs.size() > 0 ) 
    		  for ( File file : dirs.pop().listFiles() ) 
    			  if ( file.isDirectory() ) 
    			  { 
    				  dirs.push( file );  
    			  } 
    			  else 
    			  { 
    				  files.add( file ); 
    			  } 
          }
          
          catch (Exception e)
          {
        	  System.out.println(file3.getName());
          }
         
		}
      return files; 
   }

 

[SlaterB]

du brauchst keine Exeption, du musst nur prüfen, ob listFiles() null zurückgibt oder nicht, da reicht ein einfaches if,

vorallem ist bei einer Exception deine gesamte Bearbeitung beendet,
auch andere 'heile' Verzeichnisse werden nicht mehr verarbeitet
-> ließe sich vermeiden, wenn das try/ catch IN die Schleife käme,
if/ else ist aber besser

 

[sdt105]

also das mit dem if/else bekomm ich net hin. Da hab ich jedesmal wieder ne Exception... meinst du try/catch ist langsamer/verbraucht viel mehr Ressourcen?

public static List<File> find( String start ) 
	{ 
		File filealt = new File(start);
		File[] filelist = filealt.listFiles();
	    
		List<File> files = new ArrayList<File>( 65536 ); 
		Stack<File> dirs = new Stack<File>(); 
      
		for (File file3 : filelist) 
		{
			
			if ( file3.isDirectory() ) 
			{
				dirs.push( file3 );
			}
			else 
			{
				files.add( file3 );
			}
    	  
			while ( dirs.size() > 0 ) 
    		 
			try
			{
				
				for ( File file : dirs.pop().listFiles() ) 
					if ( file.isDirectory() ) 
					{ 
						dirs.push( file );  
					} 
					else 
					{ 
						files.add( file ); 
					} 

			}
    	      
			catch (Exception e)
			{
				System.out.println(file3.getName());
			}

		}
		return files; 
	}

 

[SlaterB]

das mit dem langsamer kommt zwar hin, ist aber praktisch bedeutungslos,

wichtiger ist, dass es im Code katastrophal aussieht,
und die Übersicht zerstört (man weiß ja nicht, welche NullPointerException-Möglichkeit du damit meinst)

außerdem fängst du damit auch weitere NullPointerExceptions ab, was vielleicht nicht gewünscht ist

-----------

du hast immer noch
> for ( File file : dirs.pop().listFiles() )

was sauberer programmiert

File[] x = dirs.pop().listFiles();
for ( File file : x)

entspricht,
wenn x null ist dann wird die for-Schleife zu einer Exception führen,
es ist doch nun kinderleicht, auf null zu püfen:
if (x==null) {
// nix zu tun mit diesem Verzeichnis
} else {
// Verzeichnis mit for-Schleife durchlaufen
}

----------

noch sauberer könnte man

File[] x = dirs.pop().listFiles();
in
File y = dirs.pop();
File[] x = y.listFiles();
zerlegen und etwa y auf null prüfen,
dass kann hier aber nicht passieren, insofern ist die Abkürzung halbwegs zu akzeptieren

 

[sdt105]

public static List<File> readDrive( String start ) 
	{ 
		
		File startpath = new File(start);
		File[] filelist = startpath.listFiles();
	    
		List<File> files = new ArrayList<File>( 65536 ); 
		Stack<File> dirs = new Stack<File>(); 
      
		for (File file : filelist) 
		{
			
			if ( file.isDirectory() ) 
			{
				dirs.push( file );
			}
			else 
			{
				files.add( file );
			}
    	  
			File[] x = dirs.pop().listFiles(); 
			
			while ( dirs.size() > 0 ) 
			
			if ( x == null )
			{
				
				System.out.println(file.getName());

			}
    	      
			else
			{
				for ( File file2 : x ) 
					if ( file2.isDirectory() ) 
					{ 
						dirs.push( file2 );  
					} 
					else 
					{ 
						files.add( file2 ); 
					} 
				
			}
			
		}
		return files; 
	}

wenn ich dann z.B. nen Ordner durchsuche hab ich immer noch ne NullPointerException und bei nem Laufwerk bekomme ich keine Dateien als Output...

 

[SlaterB]

nun, das ganze war natürlich auf meinen Test begründet, bei dem genau bei x die NullPointerException kam,
wenn du noch an anderer Stelle Exceptions hast, dann muss du die natürlich ähnlich bearbeiten,
als erstes wäre dann die Aufgabe, die Zeilennummer der Exception herauszufinden,

---------
> File[] filelist = startpath.listFiles();
> List<File> files = new ArrayList<File>( 65536 );
> Stack<File> dirs = new Stack<File>();
> for (File file : filelist)
> {

hier ist filelist übrigens sowas wie x, aber du prüfst nicht auf null..

--------

wieso hast du eigentlich seit einiger Zeit den Code verdoppelt?
kehre wieder zu der ersten Version von gestern vormittag zurück,
das StartFile in die Liste zu pushen war eine gute Idee,
jetzt machts du alles doppeltgemoppelt

 

[sdt105]

also ich wüsste nicht, was ich da noch großartig kürzer machen kann...
ich hab das doch extra so eingebaut, dass auch ein Laufwerk durchsucht werden kann. Das ging mit der alten Methode nicht...

und sorry, aber ich versteh dich ncith so ganz... die Exceptions an der anderen Stelle kommen erst durch die if-Abfrage...
hab das Programm auf 2 Laufwerken ausprobiert, die voll gespickt sind mit Dateien. Es sind auch Ordner dabei, auf die kein Zugriff möglich ist. Und das Programm funzt 1a...
Ist es wirklich so schlimm da try{}/catch{} reinzubauen?

 

[SlaterB]

> ich hab das doch extra so eingebaut, dass auch ein Laufwerk durchsucht werden kann. Das ging mit der alten Methode nicht...

wenn du das Startfile in den Stack steckst, dann wird es als erstes Pop rausgeholt und dessen listFiles() durchlaufen,

wenn das nicht ginge, dann hätte die alte Methode ja gar nix gemacht?

> und sorry, aber ich versteh dich ncith so ganz...

da du wirklich wenig verstehts bezw freundlicher ich wahrscheinlich nicht genau genug erkläre,
werde ich nicht versuchen irgendwas nochmal zu erklären,
falls du noch eine Frage hast, dann stelle sie

> die Exceptions an der anderen Stelle kommen erst durch die if-Abfrage...

ich kann mir das nicht vorstellen und weiß ja auch nicht was genau du meinst,
falls du das noch im Detail besprechen willst, musst du konkreter werden,
(z.B. immer noch: wo genau tritt eine Exception auf)

> Ist es wirklich so schlimm da try{}/catch{} reinzubauen?

aus meiner Sicht auf jeden Fall, wenn du allerdings im if/ else Probleme hast und meine Tipps zu anderen Fehlern führen,
dann ist try{}/catch{} in der Tat die bessere Wahl

 

[sdt105]

ich war eigentlich der Meinung, dass du das Programm nebenbei bei dir testest... habs doch wieder kürzer bekommen...
aber die Exception lass ich mal lieber. Funnktioniert ja ziemlich gut so...

Danke für deine Hilfe!

Zum Abschluss nochmal der Code, für alle, die das mal brauchen sollten:

public static List<File> find( String start ) 
{ 
	List<File> files = new ArrayList<File>( 65536 ); 
	Stack<File> dirs = new Stack<File>(); 
	File startdir = new File( start ); 
	             
	if ( startdir.isDirectory() ) dirs.push( startdir ); 
	while ( dirs.size() > 0 ) 
	    	  
		try 
		{
			for ( File file : dirs.pop().listFiles() ) 
				if ( file.isDirectory() ) 
				{ 
					dirs.push( file );  
				} 
				else 
				{ 
					files.add( file ); 
				} 
		}
	      
	catch (Exception e)
	{
	}
	       
return files; 
}

 

[SlaterB]

immer diese Arbeit..


ok, nun habe ich mal die obige Version eingetippt,
Exception bei:

> File[] x = dirs.pop().listFiles();

kein Wunder wenn du das nun VOR
while (dirs.size() > 0)
stehen hast, während es vorher DAHINTER stand,

sorry dass ich dir diese Aufgabe nicht abgenommen haben..

für DVD-Laufwerk D bekomme ich auch die Exception ganz am Anfang, habe ja geschrieben dass das listFiles() dort nicht auf null geprüft wird bzw. bei der kürzeren Variante keine Problem mehr ist,
egal ob beim schönen if(null) oder schrecklichen try/ catch

 

[sdt105]

wenn ich das nach dem while eingebe, dann bekomme ich die Fehlermeldung misplaced constructs für File[]...

 

[SlaterB]

aha, nach einem while kann man in Java also bestimmten Code nicht posten..

deine ... klingen immer nach 'warum gibts du so komische Tipps',
nicht nach 'hier ist mein aktueller Code mit folgender Fehlermeldung, was mache ich falsch'?
da vergeht mir die Lust, sorry

 

[sdt105]

public static List<File> find( String start ) 
	{ 
		
		List<File> files = new ArrayList<File>( 65536 ); 
		Stack<File> dirs = new Stack<File>(); 
		File startdir = new File( start ); 	             
		
		if ( startdir.isDirectory() ) dirs.push( startdir ); 		

		while ( dirs.size() > 0 ) 
		{
			File[] x = dirs.pop().listFiles(); 
			
			if (x != null)
			{
				for ( File file : x ) 
					if ( file.isDirectory() ) 
					{ 
						dirs.push( file );  
					} 
					else 
					{ 
						files.add( file ); 
					} 
			}
			else
			{
			}
		}
		
	return files; 
	}

so zufrieden?
habs jetzt alles hinbekommen und teste das gerade auf allen Laufwerken. Ich wollt dir auf keinen Fall irgendwas
vorwerfen, ich war nur der Meinung, dass der Code diesmal nicht nötig war. Sonst habe ich ihn doch auch immer mitgepostet, oder nicht? der Fehler war übrigens ne vergessene Klammer

jedenfalls danke ich dir nochmal für deine Hilfe. Bin neu hier und muss mich erstma an alles gewöhnen. Übrigens siehts mit Hansa zur Zeit ziemlich schlecht aus (wenn du Hansa Rostock meinst)... Bin zwar sonst Bayer04-Fan, aber die Ost-Vereine liegen mir doch alle sehr am Herzen...

 

[SlaterB]

ach, Fussball is eh überbezahltes Merchandising,
da können sie auch ruhig mal Meister der zweiten oder dritten Liga werden