Dateien aus Verzeichnis filtern, aber nicht nach Endung

[moerkstar]

Hallo Leute,

ich hab jetzt mehrere Tage das gesamte Internet durchsucht, aber ich glaube Ihr könnt mir am besten helfen.

Mein Problem ist folgendes:
Ich habe diesen Code zum Filtern von Dateien in einem Verzeichnis. Ich möchte jetzt aber nicht anhand der Dateiendung filtern, sondern anhand von Zeichenketten innerhalb des Dateinamens (Also egal ob am Anfang, in der Mitte oder am Ende).
Irgendwie bin ich zu blöd dafür, aber ich weiß, dass Ihr mir helft. Lasst mich nicht im Stich!

import java.io.*; 
import java.util.*; 
import java.util.regex.Pattern; 
 
public class FileFinder 
{ 
  public static void main( String[] args ) 
  { 
    String filter = args[0];
	String path = new File( System.getProperty("user.dir") ).getParent(); 
 
    System.out.println( "Looking in path: " + path ); 
 
    FileFinder ff = new FileFinder(); 
    List<File> files = ff.find( path, (".*"+filter+"*") ); 
 
    System.out.printf( "Found %d file%s.%n", 
                       files.size(), files.size() == 1 ? "" : "s" ); 
    for ( File f : files ) 
      System.out.println( f.getAbsolutePath() ); 
  } 
 
  public List<File> find( String start, String extensionPattern ) 
  { 
    final List<File> files = new ArrayList<File>( 1024 ); 
    final Stack<File> dirs = new Stack<File>(); 
    final File startdir = new File( start ); 
    final Pattern p = Pattern.compile( extensionPattern, Pattern.CASE_INSENSITIVE ); 
 
    if ( startdir.isDirectory() ) 
      dirs.push( startdir ); 
 
    while ( dirs.size() > 0 ) 
    { 
      for ( File file : dirs.pop().listFiles() ) 
      { 
        if ( file.isDirectory() ) 
          dirs.push( file ); 
        else 
          if ( p.matcher(file.getName()).matches() ) 
            files.add( file ); 
      } 
    } 
 
    return files; 
  } 
}

 

[mikachu]

hm, das geht doch am einfachst mit File#listFiles( FileNameFilter );

public List<File> find( String path, String filter )
{
    File searchIn = new File( path );
    return Arrays.asList( searchIn.listFiles( new FileFilter()
    {
        public boolean accept( File file )
        {
            return file.getName().contains( filter ); // weiß nicht, ob getName() reicht
        }
    } ) );
}

 

[moerkstar]

warscheinlich stell ich mich echt doof an, wie bau ich das jetzt bei mir ein?

Sorry wenn es dir zu blöd ist!

 

[mikachu]

hm, ersetz einfach deine methode mit der meinigen.

führ aber vorher mit meiner noch ein paar tests durch, weil ich die nicht getestet habe...

also deine lieber erstmal auskommentieren, bevor du die vollständig aus dem quellcode entfernst

#edit 1:
ich hab das erstmal nicht mit patterns gemacht, weil ja nicht dasteht, wie ein filter beispielsweise aussieht

 

[moerkstar]

Danke nochmal

            return file.getName().contains( filter );

Jetzt gibt er hier noch den Fehler:

Cannot refer to a non-final variable filter inside an inner class defined in a different method

 

[moerkstar]

bezogen auf filter

 

[mikachu]

probier mal, den parameter auf final zu setzen...

public List<File> find( String path, final String filter )

 

[moerkstar]

So das klappt jetzt erstmal.

import java.io.*; 
import java.util.*; 
//import java.util.regex.Pattern; 
 
public class FileFinder 
{ 
  public static void main( String[] args ) 
  { 
    String filter = args[0];
	String path = new File( System.getProperty("user.dir") ).getAbsolutePath(); 
 
    System.out.println( "Looking in path: " + path ); 
 
    FileFinder ff = new FileFinder(); 
    List<File> files = ff.find( path, (filter) ); 
 
    System.out.printf( "Found %d file%s.%n", 
                       files.size(), files.size() == 1 ? "" : "s" ); 
    for ( File f : files ) 
      System.out.println( f.getAbsolutePath() ); 
  } 

  public List<File> find( String path, final String filter ) 
  { 
      File searchIn = new File( path ); 
      return Arrays.asList( searchIn.listFiles( new FileFilter() 
      { 
          public boolean accept( File file ) 
          { 
              return file.getName().contains( filter );
          } 
      } ) ); 
  } 
}

Jetzt habe ich noch das Problem, dass ich nach mehreren Argumenten suchen will.
Also beliebig viele, die alle im ergebnis vorkommen sollen.
Leider hab ich dahingehend gar keinen Plan.

Danke Dir schonmal im Voraus!


P.S.: Dieses Forum ist genial und die Leute hier sind total korrekt!

 

[mikachu]

das heißt also, dass in der main-methode mehrere parameter angegeben sind ?

wenn ja, dann einfach in der method find() den zweiten parameter als String[] bauen, und dann in der accept() eine schleife über all die strings laufen lassen, und bei erfolgreichem test ein true zurückliefern, ansonsten ganz am ende der accept() methode ein false zurückliefern.

probier erstmal selber

 

[moerkstar]

Also deine Hilfe war genial bis jetzt.

Leider versteh ich den zweiten Teilder Hilfe nicht (der Teil mit der Schleife).

Hab das jetzt so gemacht, aber das ist sicherlich falsch.

import java.io.*; 
import java.util.*;  
 
public class FileFinder 
{ 
  public static void main( String[] args ) 
  { 
    String filter[] = args;
	String path = new File( System.getProperty("user.dir") ).getAbsolutePath(); 
 
    System.out.println( "Looking in path: " + path ); 
 
    FileFinder ff = new FileFinder(); 
    List<File> files = ff.find( path, (filter) ); 
 
    System.out.printf( "Found %d file%s.%n", 
                       files.size(), files.size() == 1 ? "" : "s" ); 
    for ( File f : files ) 
      System.out.println( f.getAbsolutePath() ); 
  } 

  public List<File> find( String path, final String[] filter ) 
  { 
      File searchIn = new File( path ); 
      return Arrays.asList( searchIn.listFiles( new FileFilter() 
      {
          public boolean accept( File file ) 
          { 
        	  if (filter.length>1)
        	  {
        	  for (int i=0; i<filter.length; i++)
        	  return file.getName().contains( filter[i] );
        	  }
        	  else
        		  return file.getName().contains( filter[0]);
          } 
      } ) ); 
  }
}

bei

public boolean accept( File file )

sagt er jetzt:

This method must return a result of type boolean

Hilf mir doch bitte nochmal.


langsam sollte ich dich schon fast stundenweise bezahlen[/code]

 

[Leroy42]

public boolean accept( File file ) {
  boolean accepted = true;
  for (int i=0; i < filter.length(); i++)
    if (!file.getName().contains(filter[i])) {
      accepted = false;
      break;
    }
    return accepted;
}

prüft ob file.getName() alle filter enthält

 

[Leroy42]

Oder alternativ und einfacher zu verstehen:

public boolean accept( File file ) { 
  for (String s : filter) 
    if (!file.getName().contains(s))
      return false;
  return true; 
}

 

[moerkstar]

Super, das klappt prima!
Danke

Könnt ihr mir mal kurz erklären, was der Doppelpunkt hier bedeutet?

for (String s : filter)

 

[Leroy42]

Das ist die Syntax für die erweiterte for-Schleife,
die seit Java 1.5 in die Sprache aufgenommen wurde.

 

[moerkstar]

Danke vielmals

Wie würde denn dieses hier mit nicht erweiterter for-Schleife heißen?

    for ( File f : files )

 

[Leroy42]

for (int i=0; i < files.length; i++) {
  File f = files[i];
  ...
}

 

[mikachu]

for( int index = 0; index < files.length; ++index)
{
    File file = files[index];
}

 

[moerkstar]

Wenn ich das ersetze, dann klappt das nicht. Trotzdem danke für eure Mühe!

Wenn ihr noch wollt, dann hätt ich noch die Frage, was ich machen muss um auch Unterverzeichnisse mit zu durchsuchen?

 

[Leroy42]

Prinzipielle Vorgehensweise (Rekursion)

public void scanDir(File rootDir) {
  for (File f: rootDir.listFiles())
    if (f.isDirectory())
      scanDir(f);
    else
      // stelle was mit der Datei f an
}

 

[moerkstar]

Ich weiß, dass ich wahrscheinlich anfange zu nerven, aber wo muss ich das denn jetzt einbauen?

Nochmal Sorry

 

[mikachu]

public boolean accept( File file ) { 
  for (String s : filter) 
    if (!file.getName().contains(s))
      return false;
  return true; 
}

dohh, hab ich jez nen hänger, oder seh ich das richtig, wenn eine datei den ersten filtertest nicht besteht, aber vll mit dem zweiten filter klappen würde, dass diese datei nicht in die liste aufgenommen wird?

dann nich lieber sorum?

public boolean accept( File file ) { 
  for (String s : filter) 
    if (file.getName().contains(s))
      return true;
  return false; 
}

 

[moerkstar]

Ich glaub das siehst du richtig.

Soll aber auch so sein, denn ich will, dass alle Filter getroffen werden.

 

[mikachu]

wenn du für jeden filter eine datei haben willst, dann musst du das ganz anders angehen...
so, wie ich das verfolgt und verstanden habe, willst du die dateien haben, auf welche einer der filter passt.

also dateiorientiert, und nicht filterorientiert... ???:L

 

[moerkstar]

Ich möchte mehrere Filter übergeben, z.B. txt kop 1, und möchte dann die Dateien aufgelistet haben, in denen alle diese Filter vorkommen.

Und genau das macht das Programm bis jetzt auch.

 

[mikachu]

achso, also nur die dateien, die alle filter besitzen... verstehe

also eine AND filterung statt einer OR filterung -> alle kriterien müssen erfüllt sein :idea:

 

[moerkstar]

:toll:

 

[mikachu]

Jetzt habe ich noch das Problem, dass ich nach mehreren Argumenten suchen will.
Also beliebig viele, die alle im ergebnis vorkommen sollen.
Leider hab ich dahingehend gar keinen Plan.

das hab ich schlicht und einfach überlesen ^^

#edit 1:
in dieser hinsicht revidier ich alle meine beiträge auf dieser seite, welche sich nicht auf dieses zitat bezogen

 

[Leroy42]

achso, also nur die dateien, die alle filter besitzen... verstehe

also eine AND filterung statt einer OR filterung -> alle kriterien müssen erfüllt sein :idea:

Mit Lesen wär dir das nicht passiert!:

Jetzt habe ich noch das Problem, dass ich nach mehreren Argumenten suchen will.
Also beliebig viele, die alle im ergebnis vorkommen sollen.

Also los, ab in die Ecke und schäm dich!

 

[Leroy42]

Ohh! mika hat seinen Fehler ja schon selbst bemerkt.

Dann ist Hopfen und Malz bei ihm ja noch nicht verloren! :wink:

 

[moerkstar]

Ich find das richtig cool, dass sich sogar schon zwei Leute um mich kümmern!

Könnt ihr mir denn noch sagen, wo ich das mit den Unterverzeichnissen einbauen muss?

 

[mikachu]

hey, es ist nicht umsonst freitag, oder

 

[Leroy42]

Ich find das richtig cool, dass sich sogar schon zwei Leute um mich kümmern!

Na dann,

Könnt ihr mir denn noch sagen, wo ich das mit den Unterverzeichnissen einbauen muss?

halte ich mich mal vornehm zurück und überlasse mika die Antwort

(da hab ich ja mal wieder geschickt meine Faulheit überspielt)

 

[mikachu]

aber Leroy bitte... du bist grad richtig in fahrt gekommen, ich hab schon den freitag nachmittag eingeläutet

public List<File> scanDir(File rootDir, final String... filters )
{
    List<File> returnList = find( rootDir.getPath(), filters );
    for (File f: rootDir.listFiles()) 
    {
        if (f.isDirectory())
        {
            returnList.addAll( scanDir(f, filters ) );
        }
    }
    return returnList;
}

also mit scanDir() statt find() aufrufen in der main

also hier übernehm ich jez garkeine garantie... weil nicht getestet :/

 

[Leroy42]

Na gut, moment...

 

[Leroy42]

public List<File> findFiles(File rootDir, String[] filters) {
  return findFiles(rootDir, filters, new ArrayList<File>());
}

public List<File> findFiles(File root, String[] filters, List<File> found) {
  for (File f : root.listFiles())
    if (f.isDirectory())
      found = findFiles(f, filters, found)
    else {
      boolean accepted = true;
      for (String s : filters)
        if (!f.getName().contains(s)) {
          accepted = false;
          break;
        }
      if (accepted)
        found.add(f);
    }
  return found;
}

Achtung: Nur im Post gecodet; nicht compiliert oder ausprobiert!

 

[mikachu]

na nun bin ich ja mal gespannt, ob, und wenn ja, welche version denn funktioniert :lol:

*prosit zu Leroy ausspricht*

 

[Leroy42]

Meine Zuweisung an found ist ja unnötig... :shock:

public List<File> findFiles(File rootDir, String[] filters) { 
  return findFiles(rootDir, filters, new ArrayList<File>()); 
} 

public List<File> findFiles(File root, String[] filters, List<File> found) { 
  for (File f : root.listFiles()) 
    if (f.isDirectory()) 
      findFiles(f, filters, found) 
    else { 
      boolean accepted = true; 
      for (String s : filters) 
        if (!f.getName().contains(s)) { 
          accepted = false; 
          break; 
        } 
      if (accepted) 
        found.add(f); 
    } 
  return found; 
}

So isse besser!

Aufrufen mit

List<File> founded = findFiles(root, stringArray);

 

[moerkstar]

import java.io.*; 
import java.util.*;
 
public class FileFinder 
{ 
  public static void main( String[] args ) 
  { 
    String filter[] = args;
	String path = new File( System.getProperty("user.dir") ).getAbsolutePath(); 
 
    System.out.println( "Looking in path: " + path ); 
 
    FileFinder ff = new FileFinder(); 
    List<File> files = ff.scanDir( path, (filter) ); 
 
    System.out.printf( "Found %d file%s.%n", 
                       files.size(), files.size() == 1 ? "" : "s" ); 
    for ( File f : files ) 
      System.out.println( f.getAbsolutePath() ); 
  } 

  public List<File> find( String path, final String[] filter ) 
  { 
      File searchIn = new File( path ); 
      return Arrays.asList( searchIn.listFiles( new FileFilter() 
      {
    	  
    	  
    	  public boolean accept( File file ) { 
    		  boolean accepted = true; 
    		  for (int i=0; i < filter.length; i++) 
    		    if (!file.getName().toLowerCase().contains(filter[i])) { 
    		      accepted = false; 
    		      break; 
    		    } 
    		    return accepted; 
    		}
      } ) ); 
  }

  
public List<File> scanDir(File rootDir, final String... filters ) 
{ 
    List<File> returnList = find( rootDir.getPath(), filters ); 
    for (File f: rootDir.listFiles()) 
    { 
        if (f.isDirectory()) 
        { 
            returnList.addAll( scanDir(f, filters ) ); 
        } 
    } 
    return returnList; 
}  
  
  
  
}

So sieht deins aus Mika.

Jetzt bekomm ich die Meldung:

The method scanDir(File, String...) in the type FileFinder is not applicable for the arguments
(String, String[])

bei scanDir

    List<File> files = ff.scanDir( path, (filter) );

 

[Leroy42]

mika's scanDir erwartet als erstes Argument ein File, keinen Pfadnamen,
also einfach

List<File> files = ff.scanDir(new File(System.getProperty("user.dir")), args);

schreiben.

 

[moerkstar]

Ich muss jetzt leider zum Zahnarzt, damit ich auch in Zukunft kann.

Ich meld mich spätestens Montag nochmal.

Ich danke euch erneut vielmals !

Hoffe ihr werdet mir dann immernoch helfen.


Ihr bekommt aber jetzt schon definitiv die fünf Daumen.
:toll: :toll: :toll: :toll: :toll:

 

[moerkstar]

Hallo Leute,

nochmals vielen Dank für eure Mühe!

Jetzt bin ich mit meinem Problem zurück.

Zuerst nochmal mein jetziger Code:

import java.io.*; 
import java.util.*;
 
public class FileFinder 
{ 
  public static void main( String[] args ) 
  { 
    String filter[] = args;
	File path = new File( System.getProperty("user.dir") ); 
 
    System.out.println( "Looking in path: " + path ); 
 
    FileFinder ff = new FileFinder(); 
    List<File> files = ff.scanDir(path, filter ); 
 
    System.out.printf( "Found %d file%s.%n", 
                       files.size(), files.size() == 1 ? "" : "s" ); 
    for ( File f : files ) 
      System.out.println( f.getAbsolutePath() ); 
  } 

  public List<File> find( String path, final String[] filter) 
  { 
      File searchIn = new File( path ); 
      return Arrays.asList( searchIn.listFiles( new FileFilter() 
      {
    	  
    	  
    	  public boolean accept( File file ) { 
    		  boolean accepted = true; 
    		  for (int i=0; i < filter.length; i++) 
    		    if (!file.getName().toLowerCase().contains(filter[i])) { 
    		      accepted = false; 
    		      break; 
    		    } 
    		    return accepted; 
    		}
      } ) ); 
  }

  
public List<File> scanDir(File rootDir, final String[] filter ) 
{ 
    List<File> returnList = find( rootDir.getPath(), filter ); 
    for (File f: rootDir.listFiles()) 
    { 
        if (f.isDirectory()) 
        { 
            returnList.addAll( scanDir(f, filter ) ); 
        } 
    } 
    return returnList; 
}  

  
  
}

Und dies bekomme ich in der Kommandozeile:

D:\Projekte\directory_search>java FileFinder txt
Looking in path: D:\Projekte\directory_search
Exception in thread "main" java.lang.UnsupportedOperationException
        at java.util.AbstractList.add(AbstractList.java:151)
        at java.util.AbstractList.add(AbstractList.java:89)
        at java.util.AbstractCollection.addAll(AbstractCollection.java:318)
        at FileFinder.scanDir(FileFinder.java:49)
        at FileFinder.main(FileFinder.java:14)

Danke euch schonmal im Voraus!

 

[Leroy42]

Hier ist der Übeltäter:

returnList.addAll( scanDir(f, filter ) );

Eine List kennt die Methode addAll scheinbar nicht :shock:

1) Versuch mal alle Vorkommen von List durch ArrayList zu ersetzen
oder
2) warte bis mika wieder da ist
oder
3) Nimm meine Lösung, dann brauchst du die Methode
find überhaupt nicht mehr, und musst nicht andauernd zwischen Pfadnamen und Instanzen von File hin- und herzukonvertieren.

 

[moerkstar]

Mit deinem Code hab ich jetzt das:

import java.io.*; 
import java.util.*;
 
public class FileFinder 
{ 
  public static void main( String[] args ) 
  { 
    String filter[] = args;
	File path = new File( System.getProperty("user.dir") ); 
 
    System.out.println( "Looking in path: " + path ); 
 
    FileFinder ff = new FileFinder(); 
    List<File> files = findFiles(path, filter);
    
    System.out.printf( "Found %d file%s.%n", 
                       files.size(), files.size() == 1 ? "" : "s" ); 
    for ( File f : files ) 
      System.out.println( f.getAbsolutePath() ); 
  } 

  
  public List<File> findFiles(File rootDir, String[] filters) { 
	  return findFiles(rootDir, filters, new ArrayList<File>()); 
	} 

	public List<File> findFiles(File root, String[] filters, List<File> found) { 
	  for (File f : root.listFiles()) 
	    if (f.isDirectory()) 
	      findFiles(f, filters, found); 
	    else { 
	      boolean accepted = true; 
	      for (String s : filters) 
	        if (!f.getName().contains(s)) { 
	          accepted = false; 
	          break; 
	        } 
	      if (accepted) 
	        found.add(f); 
	    } 
	  return found; 
	}
}

Bei findFiles in Zeile 14
sagt er jetzt:

Cannot make a static reference to the non-static method findFiles(File, String[]) from the type 
 FileFinder

 

[Leroy42]

List<File> files = new FileFinder().findFiles(path, filter);

 

[Leroy42]

Ach du hast ja schon eine Instanz von FileFinder;
also so:

FileFinder ff = new FileFinder(); 
List<File> files = ff.findFiles(path, filter);

 

[moerkstar]

Geil funktioniert einwandfrei !!!

D A N K E



Kannst du mir jetzt nochmal eben sagen wie das hier

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

als normale Schleife aussieht


Achja, bei den beiden eventuell auch:

	  for (File f : root.listFiles()) 
	    if (f.isDirectory()) 
	      findFiles(f, filters, found); 
	    else { 
	      boolean accepted = true; 
	      for (String s : filters) 
	        if (!f.getName().toLowerCase().contains(s)) { 
	          accepted = false; 
	          break;

 

[Leroy42]

Bitte doch!

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

ist äquivalent zu

 for (int i=0; i < files.size(); i++ ) {
      File f = files.get(i);
      System.out.println( f.getAbsolutePath() );
 }

Bei

for (File f : root.listFiles())

ist es ähnlich, nur das listFiles nur einmal aufgerufen wird.

for (String s : filters)

wird intern transformiert zu

for (int i=0; i<filters.length; i++) {
  String s = filters[i];
   ...

Also im Prinzip genau das, was man sich denkt (und wünscht)

 

[moerkstar]

Gut, ich hab das geschafft bei dem ersten und dritten.

Aber bei

for (File f : root.listFiles())

bekomm ich das einfach nicht hin.

Bitte noch eine Hilfe

 

[Leroy42]

for (File f : root.listFiles())

<==>

File[] allSubFiles = root.listFiles();
for (int i=0; i < allSubFiles.length; i++) {
  File f = allSubFiles[i];
  ...
}

for (int i=0; i < root.listFiles().length; i++) {
  File f = root.listFiles()[i];
  ...
}

wäre nicht äquivalent, da hierbei die Methode listFiles()
andauernd neu aufgerufen werden würde.

 

[moerkstar]

Jetzt schmeißt er keinen Fehler, aber er tut nichts.

D:\Projekte\directory_search>java FileFinder txt
Looking in path: D:\Projekte\directory_search

und da bleibt er stehen