è consigliabile utilizzare try-finally solo per assicurarsi che qualcosa venga eseguito al termine del metodo?

Il mio metodo ritorna in molti punti. Costruisco newData durante l'esecuzione anche in molti punti. Indipendentemente da dove torno, devo salvare e memorizzare il risultato costruito. Da non perdere "return" Ho appena circondato il codice con il blocco try-finally, quindi ora sono sicuro che i nuovi dati verranno archiviati.è consigliabile utilizzare try-finally solo per assicurarsi che qualcosa venga eseguito al termine del metodo?

List<X> newData = new List<X>(); 
try 
{ 
    .... 
    update newData 
    ..... 
    return; 
    ..... 
    .... 
    update newData 
    .... 
    update newData 
    return; 
    ..... 
    return; 
} finally 
{ 
    // copy newData to data  
}

Ma non rilevo eccezioni e questo codice non è destinato a funzionare con eccezioni. È accettabile in generale o puoi suggerire un altro approccio migliore?

fonte

2012-05-14 javapowered

Sembra che si possa usare una [istruzione switch] (http://msdn.microsoft.com/en-us/library/06tc147t%28v=vs.100%29.aspx). Non vedo perché è necessario tornare immediatamente. –

È accettabile solo se funziona come previsto. Se non funziona, prova a rifarlo in funzioni più piccole. Se lo fa, qual è il problema? Non posso dare alcun suggerimento specifico senza vedere il codice sorgente più dettagliato. – Flaise

A meno che qualcuno non tocchi la presa, il blocco 'finally' viene eseguito in ogni caso e il codice funzionerà come previsto; tuttavia, come già sottolineato da altri, esistono modi migliori per strutturare il codice. –

Vorrei suggerire il refactoring del codice all'interno del blocco try in un nuovo metodo:

data = CreateList(); 

... 

private List<X> CreateList() 
{ 
    List<X> list = new List<X>(); 
    // It's fine to return list from any point here... 
}

fonte

2012-05-14 12:33:47

Questo è quasi quello che stavo per suggerire: aggiungerei che in alcuni casi, quando per la leggibilità è meglio mettere il corpo di 'CreateList' nell'altro metodo, si può usare un delegato' Action' da cui un 'return' esce solo quel metodo anonimo. – hvd

grazie, è quello che mi serve – javapowered

L'utilizzo di finalmente è intesa come una copia di backup se qualcosa non riesce (pensare ad esso come un meccanismo di fail-safe),

Generalmente l'intera costruzione del tuo metodo è sbagliata e puoi refactoring, dato che ottenere tutti questi ritorni di solito significa che puoi adottare un altro approccio alle cose (ad esempio usando un interruttore come qualcuno nei commenti suggeriti) .

fonte

2012-05-14 12:36:21 TheBoyan

'switch' non è quello che mi serve. il codice all'interno ha molte ramificazioni e difficile da refactare in quanto la logica non è banale – javapowered

è consigliabile utilizzare try-finally solo per assicurarsi che qualcosa venga eseguito al termine del metodo?

risposta

Problemi correlati