Datei kopieren in anders verzeichnis geht nicht

[LOGiX]

Hallo,

ich habe nun das beispiel auf das minimalste beschränkt sprich die Datei test.txt auf Laufwerk D kopiere ich auch Laufwerk C.

String volume = System.getProperty("user.dir"); // user directory
		File sourceDirectoryPath = new File(volume + "/Created_txt/");  // create directory from user string
		File[] zipfiles = sourceDirectoryPath.listFiles();  // get all files from given directory name  
		 
		 
         for(int i = 0 ; i < zipfiles.length ; i++) // number of files in user directory
         {
        	 // System.out.println(sourceDirectoryPath + "/" + zipfiles[i].getName());
        	   BufferedInputStream in = new BufferedInputStream(new FileInputStream("d:/test.txt")); //source file
			   BufferedOutputStream out = new BufferedOutputStream(new FileOutputStream("c:/", true)); // target directory
			   int bytes = 0; 
			   
			   // Read file into InputStream
			   while ((bytes = in.read()) != -1)
			   { 
				  // Write bytes of file into OutputStream
			      out.write(bytes);  
			      in.close();
				  out.close();
			   }        	 
        	
         }

warum geht das nicht?

 

[Wildcard]

Weil du den Input und den Output Stream nach dem ersten byte schließt.
Dateien größer als ein paar kilobyte kopiert man übrigens mit einem FileChannel.

 

[LOGiX]

Weil du den Input und den Output Stream nach dem ersten byte schließt.
Dateien größer als ein paar kilobyte kopiert man übrigens mit einem FileChannel.

mal unabhängig von der Größe, ich habe die beiden stream#close() methoden jetzt nach der while schleife gemacht ohne erfolg?

 

[Wildcard]

Ohne Erfolg? Du bekommst eine OutOfStrawberriesException?
Wenn du kein Unix-artiges System hast, bin ich mir übrigens fast sicher, das du nicht wahllos bytes in den Mount-Point C:\ schreiben kannst :wink:

 

[LOGiX]

Ohne Erfolg? Du bekommst eine OutOfStrawberriesException?
Wenn du kein Unix-artiges System hast, bin ich mir übrigens fast sicher, das du nicht wahllos bytes in den Mount-Point C:\ schreiben kannst :wink:

hö? ich benutze doch gar net c:\ sondern c:/ ...???

jetzt bin ich verwirrt:
("c:\", true)); <-- mit einem \ bekomme ich eine fehlermeldung, also mache ich 2 mal \\ dann geht fehlermeldung weg, doch die datei test.txt wird net nach c:\\ kopiert, bekomme auch keine fehlermeldung...

noch andere Verständnisfrage: müsste es nicht String dest="C:\temp\test\ heißen also ohne das text.txt ? denn ich will eine datei nach in ein verzeichnis kopieren und nicht eine datei IN ?? eine datei gleichen namens in ein anderes Verzeichnis...

public void Copy() throws IOException{
   String src="C:\temp\text.txt";
   String dest="C:\temp\test\text.txt";
   File inputFile = new File(src);
   File outputFile = new File(dest);

   FileInputStream in = new FileInputStream(inputFile);
   FileOutputStream out = new FileOutputStream(outputFile);

   FileChannel inc=in.getChannel();
   FileChannel outc=out.getChannel();

   inc.transferTo(0,inc.size(),outc);

   inc.close();
   outc.close();

   in.close();
   out.close();
   }

 

[Wildcard]

Du liest den Inhalt einer Datei. Inhalt kann man aber nicht in einem Verzeichnis speicher, sondern in einer Datei.

 

[LOGiX]

Du liest den Inhalt einer Datei. Inhalt kann man aber nicht in einem Verzeichnis speicher, sondern in einer Datei.

ok da hast du recht... wenn man halt net datei kopiert im alltag dann in ein Verzeichnis nicht in eine Datei, daher ist das was du sagst seeeeehr gewöhnungsbedürftig ;-)

soll ich jetzt c:/ oder c:\ <-- geht net machen? denn in dem copy methode beispiel steht ja auch c:\ nur bei mir gehts net??

 

[Wildcard]

Der / ist schon ok, nur musst du eben eine Datei angeben.
Programmierer sind eben (in dieser Rolle) keine Anwender. Wenn ein Anwender eine Datei kopiert, dann wird entsprechender Code ausgeführt, der den Inhalt aus einer Datei liest, eine neue Datei erstellt und ihn dort wieder schreibt.
Das gleiche musst du also auch tun.

 

[LOGiX]

Der / ist schon ok, nur musst du eben eine Datei angeben.
Programmierer sind eben (in dieser Rolle) keine Anwender. Wenn ein Anwender eine Datei kopiert, dann wird entsprechender Code ausgeführt, der den Inhalt aus einer Datei liest, eine neue Datei erstellt und ihn dort wieder schreibt.
Das gleiche musst du also auch tun.

ok DOCH warum hat dieser Code:

String src="C:\temp\text.txt";
String dest="C:\temp\test\text.txt";

2 backslashes was bei mir net funzt wenn ich aber "c:/" nehme geht das kopieren ??? das ist sehr verwirrend...

 

[Wildcard]

\ ist ein Escapezeichen im Quellcode.
\t ist ein Tab

 

[LOGiX]

...ich aber "c:/" nehme so solls heißen wegen dem emoticon...

 

[LOGiX]

\ ist ein Escapezeichen im Quellcode.
\t ist ein Tab

klar nur erklärt das net warum ich c:/ nehmen kann und das kopieren geht, schaue ich die copy methode an die ich im forum per suche fand steht da c:\ und es geht net??? bemerkte das keiner?

 

[Wildcard]

klar nur erklärt das net warum ich c:/ nehmen kann und das kopieren geht, schaue ich die copy methode an die ich im forum per suche fand steht da c:\ und es geht net??? bemerkte das keiner?

weil c : / ein gültiger Pfad ist. von welcher Copy Methode redest du eignetlich?

 

[LOGiX]

klar nur erklärt das net warum ich c:/ nehmen kann und das kopieren geht, schaue ich die copy methode an die ich im forum per suche fand steht da c:\ und es geht net??? bemerkte das keiner?

weil c : / ein gültiger Pfad ist. von welcher Copy Methode redest du eignetlich?

von der? :wink:

public void Copy() throws IOException{
String src="C:\temp\text.txt";
String dest="C:\temp\test\text.txt";
File inputFile = new File(src);
File outputFile = new File(dest);

FileInputStream in = new FileInputStream(inputFile);
FileOutputStream out = new FileOutputStream(outputFile);

FileChannel inc=in.getChannel();
FileChannel outc=out.getChannel();

inc.transferTo(0,inc.size(),outc);

inc.close();
outc.close();

in.close();
out.close();
}

 

[Wildcard]

Wo hast du sie her?

 

[LOGiX]

na aus der forumsuche wie es sich gehört

 

[Wildcard]

Das in einem Forum nicht immer korrekter Code gepostet wird, sollte klar sein (besonders nicht in den Eingangspostings).

 

[Der Müde Joe]

sowas hilft dir vielleicht

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

public class FileCopy {

	public static void copy(File in, File out) throws IOException {
		FileInputStream inFile = new FileInputStream(in);
		FileOutputStream outFile = new FileOutputStream(out);

		FileChannel inChannel = inFile.getChannel();
		FileChannel outChannel = outFile.getChannel();

		ByteBuffer buffer = ByteBuffer.allocate(1024 * 1024);
		
		while ( inChannel.read(buffer) != -1){
			buffer.flip();

			while (buffer.hasRemaining()){
				outChannel.write(buffer);
			}
			buffer.clear();
		}

		inChannel.close();
		outChannel.close();
	}

	public static void main(String[] args) {
		String s = System.getProperty("user.dir");
		File in = new File(s + "/src/FileCopy.java");
		File out = new File("FileCopy.java_bak");

		try {
			copy(in, out);
		} catch (IOException e) {
			e.printStackTrace();
		}
	}
}

EDIT:
Code überarbeitet (aus for wird while schleife: lesbarkeit++)
Hier hats diverse Beispiele dazu:
http://www.java2s.com/Tutorial/Java/0180__File/NewIOTransfer.htm