Throw exception in try block

Question

try { 
     if (isFileDownloaded) 
      //do stuff 
     else 
      throw new CustomException() 
    } 
    catch (Exception e) 
    { 
     // something went wrong save error to log 
    } 
    finally 
    { 
     //release resources 
    } 

La mia domanda è il catch intercetta il blocco ApplicationException nel blocco try? è in uno stile di programmazione scadente? Dovrebbe essere scritto in un altro modo?

Answer 1

Il catch cattura l'eccezione (e qualsiasi altra ciò accade). Detto questo, cerco di evitare di scrivere codice come questo, quando possibile.

Personalmente, vedo poche ragioni per avere una gestione delle eccezioni (catch) per un'eccezione generata nello stesso ambito. Se riesci a gestire il tuo errore nel tuo metodo, metti anche la gestione delle eccezioni (cioè: logging) direttamente nel blocco try.

L'utilizzo di catch è più utile, IMO, per rilevare eccezioni generate da metodi all'interno del blocco try. Ciò sarebbe più utile, ad esempio, se la tua sezione // do stuff abbia chiamato un metodo che ha generato un'eccezione.

Inoltre, si consiglia di non rilevare tutte le eccezioni (Exception e), ma piuttosto i tipi specifici di eccezioni che è possibile gestire correttamente. L'unica eccezione potrebbe essere se si sta rilanciando l'eccezione all'interno del proprio catch, ad es .: usandolo per scopi di logging ma continuando a farlo esplodere nello stack delle chiamate.

Answer 2

Sì, prenderà ApplicationException come deriva da Exception.

Gestione eccezione di base dovrebbe andare bene nella maggior parte dei casi a meno che non devi effettuare il login o fare qualcosa con un diverso tipo di eccezione ...

try{ 
    if (isFileDownloaded) 
     //do stuff 
    else 
     throw new ApplicationException(); 
} 
catch(ApplicationException ae) 
{ 
    // log it application exception here... 
} 

catch(Exception ex) 
{ 
    // log all other exceptions here... 
} 
finally 
{ 
    // release resources... 
} 

Answer 3

Sì, il catch catturerebbe la tua ApplicationException e sì è uno stile di codifica scadente. Come regola generale, prendi in considerazione solo le eccezioni specifiche e quelle con cui dovrai fare qualcosa, ad esempio correggere lo stato dell'applicazione.

Answer 4

Inoltre, FYI, ApplicationException è stato deprecato poiché .NET 2.0 è un'eccezione da cui derivare. Non è mai stato concepito come un'eccezione da lanciare da solo, quindi probabilmente non dovresti usarlo affatto.