¿Las condiciones deberían ser positivas o negativas?

Question

Lo siento si el título es bastante ambiguo, no estaba seguro de cómo decirlo.

¿Es mejor expresar una condición tal que el resultado que no desea entre en la instrucción if luego salga de la función o debería probar el resultado que quiero y seguir la declaración con mi código.

Quizás algunos ejemplos ayudaría:

Lo que quiero decir mediante pruebas de resultado negativo:

if(myObject == null) { 
    return; 
} 

//do whatever with myObject 

Lo que quiero decir mediante pruebas de resultado positivo:

if(myObject != null) { 
    //do whatever with myObject 
} 

En este momento, si alguien puede decirlo mejor que yo, por favor hazlo.

Answer 1

Personalmente prefiero el primer método para verificar si el objeto es nulo y luego regresa inmediatamente. Permite que el "código real" permanezca sin defectos, lineal y puede evitar muchas sentencias if anidadas, que me parecen más legibles. De lo contrario, ambas formas son válidas y tendrán el mismo resultado. Elija el que mejor se adapte a su situación (que puede depender de cualquier otra cosa o de las declaraciones).

He aquí un buen ejemplo:

if (object1 == null) { 
    return; 
} 
// do some stuff 
if (object2 == null) { 
    return; 
} 
// do some stuff 
if (object3 == null) { 
    return; 
} 

contraposición a:

if (object1 != null) { 
    // do some stuff 
    if (object2 != null) { 
     // do some stuff 
     if (object3 != null) { 
      // do some stuff 
     } 
    } 
} 

encuentro el primer para ser mucho más legible.

Answer 2

Cuando hay una acción válida que se puede tomar para satisfacer una condición positiva, como registrar que un conjunto de resultados está vacío o que no se asignó una variable, entonces es mejor usar condiciones positivas. Las API pueden ser útiles aquí, como Apache Commons StringUtils isNotBlank(), cuando está probando cadenas. Sin embargo, a veces lo más claro es realizar una prueba negativa, por ejemplo, solo permitiendo que el proceso proceda donde una variable no es nula.