FileOutputStream aktuelle Speicher Pfad ausgeben

[jaykop]

Hallo,

ich habe hier ein Kleines Programm das eine Datei Kopiert. Das Programm kann man in der Konsole mit folgendem Befehl aufrufen:

>java FileCopy quelldatei zieldatei

Meine Frage ist jetzt wie ich ausgeben kann wohin die Datei kopiert wurde. Standardmäßig wird es ja dahin kopiert wo sich auch das Programm FileCopy.java befindet. Wenn man keinen Speicherort angegeben hat sondern nur eine Zieldatei. Ob dies jetzt einen sinn hat ist dahingestellt weil man ja schon vorher weiß wo sich die Datei befindet.

import java.io.FileInputStream;
import java.io.FileOutputStream;
import java.io.File;
import java.io.FileNotFoundException;

public class FileCopy {

    public static void main(String[] args) throws java.io.IOException {

        FileInputStream fis =null;
        FileOutputStream fos =null;
        
        try {   // Abfangen wenn Datei nicht vorhanden
            
            // Einlesen zu Kopierende Datei
            fis = new FileInputStream(args[0]);
        
        } catch (FileNotFoundException filenotfoundex) {
            System.out.println("Fehler! Datei nicht gefunden");
        } catch (ArrayIndexOutOfBoundsException arrayEx) {
            System.out.println("Fehler! Keine Datei zum Kopieren vorhanden");
        }
        
        try { // Abfangen wenn Ort zum Speichern nicht angegeben
            
            // Einlesen Speicherort oder Datei Name
            fos = new FileOutputStream((args[1]));
            int c;
            c = fis.read();

            while (c != -1) {
                fos.write(c);
                c = fis.read();
            }
            fis.close();
            fos.close();
        
        } catch (ArrayIndexOutOfBoundsException arrayEx) {
            System.out.println("Fehler! Kein Ort zum Speichern angegeben");
        }
        

    }
}

 

[The_S]

System.out.println(new File(args[1]).getAbsolutePath());

 

[jaykop]

Danke The_S für die schnelle Antwort.
Wie ist es eigentlich mit meiner Behandlung der exeptions kann man dies übersichtlicher gestallten oder ist das so in Ordnung ?

Hier das Komplette Programm falls es einen interessiert:

import java.io.*;

public class FileCopy {

    public static void main(String[] args) throws java.io.IOException {

        FileInputStream fis = null;
        FileOutputStream fos = null;

        try {   // Abfangen wenn Datei nicht vorhanden

            // Einlesen zu Kopierende Datei
            fis = new FileInputStream(args[0]);

        } catch (FileNotFoundException filenotfoundex) {
            System.out.println("Fehler! Datei nicht gefunden");
        } catch (ArrayIndexOutOfBoundsException arrayEx) {
            System.out.println("Fehler! Keine Datei zum Kopieren vorhanden");
        }

        try { // Abfangen wenn Ort zum Speichern nicht angegeben

            // Einlesen Speicherort oder Datei Name
            fos = new FileOutputStream((args[1]));
            int c;
            c = fis.read();

            /*
             * Die einzelnen Bytes so lange auslesen bis fis.read() als
             * ergebniss -1 liefert da erst so erkannt wird das man am letzten
             * Byte gelangt ist.
             *
             */
            while (c != -1) {
                fos.write(c);
                c = fis.read();
            }
            System.out.println("Die datei wurde erfolgreich Kopiert nach: \n" + new File(args[1]).getAbsolutePath());
        } catch (ArrayIndexOutOfBoundsException arrayEx) {
            System.out.println("Fehler! Kein Ort zum Speichern angegeben");
        } finally {
            try { // FileInputStream und  FileOutputStream schließen wenn was eingelesen wurde
                if (fis != null) {
                    fis.close();
                }
                if (fos != null) {
                    fos.close();
                }
            } catch (IOException ex) {
            }
        }



    }
}

 

[The_S]

Nö, passt in deinem konkreten Fall so. Schließen von Streams sollte aber in den (bei dir noch nicht vorhandenen) finally-Block.

Kopieren kann man aber wesentlich schneller .

 

[jaykop]

Ok Danke. Dies habe ich dann auch oben im Code eingefügt in einem extra Block.

Ja dies war in meinem Fall nur eine Übung um die Exceptions abzufangen. Danach habe ich aber gemerkt das es nicht schlecht wäre den Speicher Ort auszugeben damit man nicht so lange suchen muss wo die Datei abgespeichert wurde. Wie man schneller Kopiert habe ich noch nicht recherchiert.
Wenn du Lust hast kannst du mir ja einen Tipp geben ?

 

[The_S]

Die einfachste Erweiterung wäre es, nicht jedes Byte einzeln zu lesen und zu schreiben, sondern ganze Blöcke, bei kleineren Dateien die komplette Datei, auf einmal zu lesen und zu schreiben. Noch schneller geht es mit Java NIO. Und seit "kurzem" gibt es auch NIO2, das noch mal eine gute Ecke schneller als NIO ist.

[EDIT]Bei deiner aktuellen Version hast du das Problem, dass dein OS nicht geschlossen wird, sollte es beim Schließen vom IS zu einer Exception kommen.[/EDIT]

 

[jaykop]

Cool muss ich mir mal anschauen vieleicht lege ich dies auch später als ergänzung hier rein.
Damit man die Geschwindigkeit vergleichen kann.

[EDIT]Bei deiner aktuellen Version hast du das Problem, dass dein OS nicht geschlossen wird, sollte es beim Schließen vom IS zu einer Exception kommen.[/EDIT]

Dies habe ich leider nicht verstanden.

 

[The_S]

Du wirst über den Geschwindigkeitsboost, gerade gegenüber deiner aktuellen Version, sehr überrascht sein

 

[The_S]

Dies habe ich leider nicht verstanden.

Normalerweise macht man für jede "Schließbare Ressource" einen eigenen try-catch-Block.

 

[jaykop]

Normalerweise macht man für jede "Schließbare Ressource" einen eigenen try-catch-Block.

Mit diesem Code Block wird doch dies verhindert oder muss ich da nochwas ergänzen ?

finally {
            try { // FileInputStream und  FileOutputStream schließen wenn was eingelesen wurde
                if (fis != null) {
                    fis.close();
                }
                if (fos != null) {
                    fos.close();
                }
            } catch (IOException ex) {
            }
        }

 

[The_S]

Nein, machst du nicht. fis und fos werden im selben try-catch-Block geschlossen. Wirft jetzt fis beim schließen eine Exception wird fos nie geschlossen.

 

[jaykop]

OK dann lege ich diese lieber in ihre eigenen try Blöcke. Mir fällt aber kein fall ein wo fis z.B beim schließen eine Exeption auswerfen kann. Kein mir einer vieleicht da einen Beispiel nennen?

finally {
            try { // FileInputStream und FileOutputStream schließen wenn was eingelesen wurde
                if (fis != null) {
                    fis.close();
                }
            } catch (IOException ex) {
            }
            try {
                if (fos != null) {
                    fos.close();
                }
            } catch (IOException ex) {
            }

 

[The_S]

Prinzipiell reicht es, dass die API-Doc schreibt, dass es sein kann . Bei Datenströmen ist das - zugegeben - auch nicht ganz so kritisch wie bei anderen schließbaren Ressourcen wie bspw. Datenbankverbindungen.

 

[ThreadPool]

Normalerweise macht man für jede "Schließbare Ressource" einen eigenen try-catch-Block.

Wobei hier, falls denn Java 7 verwendet wird, es möglich ist das etwas abzukürzen indem man das neue "try-catch-with-resources" verwendet und die gemeine Resource das Closeable oder das allgemeinere AutoCloseable implementiert. Die unterdrückten Exceptions die beim Schließen fliegen muss man sich aber selbst rausfummeln. Die hängen dann für gewöhnlich an der IOException (bei Verwendung von Closeable).

public class FileCopy  {
    public static void main(String[] args){
 
        try (FileInputStream fis  = new FileInputStream(args[0]);
             FileOutputStream fos = new FileOutputStream((args[1]))){   
            int c = -1;
            while ((c = fis.read())!= -1) {
                fos.write(c);
            }
            System.out.println("Die datei wurde erfolgreich Kopiert nach: \n" + args[1]);
        } catch (FileNotFoundException filenotfoundex) {
            System.out.println("Fehler! Datei nicht gefunden");
        } catch (ArrayIndexOutOfBoundsException arrayEx) {
            System.out.println("Fehler! Keine Datei zum Kopieren vorhanden");
        } catch (IOException e) {
            System.out.println("Fehler! Lesen oder Schreiben fehlerhaft!");
        }
    }
}