Beste Art Bilder zu kopieren?

[Matscher]

Huhu! Hab mal wieder ne Frage,

und zwar mein Programm soll Bilder (*.jpg, *.gif, *.png) kopieren können,
dies hab ich zur Zeit mit folgender Funktion gemacht:

	private void BildKopieren(String Name, String Pfad)	{	// Funktion für das Bilderkopieren
		try {
			RandomAccessFile datei		= new RandomAccessFile(Pfad,"r");
			RandomAccessFile neudatei	= new RandomAccessFile("Bilder/"+Name, "rw");

		while (neudatei.length() < datei.length()) {
	        	neudatei.write(datei.read());
	        }
		datei.close();
		neudatei.close();

		System.out.println(Name+" wurde kopiert");
		}
		catch (IOException ex) { System.out.println("MÖP!!!"); }
	}

Geht das nich auch irgendwie besser, das is nähmlich extrem CPU-lastig soweit ich das beurteilen kann, ausserdem wollte ich noch einen JProgressBar hier ein bauen und das macht sich irgendwie doof, da die CPU völlig mit dieser Schleife beschäftigt is und eigentlich gar nich hinterher kommt die JProgressBar zu erneuern!

Also geht das nich mit ner spezielleren Funktion?

 

[Illuvatar]

1. Solltest du nicht immer nur 1 byte lesen und wieder schreiben. Stichworte: read(byte[]) oder BufferedReader/-Writer.
2. Das mit der JProgressbar liegt daran: KLICK MICH

 

[Beni]

Wenn du schon RandomAccessFiles benutzt, kannst du auch gleich die nio benutzen: du schnappst dir die Channels der Dateien (getChannel) und mit einem java.nio.ByteBuffer kannst du dann gleich mehrere Kilobytes in einem Auruf kopieren. Hübscher als die Geschichte mit den Arrays ist es allemal (vielleicht sogar einfacher denn: bei read(byte[]) oder write(byte[]) wird nicht immer der gesammte Array gelesen/geschrieben :!: )

 

[Guest]

Wenn du schon RandomAccessFiles benutzt, kannst du auch gleich die nio benutzen: du schnappst dir die Channels der Dateien (getChannel) und mit einem java.nio.ByteBuffer kannst du dann gleich mehrere Kilobytes in einem Auruf kopieren. Hübscher als die Geschichte mit den Arrays ist es allemal (vielleicht sogar einfacher denn: bei read(byte[]) oder write(byte[]) wird nicht immer der gesammte Array gelesen/geschrieben :!: )

Kann ich Beni nur zustimmen, ich hab mir ein kleines Tool geschrieben, mit dem ich 2 Verzeichnisse auf die enthaltenen Dateien prüfe und dann auch gleich abgleiche, sprich die unterschiedlichen Datein hin und her kopiere. Das Kopieren selber mache ich auch mit Channels. Hier haste mal meinen Code für:

   private synchronized void copy(final File from, final File to) {

      try {
         final FileChannel in = new FileInputStream(from).getChannel();
         final FileChannel out = new FileOutputStream(to).getChannel();

         out.transferFrom(in, 0, in.size());

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

         SKGlobals.debug(from.getPath() + " - erfolgreich kopiert.");
      } catch (FileNotFoundException fnfe) {
         SKGlobals.debug("Replicate - copy() - Datei nicht gefunden (" + from.getPath() + ")");
         SKGlobals.debug(fnfe);
      } catch (IOException ioe) {
         SKGlobals.debug("Replicate - copy() - IOException (" + from.getPath() + ")");
         SKGlobals.debug(ioe);
      }

      // Dafür sorgen, dass sich das Änderungsdatum der Dateien nicht unterscheidet
      to.setLastModified(from.lastModified());
   }

Vielleicht hilft dir das ja weiter.

 

[Eminent]

Mist vergessen einzuloggen!

Der Beitrag drüber is von mir. Die SKGlobals-Zeilen kannste natürlich vergessen, die sind nur für mich von nutzen in meinem Tool!