Come gestire le eccezioni?

Question

È una domanda sulla migliore pratica .net. Ho sempre preso e memorizzate le informazioni sulle eccezioni in tutte le mie applicazioni web e finestre forme come segue:

1. circondato ogni metodo con il tentativo di cattura (eccezione Exception)
2. Per ogni strato tranne strato anteriore, gettò eccezione alla strato sopra
3. Nel livello anteriore, ha registrato l'eccezione in un file di registro (in genere utilizzando log4config.dll) e ha presentato all'utente un messaggio di facile utilizzo.

È questo il modo migliore per gestire le eccezioni? O dovrei fare qualcosa di diverso nei miei prossimi progetti?

Answer 1

Il seguente codice è problematico perché sovrascrive l'analisi dello stack originale per e, il che rende i problemi più difficili da diagnosticare:

public void Foo() { 
    try { 
     Bar(); 
    } catch(Exception e) { 
     throw e; // re-throw; overwrites original stacktrace in 'e' 
    } 
} 

Il seguente codice non dispone quanto sopra stacktrace problema di sovrascrittura, ma è ancora inutilmente prolisso :

public void Foo() { 
    try { 
     Bar(); 
    } catch(Exception e) { 
     throw; // re-throw; preserves original stacktrace in 'e' 
    } 
} 

Entrambi sarebbero meglio scritti come di seguito. Questo perché, se l'unica cosa che si sta facendo nel vostro blocco catch è ri-gettando l'eccezione, non c'è bisogno di scrivere il blocco catch a tutti:

public void Foo() { 
    Bar(); 
} 

Answer 2

La risposta migliore che si può ottenere presso Best Practices for Handling Exceptions

Answer 3

non vorrei aggiungere 1 & 2 a meno che non ho avuto qualche motivo specifico; per esempio per alterare (avvolgere) il messaggio; non ce n'è bisogno visto che le eccezioni saliranno comunque verso l'alto. E fatto in modo errato si può rimuovere per errore l'importantissima traccia dello stack (throw; vs throw ex; - il primo preferito).

Answer 4

Ecco come NON a gestire le eccezioni.

public void method1(){ 
    try{ 
     .... 
     // all the code goes here 
     // declare any helper methods throw throwable 
     // eg: private boolean check1() throws Throwable{ ...} 
     .... 
    }catch(Throwable t){ 
     System.out.println("oops...error...good luck debugging. "); 
    } 
}