Hallo Leute,

ich will aus einem Ordner bestimmte vorgegebene Dateien löschen. Meine Umsetzung sieht so aus.
Ich habe die Dateinamen ohne Pfadangabe in ein Array geschrieben.
Dann wird für jeden Dateinamen eine neue Datei deklariert und diese wird dann in dem Ordner gelöscht. Der Code funktioniert, leider hängt sich die Form aber auf und es dauert auch sehr lange, bis das Dingens fertig ist.

kann mir da evtl. jemand weiterhelfen?

hier der Code



Danke

Tom

Erstell dir doch nen Thread (nicht hier im Forum, sondern ne System.Threading.Thread :D), in dem du das dann erledigst. Der läuft, ohne deine Form aufzuhängen.

ok, da müsste ich mich erst mal einlesen. Hat das dann auch Auswirkungen auf die Dauer des Progs?
Wird es dadurch schneller?

Es beeinflusst weder die Dauer noch die Geschwindigkeit des Ablaufes, das Löschen passiert dann einfach unabhängig von der Form (die Form bleibt einfach nicht hängen sondern läuft normal weiter).

hab es jetzt so gelöst





Die Form hängt sich nun nicht mehr auf, aber es dauert wirklich ewig bis das Ding fertig ist.

Gibt es da keine andere Lösung?

EDIT:

es dauert 49 Sekunden

und wie lange hat es vorher gedauert, bis die Form wieder "aufgetaut" is?

Die frage ist halt, wie groß die Dateien sind. wenn die mehrere GB belegen dauert das nun schon mal ne weile, bis windows gesagt hat, "hey hier is wieder Speicherplatz!"

wie sieht es denn aus, wenn du die Dateien von Hand löschst, geht es da schneller?

wie nafets3646 schon gesagt hat.

nafets3646 schrieb:

Es beeinflusst weder die Dauer noch die Geschwindigkeit des Ablaufs

zur Dateigröße:
jede Datei ist zwischen 1 und 3 mb groß. Also sollte das Löschen der Dateien nicht so lang dauern.

Wie lange dauert es denn, wenn Du sie von Hand löschst?
Länger als 2...3 Sekunden sollte es nicht dauern.

@RodfromGermany

genau so ist es. Es dauert wirklich nicht länger. So stell ich mir das von meinem Programm auch vor

EDIT:

Ich bin mittlerweile nicht mehr im Büro am Rechner, sondern im Zug mit Laptop.

Wenn ich das Ding auf dem Laptop ausführe, dann dauert es "nur" 11 sek.

vlt. verwendet File.Delete einen umständlichen Löschvorgang (etwa mit Verschieben in den Papierkorb).
guck dir mal die FileInfo - klasse an, und ob das zackiger deleted.

also,

es funktioniert jetzt. Dauert im Büro auch nur noch ca. 10-15 sek. Je nachdem, wieviel Dateien gelöscht werden. Das passt soweit.


Ich wollte jetzt als Feature noch eine Progressbar einfügen.

Die Texturen werden in Sub gelöscht, die ich von einem Thread aus starte






wenn ich jetzt in dieser Sub die Progressbar ansprechen möchte, dann kommt ne Fehlermeldung mit "Threadübergreifend usw."

woran kann das liegen und wie kann ich den Fehler beheben?


EDIT:

Wenn ich die Sub nicht in einem eigenen Thread starte, funktioniert zwar die Progressbar, aber die Form hängt sich wieder auf

Tom

TomWi schrieb:

Threadübergreifend

Du kannst von einem anderen Thred aus nicht GUI-Elemente verändern (wohl aber auslesen).
Das Zauberwort heißt Invoke.

dann kommen in die Sub RemoveOldTextures(openfilename)) folgende Zeilen?

Wie bereits gesagt invoken. Was jedoch zusätzlich wichtig ist, dass du die ProgressBar angemessen verwendest. Was heißt das: Wenn ich Beispielsweise 500.000 Datensätze in ne Datenbank schreibe und bei jedem die Progressbar aktualisiere, wird der Vorgang gleich nen vielfaches an Zeit verschlingen. Was ich damit sagen möchte ist, dass gerade bei Performanceintensiven sachen die Aktualisierung von Fortschritten nur in gewissen Intervalen vorgenommen werden sollte. In meinem Beispiel z.B. alle 1000 Datensätze. Ich weiß nicht genau wie dein Szenario aussieht aber grundsätzlich einfach darauf achten, dass du nicht zu oft aktualisierst, da dies a) alles verlangsamt und b) wieder den GUI-Thread belastet und die Form somit wieder laggy wird.

EDIT: Nö. Wenn du jedes mal eine neue Form instanzierst wird das recht schnell abschmieren und außerdem ist das auch sinnlos. Du solltest ausschließlich eine Instanz der Form haben(du willst den Status ja nicht immer auf einer neuen Form anzeigen sondern alle Aktualisierungen auf einer einzigen). Optimal wäre es natürlich wenn du ein Event erstellst, welches z.B. einen Prozentsatz und einen Text mitliefert. Dieses Event verarbeitest du dann in der Form und nimmst verwendest dort das Invoke. So wäre eine halbwegs schöne Architektur gewährleistet.

TomWi schrieb:

VB.NET-Quellcode

1. Dim Form1 As New Form

Hier erstellst Du doch eine neue Instanz. Du musst die vorhandene Instanz nehmen, z.B.

Hab es jetzt geschafft, die Progressbar zum laufen zu bringen.
Die Lösung ist der Backgroundworker.

Ich starte die Sub jetzt nicht mehr aus einem Thread, sondern über den Backgroundworker. Somit hängt sich die Form nicht mehr auf und die Progressbar läuft auch.

Danke für die Hilfe

Tom