Sucher erfasst nur Ordner

[André B.]

Ich habe zu Testzwecken eine kleine Suchfunktion geschrieben, die rekursiv Ordner und Dateien suchen soll. Das funktioniert soweit ganz gut, aber ich habe festgestellt, dass der nur Ordner findet und ich komm nicht drauf, woran das liegen könnte. Ich habe sogar mal ein wenig mit dem rumprobiert, was ich hier im Forum so an Code zu dem Thema gefunden habe.
Das ist die Klasse:

import java.io.File;
import java.util.Vector;

import javax.swing.JLabel;

public class Search extends Thread
{
	private File dir;
	private String term;
	private Vector treffer;
	private long start;
	private long end;
	private JLabel currentDir;
	private int count;
	
	public Search(File dir, String searchTerm, JLabel currentDir)
	{
		this.dir = dir;
		this.term = searchTerm.toLowerCase().trim();
		this.treffer = new Vector();
		this.currentDir = currentDir;
		this.count = 0;
	}
	
	private void search(File dir) //Hier wird gesucht
	{
		currentDir.setText(dir.getAbsolutePath());
		File[] files = dir.listFiles();
		if(files != null)
		{
			for(File file : files)
			{
				count++;
				//System.out.println(file.getName().toLowerCase());
				if(file.getName().equalsIgnoreCase(term) | term.equals(""))
				{ 
					treffer.addElement(file.getName());
					System.out.println(treffer.size() + ": " + file.getPath());
				}
				if(file.isDirectory())
					search(file);
			}
		}
	}
	
	public void run()
	{
		start = System.currentTimeMillis();
		search(dir);
		end = System.currentTimeMillis();
		long time = end - start;
		System.out.println("Verbrauchte Zeit: " + time + "ms");
		System.out.println("Durchsuchte Dateien: " + count + " Dateien");
		System.out.println("Geschwindigkeit: " + count/(time/1000) + " Dateien pro Sekunde");
		System.out.println("Treffer: " + treffer.size());
		System.exit(-1);
	}
}

Danke im Vorraus.

Edit:
Mit dieser Klasse teste ich das ganze:

import java.io.File;

import javax.swing.JFrame;
import javax.swing.JLabel;

public class Main
{
	private JFrame frame;
	private JLabel dir;
	
	public Main()
	{
		frame = new JFrame("Suche");
		dir = new JLabel("");
		frame.setSize(700,50);
		frame.getContentPane().add(dir);
		frame.setVisible(true);
		new Search(new File("C:/"), "class", dir).start();
	}
	
	public static void main(String[] args)
	{
		new Main();
	}

}

 

[André Uhres]

               String fn = file.getName().toLowerCase();
                if(fn.startsWith(term) || term.equals("")) {

 

[Murray]

Bei mir funktioniert das Programm (auch ohne die oben vorgeschlagene Änderung; grundsätzlich ist es bei solchen Vergleichen aber schon sinnvoll, || anstelle von | zu verwenden, da in diesem Fall der zweite Teil der Bedingung nicht mehr geprüft werden muss, wenn der ersten Teil den gesamten Ausdruck bereits bestimmt).

Mit dem Testfall gibt es so allerdings wohl meist nicht viel zu finden, weil Dateien mit dem Namen "class" eher selten sind. Sucht man aber z.B. nach Main.class, wird man die wohl finden.

 

[André B.]

Es war ja so, dass wen ich mir den Dateinamen geben lassen habe, dass das dann immer sowas wie Main.class war und der ja eigentlich dann die Datei finden müsste, wenn ich nach "class" suche, aber auch, wenn ich es mit anderen Begriffen gemacht habe, die 100%ig da sein müssen, hat er nur eventuell Ordner, aber keine Dateien gefunden. Und mit startWith() wird doch bloß überprüft, ob der String damit anfängt. Ich möchte aber wissen, ob der Suchbegriffe im Namen enthalten ist, sodass ich beim Suchbegriff "date" auch z.B. "Textdatei.txt" finde.

 

[Bleiglanz]

String#indexOf

 

[André B.]

if(file.getName().indexOf(term) != -1 || term.equals(""))

So siehts jetzt aus und es funktioniert super. Danke

 

[Illuvatar]

String#contains wäre performanter, und auch irgendwie logischer, finde ich

 

[Bleiglanz]

String#contains wäre performanter

wirklich?

glaub ich nicht

 

[Illuvatar]

Hm oder wenigstens gleichschnell

Aber der Code würde trotzdem intuitiver/lesbarer

 

[André B.]

Habs probiert. Ist kaum ein unterschied. Danke für die netten Tipps
MfG Ikarus

 

[Bleiglanz]

Hm oder wenigstens gleichschnell

Aber der Code würde trotzdem intuitiver/lesbarer

jep, würde deshalb auch unbedingt zu "contains" raten!