Methode läuft nicht unter UNIX

[Basti91]

Hallo, ich habe eine Methode geschrieben, die unter Windows wunderbar läuft.
Sie durchsucht rekursiv das übergebene Verzeichnis nach .java Dateien und gibt die Pfäde zu diesen als ArrayList zurück.
Einzige Änderung dich ich seit der UNIX-Umstellung getroffen habe, ist das Erstetzen von "\\" zu "/".

Hier der Code:

	private ArrayList<String> searchforJava(File dir) {
		File[] files = dir.listFiles();
		ArrayList<String> matches = new ArrayList<String> ();
		if (files != null) {
			//filling matches 
			for (int i = 0; i < files.length-1; i++) {
				if (files[i].getName().endsWith(".java")) { 
						matches.add(files[i].toString().substring(0,files[i].toString().lastIndexOf("/")));
					}
				if (files[i].isDirectory()) {
					matches.addAll(searchforJava(files[i]));
				}
			}
			//delete multiple entries
			for (int i=0;i<=matches.size()-1;i++){
				for(int j=0;j<=matches.size()-1;j++){
					if(j==i){
						continue;
					}
					else if(matches.get(i).equalsIgnoreCase(matches.get(j))){
						matches.remove(j);
					}
					
				}
			}
			}
		
		return matches;
	}

 

[Deros]

und eine Fehlermeldung oder eine Erklärung was nicht "läuft" hälst du für unnötig?

 

[Basti91]

Fehlermeldung gibt es keine, er findet einfach nur keine Datei un gibt ein leeres Array zurück

 

[SlaterB]

das ist ja ganz schön knapp als Frage,
vielleicht fehlen dir allein schon Rechte in UNIX?
ob du den Aufruf drumherum hinbekommst ist auch eine Ausgangsfrage, vielleicht gehst du von anderem Grundverzeichnis aus,
muss alles nichts mit dem Code zu tun haben

du musst schon debuggen bzw. System.out.println() einbauen, ganz zu Beginn der Methode um überhaupt zu klären ob der Code je drankommt,
dazu den kompletten Pfad des Parameters, File.getCanonicalPath(), vergleiche mit Dateisystem,
gib die gelisteten Files aus,
gib aus ob der Test [c]if (files.getName().endsWith(".java"))[/c] gelingt,
gib aus ob lastIndexOf() was findet, was bei subString() rauskommt usw.

damit es nicht zuviel wird mit möglichst einfachen Verzeichnissen testen

alles ist Arbeit für dich, Zauberlösungen gibts nicht immer

 

[SlaterB]

das Löschen der Doppelten ist übrigens recht buggy,
wenn du einen Index löschst (z.B. 5) und dann im nächsten Schleifendurchlauf den Index auf 6 erhöhst, wird ein Element übersprungen, welches von gerade von 6 auf 5 gerutscht ist,
durch den Doppeldurchlauf kommt es vielleicht noch später dran, aber wenn dann gar Elemente vor dem aktuellen Vergleichselement gelöscht werden, wirds noch schlimmer,

folgendes Programm endet in Exception:

public class Test {
    public static void main(String... args)   {
        ArrayList<String> matches = new ArrayList<String>();
        Collections.addAll(matches, "A", "B", "C", "B", "B", "D", "B", "B", "D");

        // delete multiple entries
        for (int i = 0; i <= matches.size() - 1; i++)    {
            for (int j = 0; j <= matches.size() - 1; j++)    {
                if (j == i)       {
                    continue;
                }   else if (matches.get(i).equalsIgnoreCase(matches.get(j)))   {
                    matches.remove(j);
                }
            }
        }
        System.out.println(matches);
    }
}

Doppelte kann man gut mit einem HashSet entfernen

 

[Basti91]

Habe jetzt mal mit dem debuggen begonnen, anscheinend wird schon durch

File[] files = dir.listFiles();

ein leeres Array erzeugt

 

[SlaterB]

siehe auch mein Post zwischendurch,

wichtig ist nun exakt auszugeben was 'dir' ist, dir.getCanonicalPath() und wie gesagt mit Dateisystem vergleichen,
auch exists() abfragen,

wie auch immer, dein Code ist damit vorerst obsolet, mindestens ist die Frage auf eine Grundfunktion von File reduziert,
bzw. wahrscheinlicher wählst du das falsche Verzeichnis aus oder hast dort keine Rechte