He tenido una discusión con un compañero de trabajo sobre romper una return
declaración y la declaración que calcula el valor de retorno en dos líneas.
Por ejemplo
private string GetFormattedValue()
{
var formattedString = format != null ? string.Format(format, value) : value.ToString();
return formattedString;
}
en lugar de
private string GetFormattedValue()
{
return format != null ? string.Format(format, value) : value.ToString();
}
En cuanto al código, realmente no veo un valor en la primera variante. Para mí, este último es más claro, particularmente para los métodos que son cortos. Su argumento fue que la variante anterior es más fácil de depurar, lo cual es un mérito bastante pequeño, ya que VisualStudio nos permite una inspección muy detallada de las declaraciones, cuando la ejecución se detiene debido a un punto de interrupción.
Mi pregunta es, si todavía es un punto válido para escribir código menos claro, ¿solo para hacer más fácil la depuración? ¿Hay más argumentos para la variante con el cálculo dividido y la return
declaración?
private string GetFormattedValue() => string.Format(format ?? "{0}", value);