Datei löschen funktioniert nicht sauber

[derToby]

Guten Abend zusammen,
ich habe ein kleines Problem, bei dem ich eure Hilfe benötige:
Und zwar habe ich ein kleines Programm geschrieben, welches mir eine csv. - Datei ("datalt") löschen soll und anschliessend eine andere csv. - Datei("datei2") wieder in "datalt" umbenennen soll. Unten hab ich den Codeauszug angehängt. Es tritt folgendes auf: Beim ersten progdurchlauf funktioniert es einwandfrei, allerdings beim zweiten schon nicht mehr. Leider kann ich mir gar nicht erklären, woran das liegen könnte...

Code in der Main:

if(datlösch(datalt)== true) System.out.println("Fehler0");
if(new File(datei2).renameTo(new File(datalt)) ==  false) System.out.println("Fehler1");

Methoden:

public static boolean datlösch(File path) {
	boolean err = false;
	try {
    	         path.delete();
  	}	
	catch(Throwable e) {
		System.out.println("Fehler: (catch datlösch)");
		err = true;
	}
	return err;
}
	
public static boolean  datlösch(String path ) {
    	if(datlösch( new File(path)) == true) return true;
    	else return false;
}

Hat jemand von euch eine Idee?

gruß derToby

PS: Ich verwende 2 datlösch Methoden, da der Aufruf im weiteren Progverlauf entweder als file oder als String erfolgen kann.

 

[na-oma]

Spezifizier den Fehler oder das unkorrekte Verhalten mal genauer.

u.U. solltest du Dateien schließen, wenn du sie fertig bearbeitet hast.

public static boolean  datlösch(String path ) {
       if(datlösch( new File(path)) == true) return true;
       else return false;
}

==

public static boolean  datlösch(String path ) {
       return datlösch(new File(path));
}[code]
sauberer, lesbarer, wartbarer, kürzer

Umlaute sind allgemein als böse angesehen, ich hab damit keine Erfahrungen, würde aber davon abraten.

[code]catch(Throwable e)

man sollte nur die Errors catchen, die man erwartet(und behandeln kann)
Gewöhn dir sowas gar nicht erst an. Sonst fängst du was wichtiges weg, was villeicht die Lösung des Problems ist.

Wenn es unbedingt sein muss gib wenigstens die genaue Art der Exception und so aus:

catch(Exception e) {
e.printStackTrace();
}

 

[derToby]

Hallo und ersteinmal danke für deine Hilfe!

Danke für deine Code bzw. Codestile Korrekturen. Man lernt nie aus, ist eindeutig besser so. Auch die Umlaute werde ich in Zukunft meiden.

Zu meinem Problem:
Ich lösche also die Datei datalt. Dann benenne ich die Datei datei2 in datalt um. Nun wird die neue Datei datalt ausgewertet, d.h. auch geöffnet. Nach der Auswertung, fängt es wieder von vorne an, d.h. die Datei datalt wird gelöscht, usw...
So sollte es zumindest laufen, allerdings funktioniert das Ganze immer nur beim ersten Durchlauf. Das Programm ist von einer Endlosschleife umgeben. Beim zweiten, usw... Durchlauf funktioniert es leider nicht mehr, d.h. die Datei datalt wird gar nicht mehr erst gelöscht und deshalb auch gar nicht mehr neu durch das Umbennen von datei2 erzeugt.
Wenn ich das so lese, kannst du eigentlich nur Recht haben. Kann das leider erst morgen wieder nach prüfen, weil ich dann wieder zu Hause bin.
Ich vermute, ich öffne zum Auswerten der Datei einen Stream, der nicht korrekt wieder geschlossen wird und deshalb blockt Windows das Löschen der Datei.

Ich werde es ausprobieren und mich dann noch mal melden!

schönen Dank schonmal!

derToby

PS: Kannst du mir ein Buch oder einen Link bezüglich guten Programmierstiel empfehlen? Denke da hab ich noch Nachholbedarf... ;-)

 

[L-ectron-X]

Direkt vom Chef: :wink:
http://java.sun.com/docs/codeconv/html/CodeConvTOC.doc.html

 

[derToby]

Danke euch beiden. habt mir sehr geholfen, der springende Punkt war, dass ich den FileReader nicht sauber geschlossen hab.

gruß

derToby