Ordner/Unterordner nach Dateien durchsuchen

Basti · 22. Januar 2010, 00:39

Also ich sitze jetzt schon seit ca. 4 Stunden vor folgendem Problem:

Ich möchte gerne, dass alle mp3-Dateien eines bestimmten Ordners (inkl. mp3-Dateien aus den Unterordnern) in eine Liste abgespeichert werden.
Dazu habe ich mir das Script "mp3_list" programmiert, was rekursiv aufgerufen werden soll.

GML-Quellcode

list = ds_list_create();
file = file_find_first(argument0+"\*",fa_directory)
while(file != "")
{
if(file != "." && file != "..")
{
//Ist file ein Ordner, so füge der Liste die Liste aus diesem Ordner hinzu
if(directory_exists(argument0+"\"+file))
{add_list(list,mp3_list(argument0+"\"+file,true));}
//Ist file eine mp3-Datei, so füge sie in die Liste ein
if(filename_ext(file) == ".mp3" || filename_ext(file) == ".MP3")
{ds_list_add(list,argument0+"\"+file);}
}
file = file_find_next();
}
file_find_close();
return list;

Alles anzeigen

Zum Testen habe ich einen Ordner "Ordner" in dem zwei Ordner "OrdnerA" und "OrdnerB" und eine mp3-Datei enthalten ist.
Desweiteren ist in "OrdnerA" und "OrdnerB" auch jeweils eine mp3-Datei drin.

Also sollte insgesamt nacher 3 Dateien in der Liste abgespeichert sein.
Es befindet sich nach dem Aufruf des Scripts aber nur eine Datei in der Liste, und zwar die aus dem "OrdnerA".

Wo liegt der Fehler?
Ich hoffe, dass sich jemand die Mühe macht, mein Script zu verstehen und mir helfen kann.
Wahrscheinlich ist das Problem viel einfacher, als es aussieht...

€dit: Die Methode "add_list(argument0,argument1)" habe ich selbst geschrieben und fügt einfach die zwei Listen "argument0" und "argument1'" zusammen zu "argument0".
Diese Methode ist getestet und funktioniert. Daran kann es also nicht liegen.

Dustboy · 22. Januar 2010, 13:04

Das Problem was du hast ist wirklich einfach.
Es ist nämlich so, dass eine Variable, die du in einem script initialisiertst in das Objekt gelegt wird, dass das script ausführt. Spirch: Deine Varialbe liste liegt dem Objekt, dass das script ausführt. Wenn das script bei Zeile 10 ankommt wird es neu ausgeführt. Das neu ausgeführte Script überschreibt die Variable, die die andere Liste speichert, da es ja selbst eine Variable liste im Objekt initialisiert. Damit sind dann sämtliche bisherigen Einträge verloren gegangen.
Du musst also nur dafür sorgen, dass immer eine andere Variable für deine Liste initialisiert wird (z.B. mit einem Array)

MfG
Dustboy

Basti · 22. Januar 2010, 13:23

Wie doof.
Warum ist das denn so?
Hat das Vorteile?

Aber Array ist doch auch schlecht.
Ich weiß ja gar nicht, wie viele Unterordner es gibt...
Also würde ich dann eher eine Liste nehmen.
Nur wie mach ich das dann?

Dustboy · 22. Januar 2010, 13:46

Na ja das hat den Vorteil, dass du die Variablen in einem Objekt mittels eines Scripts maipulieren kannst

Zu deinem Problem: Ich geh einfach mal davon aus, dass dein Script add_list nicht mehr macht als den Inhalt der liste argument1 an den Inhalt den liste argument0 anhängt. In dem Fall könntest du einfach die Variable list außerhalb des scriptes initialisieren (wegen mir ein einem neuen script was lediglich die Variable initialisiert und dann dieses Script aufruft. Das würde dann auch die Funktion add_list unnötig machen.

Noch ein weiteres Problem fällt mir da grad auf: Mit deiner suche wirst du das gleiche Problem haben. Daher wäre es am Besten wenn du erst alle Ordner des Aktuellen Ordners in eine temporäre liste speicherst und die am Schluss abarbeitest.

Basti · 22. Januar 2010, 14:49

Naja ob das so ein Vorteil ist, ich weiß nicht.
Solche Variablen var von dem Objekt obj würde ich dann lieber mit obj.var oder so ansprechen.
Aber darum geht es ja jetzt nicht.

Wennn ich aber alle Ordner in eine Liste speichern möchte, dann muss ich dass doch rekursiv machen, da jeder Unterordner ja wieder Unterordner haben kann.
Nur so wie ich das verstanden habe ist Rekursion so gut wie unmöglich, da ja kein script seine "eigenen" Variablen haben darf...

MasterXY · 22. Januar 2010, 16:32

Hab mal vor längerer Zeit was ähnliches gemacht vll. hilft dir das von der grundidee her:

GML-Quellcode

// scr_scansubdirs(dir,list to save to)
var f,d,sub;
sub=''
f=file_find_first(argument0+'\*.*',noone)
while (f!='')
{
if (string_lower(filename_ext(f))='.mp3')
|| (string_lower(filename_ext(f))='.wma')
|| (string_lower(filename_ext(f))='.ogg')
|| (string_lower(filename_ext(f))='.wav')
{
ds_list_add(argument1,argument0+'\'+f)
}
else if (string_lower(filename_ext(f))=''&&filename_name(f)!='..'&&filename_name(f)!='.')
{
sub+=argument0+'\'+f+'|'
}
f=file_find_next()
}
while (string_count('|',sub))
{
scr_scansubdirs(string_copy(sub,1,string_pos('|',sub)-1),argument1)
sub=string_delete(sub,1,string_pos('|',sub))
}

Alles anzeigen

Speichert gefundene Unterordner in einer Variable mit Trennzeichen | und führt dann im selben Skript nochmal dasselbe im ausgewählten unterordner aus.

Basti · 22. Januar 2010, 22:47

Vielen Dank!
Es funktioniert!

Aber eine Frage habe ich noch:
Warum findet der Ordner, wenn bei file_find_first()
noone steht?
Ich dachte da müsste dann sowas rein wie fa_directory!?

MasterXY · 22. Januar 2010, 22:54

Wenn man fa_directory reinschreibt, findet er nur Ordner. Bei noone findet er alles.

Basti · 23. Januar 2010, 00:15

Doch noch nicht ganz perfekt...
Wenn in einem Ordnernamen ein . (Punkt) drin ist, dann wird dieser übersprungen.
(z.B. "Bravo Hits vol. 34")

Aber wenn man statt

GML-Quellcode

string_lower(filename_ext(f))=''

besser

GML-Quellcode

directory_exists(argument0+'\'+f)

dann gehts.
Danke nochmal!