Ho avuto una discussione con un collega sulla rottura di return
un'istruzione e l'affermazione che calcola il valore di ritorno in due righe.
Per esempio
private string GetFormattedValue()
{
var formattedString = format != null ? string.Format(format, value) : value.ToString();
return formattedString;
}
invece di
private string GetFormattedValue()
{
return format != null ? string.Format(format, value) : value.ToString();
}
Per quanto riguarda il codice, non vedo davvero un valore nella prima variante. Per me, quest'ultimo è più chiaro, in particolare per i metodi così brevi. La sua argomentazione era che la prima variante è più facile da eseguire il debug, il che è un merito piuttosto piccolo, dal momento che VisualStudio ci consente un'ispezione molto dettagliata delle dichiarazioni, quando l'esecuzione viene interrotta a causa di un punto di interruzione.
La mia domanda è, se è ancora un punto valido per scrivere il codice in modo meno chiaro, solo per rendere più facile il debug uno sguardo? Ci sono ulteriori argomenti per la variante con il calcolo diviso e l' return
istruzione?
private string GetFormattedValue() => string.Format(format ?? "{0}", value);