Declaraciones anidadas 'if' - 'else'

Question

Tengo un código que es muy desordenado con el if - else si comprueba que está haciendo. La cantidad de ramificaciones y ramificaciones anidadas es bastante grande (más de 20 si - else si y anidado también). Está haciendo que mi código sea más difícil de leer y probablemente sea un cerdo de rendimiento. Mis aplicación comprueba una gran cantidad de condiciones que recibe por parte del usuario y por lo que la aplicación debe comprobar todo el tiempo para diferentes situaciones, por ejemplo:

Si el texto de cuadro de texto no es 0, continúe con el siguiente ...

if ((StartInt != 0) && (EndInt != 0)) 
{ 

Y entonces aquí se comprueba si el usuario ha elegido fechas:

if ((datePickerStart.SelectedDate == null) || (datePickerEnd.SelectedDate == null)) 
{ 
    MessageBox.Show("Please Choose Dates"); 
} 

aquí, si las datepickers no son nulos continúa con el código ...

else if ((datePickerStart.SelectedDate != null) && (datePickerEnd.SelectedDate != null)) 
{ 
    // CONDITIONS FOR SAME STARTING DAY AND ENDING DAY. 
    if (datePickerStart.SelectedDate == datePickerEnd.SelectedDate) 
    { 
     if (index1 == index2) 
     { 
      if (StartInt == EndInt) 
      { 
       if (radioButton1.IsChecked == true) 
       { 
        printTime3(); 
       } 
       else 
       { 
        printTime(); 
       } 
      } 

Esto es solo una pequeña parte de los cheques que se están realizando. Algunos de ellos son funcionales y otros son para validación de entrada.

¿Hay alguna manera de hacerlo más legible y menos de un cerdo de rendimiento?

Answer 1

No es el de un cerdo de rendimiento. Una gran publicación de blog sobre cómo solucionar esos problemas comunes es Flattening Arrow Code.

Answer 2

Una forma es refactorizar encapsulando complejas condiciones de la siguiente manera:

public bool DateRangeSpecified 
{ 
    get 
    { 
    return (datePickerStart.SelectedDate != null) 
      && 
      (datePickerEnd.SelectedDate != null) 
      && StartInt != 0 && EndInt != 0; 
    } 
} 

y luego utilizar estas propiedades "condición" de fachada

Answer 3

que veo aquí algo de mezcla en la validación. Trate de mover uno campos de los demás, y validar por separado, algo como esto:

if (StartInt == 0 || EndInt == 0) 
{ 
    MessageBox.Show("Please Choose Ints"); 
    return; 
} 
if (datePickerStart.SelectedDate == null || datePickerEnd.SelectedDate == null) 
{ 
    MessageBox.Show("Please Choose Dates"); 
    return; 
} 

En este enfoque siempre se dirá al usuario lo que hizo mal, y su código es mucho más simple.

Más información de Jeff's blog

Answer 4

Utilice la sentencia return para detener la ejecución de un bloque.

Por ejemplo,

void Test() 
{ 
    if (StartInt==0 || EndInt==0) 
    { 
     return; 
    } 

    if (datePickerStart.SelectedDate == null || datePickerEnd.SelectedDate == null) 
    { 
     MessageBox.Show("Please Choose Dates"); 
     return; 
    } 
} 

Answer 5

Alguna ligera refactorización hace que sea más fácil de leer a mis ojos. Eliminé corchetes extraños y consolidé varias declaraciones de IF que son realmente lógicas AND.

if (StartInt == 0 || EndInt == 0)  
    return; 
if (datePickerStart.SelectedDate == null || datePickerEnd.SelectedDate == null) 
{ 
    MessageBox.Show("Please Choose Dates"); 
    return;   
} 
if (datePickerStart.SelectedDate != null 
    && datePickerEnd.SelectedDate != null 
    && datePickerStart.SelectedDate == datePickerEnd.SelectedDate 
    && index1 == index2 
    && StartInt == EndInt) 
{ 
    if (radioButton1.IsChecked == true) 
     printTime3(); 
    else 
     printTime(); 
} 

Answer 6

Puede definir sus propios predicados o funciones genéricas con nombres significativos y encapsular su lógica en los mismos.

Aquí es un ejemplo de código para algunos predicados:

public Predicate<DateTime> CheckIfThisYear = a => a.Year == DateTime.Now.Year; 
public Func<DateTime, int, bool> CheckIfWithinLastNDays = (a, b) => (DateTime.Now - a).Days < b; 

Ahora se puede escribir fácilmente en su código

if (CheckIfThisYear(offer) && CheckIfWithinLastNDays(paymentdate,30)) ProcessOrder(); 

Considere el uso de los delegados genéricos, como Func<> y Delegate<> para escribir pequeños bloques de su condiciones usando lambda -expresiones- salvará el espacio y hará que su código sea mucho más legible para el ser humano.