Dateien werden nicht gelöscht

[EcDev]

Kann mir jmd sagen, warum dieser Code nicht alle dateien aus meinem Ordner löscht? :bahnhof:

File f = new File("C:test");
File[] fs = f.listFiles();
for (int i = 0; i < f.listFiles().length; i++) {
System.out.println(f.listFiles().length);
fs[i].delete();
}

 

[njans]

C:test scheint mir erstmal kein valides Verzeichnis.
Dann ist die Frage, ob du überhaupt Berechtigung hast, da was zu löschen.
Dann ist die Frage, ob die Dateien in Verwendung sind und und und...

 

[BlaueBohne]

würde ich genau so sehen wie mein Vorredner. Versuchs mal mit "C:/test" oder "C:\test" mir fällt gerade nicht mehr ein, welches Zeichen man nochmal benutzen muss. xD

 

[EcDev]

es klappt... naja halb es wird nur die Hälfte gelöscht

 

[BlaueBohne]

versuch mal:

for (int i = 0; i <= f.listFiles().length; i++)

also kleiner-gleich. Möglich, dass es dann klappt?

 

[Joose]

versuch mal:

for (int i = 0; i <= f.listFiles().length; i++)

also kleiner-gleich. Möglich, dass es dann klappt?

Nein weil <= sorgt dafür das die Zählervariable "i" bis zum Wert "f.listFiles().length" steigen kann.
Wenn nun auf mit diesem Wert auf "fs" zugriffen wird bekommt er einen ArrayIndexOutOfBoundException

 

[EcDev]

jedes mal wenn ich das ausführe wird in etwa die hälfte gelöscht... Warum ?

 

[Joose]

Hast du die Rechte um diese Dateien zu löschen?
Hast du schon kontrolliert ob du mit "f.listFiles()" wirklich alle Dateien bekommst die im Ordner sind?

 

[fhoffmann]

Hallo EcDev,

das Problem besteht darin, dass du die Liste der Dateien änderst, während du darüber iterierst.
Stell dir vor, du hast eine Stapel mit 6 Büchern (nummeriert von 0 bis 5) und ziehst das unterste heraus:

5
4
3
2
1
0 -> (wird herausgezogen, und die anderen Bücher fallen nach unten)

Dann ziehst du das zweitunterste heraus:

5
4
3
2 -> (wird herausgezogen, und die anderen Bücher fallen nach unten)
1

dann ziehst du das drittunterste heraus:

5
4 -> (wird herausgezogen, und die anderen Bücher fallen nach unten)
3
1

und erhälst den Bücherstapel

5
3
1

Hier ist die for-Schleife zu Ende (im nächsten Schritt müsstest du das viertunterste herausziehen, hast aber nur noch drei Bücher). Die Hälfte bleibt also liegen.

Diesen Fehler macht jede Programmiererin und jeder Programmierer mindestens einmal , oft sogar öfter (und sucht stundenlang nach dem Fehler) - aber irgendwann riecht man den Braten.

Gruß
Fritz

 

[njans]

Soweit ich das sehe, löscht er die Dateien auf der Platte, nicht aus der Liste. Daher ist fhoffmanns Punkt ungültig.

 

[taro]

Kann mir jmd sagen, warum dieser Code nicht alle dateien aus meinem Ordner löscht? :bahnhof:

File f = new File("C:test");
File[] fs = f.listFiles();
for (int i = 0; i < f.listFiles().length; i++) {
System.out.println(f.listFiles().length);
fs[i].delete();
}

Der Schleifenkopf

for (int i = 0; i < f.listFiles().length; i++) {

wird bei jedem Schleifendurchlauf neu ausgewertet.
Da f.listFiles().length nach jedem durchlauf um 1 kleiner ist und i um 1 größer, ist die Bedingung nach "dem halben geplanten Durchlauf" bereits nicht mehr erfüllt:

length:11
i:0

length:10
i:1

length:9
i:2

length:8
i:3

length:7
i:4

length:6
i:5

Beim nächsten Schleifendurchlauf wäre length < i ...

Damit es wie geplant läuft, muss die for-Schleife wie folgt angepasst werden:

for (int i = 0; i < fs.length; i++) {
...

Man könnte natürlich auch auf length > 0 prüfen, jedoch halte ich das Neueinlesen des Verzeichnisses bei jedem Schleifendurchlauf für äußerst unperformant (wird ja in der Schleife schon nochmal getan - sollte nach Möglichkeit auch verhindert werden).

Grüße
Sven