Exceptions und IO

[Hellweg]

Ich habe eine frage ob mir jemand dort weiter helfen kann ich weiß nicht ob das richtig ist was ich gemacht habe.

Ich bitte um Hilfe.

Also :

Es sollen die Abfragen ergänzt werden und eine adäquate Behandlung der möglicherweise auftretenden Standardexceptions an allen relevanten Stellen.

import java.io.*;

public class FileCopy
{
static void copy( InputStream in, OutputStream out ) throws IOException
{
// Hier findet der eigentliche Kopiervorgang statt, diese Methode ist
//nicht Gegenstand der Betrachtung …
}
static void copyFile( String src, String dest )
{

FileInputStream fis = null;
FileOutputStream fos = null;


fis = new FileInputStream(src );
fos = new FileOutputStream( dest );


copy( fis, fos );


if ( fis != null )



if ( fos != null )


}

Ich habe auch einen Lösungsansatz weiß aber nicht ob das so richtig ist oder ob noch etwas fehlt.

Meine Lösung :

if ( fis != null )
try{
       fis.close();
} catch(IOException e){

   ; }



if ( fos != null )
ry{
       fos.close();
} catch(IOException e){

   ; }

 

[Marco13]

"copy" und der Konstruktor von File*Stream können auch eine IOException werfen. Und ganz allgemein:

Eine Exception abfangen und nicht darauf reagieren ist so ziemlich das ungünstigste, was man machen kann. Wenn man das Programm startet ist es irgendwann zuende, und man weiß nicht, ob etwas schiefgegangen ist oder nicht.

 

[Hellweg]

Hi danke schon mal für deine Hilfe. Aber das ist doch nicht nötig wo rauf soll es den reagieren oder was meinst du??

 

[SlaterB]

auf einen Fehler immer reagieren, im einfachsten Falle
throw new RuntimeException(e);

sonst tritt ein Fehler auf und wirst nicht informiert

 

[tfa]

Du verschluckst die Exceptions einfach. Generell ist das böse. Man sollte sie mindestens loggen oder weiterwerfen. Davon abgesehen hast du natürlich recht, auf eine IOException beim Schließen eines Streams kann man sonst nicht sinnvoll reagieren.

 

[Shulyn]

Wie wäre es wenn du einen ExceptionHandler benutzt?

public Class ExceptionHandler {
	
	// Exception behandeln
	public void handleException(Throwable exception, String additionalInfos){
          // exception behandeln wie du willst
        }

     // Warnings behandeln
    public void handleWarning(String message){
     // warnings behandeln wie du willst
    }
}

// --- in deiner classe ---

ExceptionHandler excHandler = new ExceptionHandler();

// in der Methode

try {
// irgendwas
}
catch (Throwable th) {
excHandler .handleException(th, "infos");
// mach noch was
 }

So kannst du alle Exceptions oder Warnings zum ExceptionHandler "schieben" und dort dann behandeln / Loggen oder oder oder
-------
So kannst du dann z.B immer gleich auf IOExceptions reagieren, oder dein Program beim auftreten einer RuntimeException "sicher" beenden..
Und du sparst dir die Arbeit in jedem Try-Catch Blog gezielt die Exception zu behandeln


//EDIT -Habe Exception durch Throwable ersetzt

 

[Hellweg]

Hi danke für euere Hilfe!!
Das ist eine gute lösung Shulyn aber es solle nur an den bestimmten stellen eine exception gesetzt werden.

 

[Hellweg]

Ich habs mal so gelöst.

import java.io.*;
 
public class FileCopy
{
static void copy( InputStream in, OutputStream out ) throws IOException
{
// Hier findet der eigentliche Kopiervorgang statt, diese Methode ist
//nicht Gegenstand der Betrachtung …
}
static void copyFile( String src, String dest ) throws IOException, FileNotFoundException{
{
 
FileInputStream fis = null;
FileOutputStream fos = null;
 
try{ 
fis = new FileInputStream(src );
fos = new FileOutputStream( dest );
 }catch (FileNotFoundException e){}
 
copy( fis, fos );
 
 if ( fis != null )
try{
       fis.close();
} catch(IOException e){ }
 
 
 
if ( fos != null )
ry{
       fos.close();
} catch(IOException e){ }

 }
} [/Java]

 

[Hellweg]

Ich hab aber auch nochmal eine andere Frage.

Wie mache ich, dass die Methode copyFile soll eine IdentException zurückgeben, wenn die übergebenen Strings identisch sind. Es soll dafür eine eigene Klasse IdentException geschrieben werdenbund bauen Sie diese Exception in die obige Anwendung ein. ??

LG Stefan

 

[SlaterB]

wo ist denn da die Frage?
wie man grundsätzlich neue Klassen schreibt, was dabei speziell hinsichtlich Exceptions zu beachten ist (wovon erben z.B.) und wie man Exceptions dann erzeugt,
das muss doch alles bekannt sein oder in deinem Lehrbuch stehen,
(alternativ hier: Galileo Computing :: Java ist auch eine Insel (8. Auflage) – 8 Exceptions )

es macht keinen Sinn, Fragen ohne eigene Beteiligung 1:1 an ein Forum weiterzureichen

 

[Hellweg]

Das eine Problem hab ich denke ich gelöst.

die Exception in das obige Programm einzubauen.

if(src.equals(dest))
    throw new IdentException("same filenames");

oder gibts eine bessere Variante.

Jetzt ist die frage wenn die übergebenen Strings identisch sind, soll dafür eine eigene Klasse IdentException geschrieben werden. Der Galileo Link hat mir nicht weiter geholfen.

 

[Marco13]

Du verschluckst die Exceptions immernoch. Mach' wenigstens eine Ausgabe, und springe dann aus der Methode raus, sonst kracht's da ohne Ende. Und die zweite Frage... versuch's mal, und poste dann ggf. den Code mit einer Frage die ... präziser ist als "Wie geht das?".

EDIT: Da hätte ich wohl mal reload drücken sollen

 

[Shulyn]

Ok jetzt habe ich dich besser verstanden. Ich hatte nur das hier gelesen.

Es sollen die Abfragen ergänzt werden und eine adäquate Behandlung der möglicherweise auftretenden Standardexceptions an allen relevanten Stellen.

Custom Exceptions würde ich nur benutzen wenn es wirklich nötig ist.
Hier mal dein Quelltext + ExceptionHandler.

/** Deine FileCopy Classe */


package //dein paclage

import java.io.FileInputStream;
import java.io.FileNotFoundException;
import java.io.FileOutputStream;
import java.io.IOException;
import java.io.InputStream;
import java.io.OutputStream;

import ExceptionHandler;  // muss importiert werden.


public class FileCopy {

       private ExceptionHandler handler = new ExceptionHandler();
        
        void copy( InputStream in, OutputStream out ) throws IOException {
            // Hier findet der eigentliche Kopiervorgang statt, diese Methode ist
            //nicht Gegenstand der Betrachtung …
        }
        void copyFile( String src, String dest ) throws IOException, FileNotFoundException {

            FileInputStream fis = null;
            FileOutputStream fos = null;
     
            try { 
                fis = new FileInputStream(src );
                fos = new FileOutputStream( dest );
            }
            catch (FileNotFoundException e) {
                handler.handleException(e,"copyFile");
                return;
            }
            try {
                copy( fis, fos );
            }
            catch (IOException e) {
                handler.handleException(e, "copy");
                return;
            }
            finally {
            try {
                if ( fis != null ) 
                    fis.close();
                if ( fos != null )
                    fos.close();
            }
            catch (IOException e) {
                handler.handleException(e, "close");
                return;
            }
            }
        }
}

/** Nochmal der ExceptionHandler */
public Class ExceptionHandler {
    
    // Exception behandeln
    public void handleException(Throwable th, String infos){
                  String neu = th.toString();
                   System.out.print("Eine Exception ist in der Methode " + infos + "aufgetreten");
                   System.out.print(neu);
// exception behandeln wie du willst
// das nur beispiel        
}
 
     // Warnings behandeln
    public void handleWarning(String message){
     // warnings behandeln wie du willst
    }
}

So sparst dir ein parr try catch blöcke... und hast die Exceptionbehandlung gleich mit drin..

 

[SlaterB]

die Streams sollten wohl auch im Falle einer Exception besser geschlossen werden,
mit finally geht das einigermaßen

 

[Shulyn]

die Streams sollten wohl auch im Falle einer Exception besser geschlossen werden,
mit finally geht das einigermaßen

Eingebaut, hatte ich vergessen

 

[tfa]

Noch nicht 100%ig sicher. Wenn beim Schließen von fis eine IOEx fliegt, bleibt fos offen.

 

[Shulyn]

Noch nicht 100%ig sicher. Wenn beim Schließen von fis eine IOEx fliegt, bleibt fos offen.

Ahh mist stimmt wieder :shock:
Aber das darf er selber "Basteln"

 

[Hellweg]

Vielen Vielen dan euch allen ihr habt mir sehr weiter geholfen. Danke

Liebe Grüße Stefan