Mehrere Datein kopieren

[Devox]

Hallo leute,

hab ein Problem bei Kopieren von mehreren Dateien. Ich habe die zukopierenden Pfade alle in einem
Array.

Das Array übergebe ich so:

for (int i = 0; i<dateiAnzahl;i++){
			copyFile(files[i], new File(Ziel));
		}

An diese Methode

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

Jetzt weiß ich nicht wie ich bei Code2 in Line 2 den Array richtig an den FileInputStream übergebe.
Außerdem habe ich Probleme dabei Line 5 zu verstehen. Die "0" ist laut der API, wenn ich das richtig verstanden hab die Position der zu Kopierenden Datei(Ist mir nicht ganz schlüssig wofür ich das brauche etc.) und das "inChannel.size()" bereitet mir noch ein bissel Kopfzerbrechen, wie kann ich den die Größe der Datei im Vorraus angeben?

mfg

 

[Michael...]

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

Der Code kompiliert doch gar nicht. Einen Konstruktor FileInputStream(File[]) gibt es nicht. Wird hier nicht sowieso ein File Objekt übergeben.

Außerdem habe ich Probleme dabei Line 5 zu verstehen. Die "0" ist laut der API, wenn ich das richtig verstanden hab die Position der zu Kopierenden Datei(Ist mir nicht ganz schlüssig wofür ich das brauche etc.)

Die 0 besagt, das der Stream ab dem ersten Byte in den outChannel übertragen werden soll.

und das "inChannel.size()" bereitet mir noch ein bissel Kopfzerbrechen, wie kann ich den die Größe der Datei im Vorraus angeben?

wozu berechnen? Die Methode size() liefert doch die Dateigröße? Ansonsten kann man auch das File Objekt mittels length() abfragen.

 

[pappawinni]

Mhm..
sowas ähnliches hab ich hier
Copy a file - Real's Java How-to
gefunden.

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

Man darf da natürlich nicht

public static void copyFile(File[B][][/B] in, File out)

schreiben.
Das würde ja bedeuten, dass du eine Array von Files übergeben musst.
Du übergibst aber genau ein File, nämlich files.
Wie dein "Ziel" generiert wird, verrät dein Code nicht. Dazu lässt sich also wenig sagen.

[EDIT]
Dein Ziel ist aber, so wie du das hier zeigst, so NICHT sinnvoll.
[/EDIT]

 

[bone2]

du for schleifst über das array und kopierst jede datei einzelnd. ganz einfach. wo ist das problem?

 

[tröööt]

naja ... wenns unbedingt mit channels sein muss ... bitte ...
hier noch zwei andere varianten

public void copyFile(File source, File destination) throws Exception
{
	FileInputStream in=new FileInputStream(source);
	FileOutputStream out=new FileOutputStream(destination);
	byte[] buffer=new byte[1048576];
	int c=0;
	while((c=in.read(buffer))!=-1)
	{
		out.write(buffer, 0, c);
	}
}

oder mit NIO.2 und einer der drei Files.copy() methoden ...

warum ihr da mit channels rumspielt versteh ich nich wirklich

 

[vanny]

Ob mit Channel oder ohne ist sicherlich ne schöne Diskussionsbasis aber nicht das Problem des TO.
Wie schon erwähnt verabeitet die copyFile()-Methode jeweils nur ein File und kein ganzes Array.

 

[Devox]

Der Code kompiliert doch gar nicht. Einen Konstruktor FileInputStream(File[]) gibt es nicht.

Ist natürlich nur ein Ausschnitt des GesamtCodes.

du for schleifst über das array und kopierst jede datei einzelnd. ganz einfach. wo ist das problem?

Das Problem ist das ich nicht weiß wie ich das genau machen kann, wenn ich es gewusst hätte, hätte ich mein Problem nicht im Forum gepostet

naja ... wenns unbedingt mit channels sein muss ... bitte ...
hier noch zwei andere varianten

public void copyFile(File source, File destination) throws Exception
{
	FileInputStream in=new FileInputStream(source);
	FileOutputStream out=new FileOutputStream(destination);
	byte[] buffer=new byte[1048576];
	int c=0;
	while((c=in.read(buffer))!=-1)
	{
		out.write(buffer, 0, c);
	}
}

oder mit NIO.2 und einer der drei Files.copy() methoden ...

warum ihr da mit channels rumspielt versteh ich nich wirklich

Müssen keine Channels sein!!!


Da es Anscheindend ein paar Verständnis Probleme gibt, werde ich jetzt die ganze Klasse Posten.

Damit jetzt keiner Fragt woher kommen den "quelle" & "ziel". Ausschnitt aus der Main.java:

DateiKopierenClass.anzahlPruefen(new File(tPfad1.getText()), tPfad2.getText());

Komplette Klasse:

import java.io.File;
import java.io.FileInputStream;
import java.io.FileOutputStream;
import java.io.IOException;
import java.nio.channels.FileChannel;

public class DateiKopierenClass {
    static int dateiAnzahl = 0;
   
    public static void anzahlPruefen(File quelle, String Ziel) throws IOException {
    	//File dir = new File(quelle);
		File[] files = quelle.listFiles();
		if (files != null) {
			for (int i = 0; i < files.length; i++) {
				if (files[i].isDirectory()) {
						anzahlPruefen(files[i],Ziel);
					}
				else {
						dateiAnzahl++;
						System.out.println("Datei: "+files[i].getAbsolutePath());
				}
			}//for
		}//if
		
		for (int i = 0; i<dateiAnzahl;i++){
			copyFile(files[i], new File(Ziel));
		}
		
	}//anzahlPruefen()
    
    public static void copyFile(File in, File out) throws IOException {
        FileChannel inChannel = new FileInputStream(in).getChannel();
        FileChannel outChannel = new FileOutputStream(out).getChannel();
        try {
            inChannel.transferTo(0, inChannel.size(), outChannel);
        } catch (IOException e) {
            throw e;
        } finally {
            if (inChannel != null)
                inChannel.close();
            if (outChannel != null)
                outChannel.close();
        }
    }
}

Freue mich auf weitere Antworten

mfg

 

[Mujahiddin]

Wieso sagst du nicht, dass du einen ganzen Ordner kopieren willst? Danach siehts nämlich aus.

Erstens solltest du dir bewusst sein, dass deine Methode "anzahlPruefen" absolut falsch benannt ist. Wenn ich als Außenstehender eine Methode namens "anzahlPruefen" sehe, erwarte ich, dass diese Methode irgendeine Menge von etwas zählt und mir dann ein int zurückgibt. Bei dir kopiert es einen ganzen Ordner irgendwohin. Zweitens kopierst du Unterverzeichnisse mit, also kann es zu Namenskollisionen führen. Wie willst du diese behandeln?

Deine Methode momentan versucht, alle Dateien in einem Verzeichnisbaum in eine Datei zu kopieren. Das bedeutet, wenn im Ordnerbaum der Quelle 100 Dateien sind, wird die Zieldatei 100 mal neu erstellt und am Schluss steht nur die 100te Datei drin.

Du müsstest schon ausführlicher beschreiben, WAS du genau vorhast. Willst du einen gesamten Ordnerbaum angefangen bei "Quelle" mit seiner ganzen Struktur genau so in einen anderen Ordner übertragen? Oder willst du alle Dateien aus "Quelle" in eine Datei schreiben, sodass du am Ende eine Zieldatei hast, die eine Folge von den n Dateien aus "Quelle" ist?

 

[Devox]

Ich bin denke ich mal falsch an die Sache rangegangen. ja eigentlich will ich einfach nur den Ordner kopieren, mit allen Unterverzeichnissen und mit allen enthaltenen Datein.

 

[Mujahiddin]

Falls du Java 7 hast, benutze die Methode

Files.walkFileTree

mit

SimpleFileVisitor<Path>

.
Das ist viel komfortabler. Du musst den Quellordner als

Path

angeben (entweder mit

file.toPath()

oder

Paths.get(blabla)

)
Dann musst du dich bei

preVisitDirectory

den Ordner erstellen mit

Files.createDirectory

.
Bei

visitFile

kannst du

Files.copy

verwenden, allerdings musst du dich darum kümmern, dass es auch in den richtigen Ordner gelangt. Das machst du am besten so:

Path start = Paths.get( "C:", "Ordner", "Soundso", "Start" ); // der Quellordner
Path p = Paths.get( "C:", "Ordner", "Soundso", "Start", "Unterordner 1", "Datei 2" ); // momentane Datei (path in visitFile)
Path current = start.relativize( p ); // "Unterordner 1/Datei 2"
Path fileDest = targetFile.resolve(current); // "x:/y/Zielordner/Unterordner 1/Datei 2"

 

[Devox]

Danke für die Hilfe, habs jetzt hinbekommen zwar etwas anders als du es mir beschrieben aber es läuft.


mfg