pro e contro di TryCatch rispetto a TryParse

Question

Quali sono i pro e i contro dell'utilizzo di uno dei seguenti metodi per estrarre un doppio da un oggetto? Al di là delle preferenze personali, solo le questioni che sto cercando feedback su includere la facilità di debug, le prestazioni, la manutenibilità ecc

public static double GetDouble(object input, double defaultVal) 
{ 
    try 
    { 
     return Convert.ToDouble(input); 
    } 
    catch 
    { 
     return defaultVal; 
    } 
} 

public static double GetDouble(object input, double defaultVal) 
{ 
    double returnVal; 
    if (double.TryParse(input.ToString(), out returnVal)) 
    { 
     return returnVal; 
    } 
else 
    { 
     return defaultVal; 
    } 
} 

Answer 1

• TryParse sarà più veloce di cattura un'eccezione
• TryParse indica qualcosa previsto - niente di eccezionale che sta accadendo qui, è solo che si sospetta i dati potrebbero non essere validi.
• TryParse non utilizza la gestione per il controllo normale flusso eccezione

Fondamentalmente, andare con TryParse :)

Tra l'altro, il codice può essere riscritto come:

public static double GetDouble(object input, double defaultVal) 
{ 
    double parsed; 
    return double.TryParse(input.ToString(), out parsed)) ? parsed : defaultVal; 
} 

Answer 2

TryParse è più efficiente di prestazioni TryCatch saggio.

Answer 3

Avere i metodi Parse lanciano eccezioni su input errati erano un difetto di progettazione. Un input errato è previsto per il comportamento quando si acquisiscono dati da un utente. Il lancio di eccezioni è costoso, non è qualcosa che si desidera eseguire abitualmente nel codice.

Per fortuna, Microsoft ha realizzato l'errore e ha aggiunto i metodi TryParse. TryParse fa non incorre nel sovraccarico di eccezioni che generano input errati, ma il lato negativo è che deve restituire due pezzi di dati, quindi è un po 'scomodo da usare.

Ora, se non avessero creato l'implemetazione rotta Parse in primo luogo, TryParse si chiamerebbe Parse.

Answer 4

TryParse è più veloce e di solito meglio, ma vorrei suggerire l'approccio TryCatch nel quadro e back-end di programmazione, perché si può dare più informazioni al cliente circa l'errore:

public double GetAge() 
{ 
    try 
    { 
     var input = _dataProvider.GetInput(); 
     return Convert.ToDouble(input); 
    } 
    catch(Exception ex) 
    { 
     throw new MyBackendException(ex); 
    } 
}