Grosse Dateien kopieren/Verschieben

[scaary]

Hallo.

Mein Programm soll "grosse" Dateien (~10 MB) kopieren koennen, und das in einer akzeptablen Zeit.

Mein Code schaut wie folgt aus:

	public static void moveFilesToDestination(File[] list, String destination) throws IOException, PersonAlreadyExists {
		for (int i = 0; i < list.length; ++i) {
			if (list[i] != null) {
				File file = list[i];
				String newDestination = folderApplications + destination + "\\";
				File newFile = new File(newDestination + file.getName());
				newFile.createNewFile();
				writeNewFileContent(newFile, getFileContent(file));
				file.delete();		
			}
		}
	}

Dabei wird eine Liste von Dateien uebergeben, die an die angegebene Adresse kopiert wird.

Bei kleinen Datein, kein Problem.

Ist die Datei allerdings 8 MB oder so gross, dauert das ganze schon wieder ewig (min 10 minuten).

Das kanns ja nicht sein.

Ich hab mich dann ein wenig im Internet schlau gemacht, und von NIO gehoert.

Hab hierfuer auch einige Beispiele gefunden:

Schnell große Dateien kopieren mit Java NIO - Java @ tutorials.de: Forum, Tutorial, Anleitung, Schulung & Hilfe
(um nur mal eins zu nennen).

Aber auch dafuer bruacht das Programm ewig.

Was mach ich falsch?

und gibt es eine bessere Loesung?

Danke!

LG scaary

 

[faetzminator]

Wenn die Datei auf der gleichen Partition bleibt, muss nur der Header überschrieben werden und es geht in wenigen ms.

Edit: @Illuvatar, ups, den hab ich vergessen

 

[Illuvatar]

Wenn die Datei auf der gleichen Partition bleibt, muss nur der Header überschrieben werden und es geht in wenigen ms.

Dafür kann die Methode renameTo verwendet werden.

 

[scaary]

Mhh, ich kann leide rnicht davon ausgehen, dass sie auf der selben Partition bleiben

Edit:

Haett ich erwaehnen sollen:

Zumal es auch sein kann, dass selbe Dateien an verschiedene Orte kopiert werden muessen.

 

[faetzminator]

Ich würds mit bytes versuchen, z.B. mit folgendem Code: Copying One File to Another (Java Developers Almanac Example)

 

[Wildcard]

8MB kopieren 10 Minuten? Wie schaut denn getFileContent und writeNewFileContent aus?
Mit NIO und transferTo geht das sehr schnell.

 

[scaary]

Hier die eiden anderen methoden:


/**
* Get text content from a file.
*
* @param file
* @return
* @throws IOException
*/

	public static String getFileContent(File file) throws IOException {
		String result = "";
		String tmp = "";
		reader = new BufferedReader(new FileReader(file));
		do {
			tmp = reader.readLine();
			if (tmp != null) {
				result = result + tmp;
			}
		} while (tmp != null);
		reader.close();
		return result;
	}

/**
* Write text content in a file.
* @param file
* @param text
* @throws IOException
*/

public static void writeNewFileContent(File file, String text) throws IOException {
		if(file.exists()) {
			output = new FileWriter(file);
			output.write(text);
			output.close();
		} else {
			throw (new FileNotFoundException());
		}
	}

Aber wie gesagt, ich hatte die Methode auch schon komplett mit dieser ersetzt:
Schnell große Dateien kopieren mit Java NIO - Java @ tutorials.de: Forum, Tutorial, Anleitung, Schulung & Hilfe

Damit haette es ja schneller gehen sollen, oder?

 

[ARadauer]

also mit nio mach ich das immer so... versuch genau mal diesen code

public static void copyFileNio(File srcFile, File destFile) throws IOException {
		FileChannel inChannel = new	FileInputStream(srcFile).getChannel();
		FileChannel outChannel = new FileOutputStream(destFile).getChannel();
		try {
			inChannel.transferTo(0, inChannel.size(),outChannel);
		}
		catch (IOException e) {
			throw e;
		}
		finally {
			if (inChannel != null) inChannel.close();
			if (outChannel != null) outChannel.close();
		}
	}

 

[scaary]

so nen aehnlichen hatte ich schonmal drin, habs aber grade nochmal mit deinem programm versucht.

EIne 8MB PDF weigert sich wehement schnell kopiert zu werden, also nach wie vor selbes problem... .

Ist doch seltsam, oder?

Kleine Dateien werden ohne probleme kopiert (also kanns auch kein Fehler irgendwo anders sein).

 

[Wildcard]

Das dein Code langsam ist wundert nicht. Du hast dir die sowohl in Bezug auf Laufzeit, als auch Memory Footprint so ziemlich die schlechtest mögliche Variante ausgesucht. String + String und alles in den Speicher, beides für sich alleine schon Schlimm, kombiniert grässlich.
Das die NIO Variante nicht signifikant schneller sein soll wage ich zu Bezweifeln. Mach ein lauffähiges Beispiel mit gegebenem NIO Code und schau ob sich das Problem reproduzieren lässt.

 

[scaary]

Ahh, es funktioniert jetzt.

Ich weiss nicht warum, aber mit deinem Code, ARadauer, hat es funktioniert.

Sehr seltsam, aber es geht jetzt, habt vielen Dank.

Jetzt will ich mich mal drum kuemmern, den schwer kritisierten String mit dem Builder zu ersetzen (ich war bisher einfach zu faul, um ehrlich zu sein).

Habt trotzdem alle vielen Dank!

LG scaary

 

[Wildcard]

Jetzt will ich mich mal drum kuemmern, den schwer kritisierten String mit dem Builder zu ersetzen (ich war bisher einfach zu faul, um ehrlich zu sein).

Nein. Wenn du Dateien kopieren willst, brauchst du dafür keine Strings, das ist langsamer, braucht mehr Speicher und bringt ganz neue Probleme wie Encoding. Man kopiert entweder per FileChannel, oder Per InputStream + OutputStream (oder PipedInput/OutputStream), nicht per String.

 

[scaary]

ah, OK.
Und wenn ich ne Datei nur auslesen moechte, kann ich da den inputStream direkt in nen String umleiten (oder gibt mir das dann kauderwelsch)?

Danke!

 

[Wildcard]

Für Strings nimmt man normalerweise Reader. Wenn deine Quelle ein InputStream ist, dann nimmt man einen InputStreamReader. Das funktioniert allerdings nur für textuelle Inhalte und nur wenn das Encoding stimmt.

 

[Leroy42]

	...
		do {
			tmp = reader.readLine();
			if (tmp != null) {
				result = result + tmp;
			}
		} while (tmp != null);
...

Kein Wunder daß es Minuten benötigt, 10 MB-Dateien einzulesen. :bae:

Du solltest einen StringBuilder benutzen anstatt die Zeilen einfach nur zu adden.