try/catch + utilizzando, giusta sintassi

Question

Quale:

using (var myObject = new MyClass()) 
{ 
    try 
    { 
     // something here... 
    } 
    catch(Exception ex) 
    { 
     // Handle exception 
    } 
} 

O

try 
{ 
    using (var myObject = new MyClass()) 
    { 
     // something here... 
    } 
} 
catch(Exception ex) 
{ 
    // Handle exception 
} 

Answer 1

io preferisco la seconda. Può anche trattenere errori relativi alla creazione dell'oggetto.

Answer 2

Entrambi sono sintassi valida. Dipende davvero da cosa vuoi fare: se vuoi catturare gli errori relativi alla creazione/eliminazione dell'oggetto, usa il secondo. In caso contrario, utilizzare il primo.

Answer 3

C'è una cosa importante, che chiamerò qui: Il primo sarà non cattura alcuna eccezione derivante chiamando il costruttore MyClass.

Answer 4

Dal momento che un blocco utilizzando è solo una semplificazione della sintassi di un try/finally (MSDN), personalmente mi piacerebbe andare con il seguente, anche se dubito che sia significativamente diversa da quella seconda opzione:

MyClass myObject = null; 
try { 
    myObject = new MyClass(); 
    //important stuff 
} catch (Exception ex) { 
    //handle exception 
} finally { 
    if(myObject is IDisposable) myObject.Dispose(); 
} 

Answer 5

E ' dipende. Se si utilizza Windows Communication Foundation (WCF), using(...) { try... } non funzionerà correttamente se il proxy nell'istruzione using si trova nello stato di eccezione, ovvero lo smaltimento di questo proxy causerà un'altra eccezione.

Personalmente, credo nell'approccio alla gestione minima, cioè gestisco solo l'eccezione di cui sei a conoscenza al momento dell'esecuzione. In altre parole, se si sa che l'inizializzazione di una variabile in using può generare una particolare eccezione, la avvolgo con try-catch. Allo stesso modo, se all'interno del corpo using può accadere qualcosa, che non è direttamente correlato alla variabile in using, allora lo avvolgo con un altro try per quella particolare eccezione. Uso raramente Exception nel mio catch es.

Ma mi piace IDisposable e using anche se quindi forse di parte.

Answer 6

Se la dichiarazione di cattura deve accedere alla variabile dichiarata in un'istruzione using, quindi all'interno è l'unica opzione.

Se l'istruzione catch richiede l'oggetto a cui si fa riferimento nell'uso prima che venga eliminato, all'interno è l'unica opzione.

Se la dichiarazione di cattura prende un'azione di durata sconosciuta, come visualizzare un messaggio per l'utente e si desidera disporre delle risorse prima che ciò accada, quindi all'esterno è l'opzione migliore.

Ogni volta che ho uno scenerio simile a questo, il blocco try-catch si trova di solito in un altro metodo più in alto nello stack di chiamate dall'utilizzo. Non è tipico per un metodo sapere come gestire le eccezioni che si verificano al suo interno in questo modo.

Quindi la mia raccomandazione generale è fuori dall'esterno.

private void saveButton_Click(object sender, EventArgs args) 
{ 
    try 
    { 
     SaveFile(myFile); // The using statement will appear somewhere in here. 
    } 
    catch (IOException ex) 
    { 
     MessageBox.Show(ex.Message); 
    } 
} 

Answer 7

Se l'oggetto che si inizializza nella Utilizzando blocco() potrebbe gettare alcuna eccezione allora si dovrebbe andare per la seconda sintassi altrimenti sia l'altrettanto valida.

Nel mio scenario, ho dovuto aprire un file e stavo passando filePath nel costruttore dell'oggetto che stavo inizializzando nel blocco Using() e potrebbe generare un'eccezione se filePath è sbagliato/vuoto. Quindi in questo caso, la seconda sintassi ha senso.

mio codice di esempio: -

try 
{ 
    using (var obj= new MyClass("fileName.extension")) 
    { 

    } 
} 
catch(Exception ex) 
{ 
    //Take actions according to the exception. 
}