Datei verschieben mit einer Schleife

[Leonhead]

Hallo,
ich versuche schon die ganze Zeit in einer for-Schleife Dateien zu verschieben aber es rührt sich nichts. Wenn ich außerhalb der Schleife eine einzelnde Datei verschieben will funktioniert es. Die Parameter neu und alt werden jedenfalls richtig übergeben. Das ist jedenfalls der Code Ausschnitt verstehe überhaupt nicht wo das Problem liegt.

File dir = new File(a);
		String neu,alt;
		
		File[] fileList = dir.listFiles() ;
		
		for(int i=0;i<fileList.length;i++){

			neu=pfad+name+"/"+fileList[i].getName();
			alt=pfad+fileList[i].getName();

		FileVerschieben(neu,alt);
}
public void FileVerschieben(String neu,String alt){

		File quelle = new File(alt);
		File ziel = new File(neu);
		
		quelle.renameTo(ziel);
		
	}

 

[kay73]

• Fliegt garantiert keine Exception?
• Mal zwischendurch File.exists() mitgeloggt?

 

[Leonhead]

Also quelle.renameTo(ziel) gibt jedes mal false zurück.

 

[Leonhead]

Das file ziel scheint nicht zu existieren, liegt das am Pfad?

 

[Murray]

Irgendwie sieht das redundant aus - wo ist denn der Zusammenhang zwischen a und pfad? M.E. wäre es auch besser, beim Umbenennen direkt das File-Objekt zu verwenden und nicht aus dem File einen String und danach aus dem String wieder ein File zu erzuegen.

 

[Jedit]

Könntest du mir bitte sagen wofür die einzlnen variablen stehen. also name vor allem.

 

[Leonhead]

An den Klammern liegt es nicht. Die Variable name ist nur ein String der an den Pfad dran gehängt werden soll und a ist auch ein String.
Murray hat näturlich recht ich kann ja gleich das file Objekt verwenden. Aber was ich nicht verstehen ist wenn ich System.out.println(ziel.exists()); mache kommt immer false raus.

 

[Murray]

Vielleicht versuchst Du es mal so: (ohne Gewähr, der Code ist nur so in den Editor getippt)

File destDir = new File( pfad+name);
if ( !destDir.exists()) throw new IOException( "Ziel " + destDir.getAbsolutePath() + " existiert nicht nicht");
if ( !destDir.isDirectory()) throw new IOException( "Ziel " + destDir.getAbsolutePath() + " ist kein Verzeichnis");
if ( !destDir.canWrite()) throw new IOException( "Ziel " + destDir.getAbsolutePath() + " kann nicht beschrieben werden");


File srcDir = new File(a);        
File[] fileList = srcDir.listFiles() ;

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

	File srcFil = filList[i];
	File destFil = new File( destDir + File.separator + srcFil.getName());
	boolean success = srcFile.renameTo( destDir + File.separator + srcFil.getName());
	if ( !success)  throw new IOException( srcFile.getAbsolutePath() + " konnte nicht in " + destDir.getAbsolutePath() + " umbenannt werden" );
}

 

[Murray]

Aber was ich nicht verstehen ist wenn ich System.out.println(ziel.exists()); mache kommt immer false raus.

Vor oder nach der Rename-Aktion?

 

[Leonhead]

Vor UND nach der Rename-Aktion.
Also in deinem Beispiel würde success false sein und die Exception ausgelöst werden.

 

[Murray]

Vor UND nach der Rename-Aktion.

VOR ist gut, NACH ist schlecht. Hast Du es mal mit meinem Ansatz versucht? Oder trifft das den Kern Deines Problems nicht?

 

[Leonhead]

Ich hab das jetzt so gemacht wie du gleich die files zu verwenden, allerdings kann man bei der renameTo Methode keinen String als Parameter übergeben. Also hab ich wieder das File ziel erzeugt und als Parameter angegeben. Es kommt aber immer noch die Exception das succes nicht erfolgreich war.

 

[Murray]

Sorry, da war natürlich ein Fehler in Zeile 14; eigentlich war es so gedacht:
[JAVA=13] File destFil = new File( destDir + File.separator + srcFil.getName());
boolean success = srcFile.renameTo( destFil);
[/code]

 

[Murray]

Es kommt aber immer noch die Exception das succes nicht erfolgreich war.

Die Prüfungen aus meinem Code-Fragement haben nicht zugeschlagen?

Wie genau lautet die Exception?

Und poste doch zur Sicherheit nochmal den aktuellen Code.

 

[Leonhead]

	File dir = new File(a);

		File[] fileList = dir.listFiles();

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

			File destDir = new File( pfad+name);			
			File quelle = fileList[i];
			File ziel = new File(destDir + File.separator + quelle.getName());

			boolean success = quelle.renameTo(ziel);
			if (!success)throw new IOException( quelle.getAbsolutePath() + " konnte nicht in " + ziel.getAbsolutePath() + " umbenannt werden" );
            }

Das renameTo gibt mir immer false zurück. Welchen Grund kann es denn haben wenn renameTo nicht funktioniert die Quelldatei aber existiert?

 

[Murray]

Welchen Grund kann es denn haben wenn renameTo nicht funktioniert die Quelldatei aber existiert?

- Die Zieldatei könnte bereits existieren und schreibgeschützt sein
- Das Zielverzeichnis könnte nicht existieren
- Das Zielverzeichnis könnte schreibgeschützt sein

Deshalb hatte ich bei mir entsprechende Prüfungen eingebaut.

Wenn das renameTo false liefert, müsste ja die Exception kommen - wie sieht die genau aus?

 

[Leonhead]

Es fällt ja nur die Exception die definiert wurde wenn succes gleich false ist.
Spielt das eine Rolle das ich meine Main- Methode in einer anderen Klasse habe, weil wenn ich einfach alles in Main reinhaue verschiebt er auch die Ordner sonst aber nicht.

 

[Leonhead]

Leute ich hab den Fehler gefunden. Ich hatte irgendwo in einer anderen Klasse einen Stream geöffnet

BufferedReader in = 
			new BufferedReader(new FileReader(datei));

und vergessen ihn wieder zu schließen

in.close();

dummes Missgeschick.