¿Qué es mejor? Múltiples instrucciones if, o una si tiene varias condiciones

Question

Para mi trabajo, tengo que desarrollar una pequeña aplicación Java que analiza archivos XML muy grandes (~ 300k líneas) para seleccionar datos muy específicos (usando Pattern), así que estoy tratando de optimizarlo un poco Me preguntaba qué era mejor entre estos 2 fragmentos:

if(boolean_condition && matcher.find(string)) 
{ 
    ... 
} 

O

if(boolean_condition) 
{ 
    if(matcher.find(string)) 
    { 
     ... 
    } 
} 

Más precisiones:

• Estas sentencias if se ejecutan en cada iteración dentro de un bucle (~ 20k iteraciones)
• El boolean_condition es un boolean calcu RELAClONADAS en cada iteración mediante una función externa
• Si el boolean se establece en false, no necesito para probar la expresión regular para partidos

Gracias por su ayuda

Answer 1

Una regla de oro sigo es a Evite anidar todo lo que pueda. Pero si es a costa de hacer que mi única condición sea demasiado compleja, no me importa anidarla.

Además, está utilizando el operador de cortocircuito &&. Entonces, si boolean es falso, ¡ni siquiera intentará hacer coincidir!

Así,

if(boolean_condition && matcher.find(string)) 
{ 
    ... 
} 

es el camino a seguir!

Answer 2

El primero. Intento evitar si anidando así, creo que es un código de estilo pobre/feo y el & & hará un cortocircuito y solo probará con matcher.find() si el valor booleano es verdadero.

Answer 3

Ambos caminos están bien, y la segunda condición no se probará si la primera es falsa.

Utilice el que hace que el código sea más legible y comprensible. Solo por dos condiciones, la primera es más lógica y legible. Puede que ya no sea así con 5 o 6 condiciones relacionadas con &&, || y !.

Answer 4

Java usa un cortocircuito para esos operadores booleanos, por lo que ambas variaciones son funcionalmente idénticas.Por lo tanto, si el boolean_condition es falso, no continuará con el

En última instancia, se trata de algo que resulta más fácil de leer y depurar, pero el anidamiento profundo puede ser difícil de manejar si termina con una gran cantidad de los apoyos al final

Una manera de mejorar la legibilidad, si la condición de ser más largo es simplemente dividirlo en varias líneas:

if(boolean_condition && 
    matcher.find(string)) 
{ 
    ... 
} 

la única opción en ese punto es si poner el & & y || al final de la línea anterior o al comienzo de la corriente.

Answer 5

Los dos métodos siguientes:

public void oneIf(boolean a, boolean b) 
{ 
    if (a && b) 
    { 
    } 
} 

public void twoIfs(boolean a, boolean b) 
{ 
    if (a) 
    { 
     if (b) 
     {  
     } 
    } 
} 

producen el mismo código de bytes exacto para el cuerpo del método por lo que no habrá ninguna diferencia de rendimiento que significa que es puramente una cuestión de estilo que se utiliza (personalmente prefiero la primer estilo).

Answer 6

En términos de rendimiento, son lo mismo.

• Pero incluso si no estaban

lo que es casi seguro que dominar el tiempo en este código es matcher.find(string) porque es una llamada a la función.

Answer 7

Tiendo a ver demasiados & & y || encadenados en una sopa lógica y son a menudo la fuente de errores sutiles.

Es muy fácil agregar otro & & o || a lo que crees que es el lugar correcto y romper la lógica existente.

Debido a esto como regla general trato de no utilizar ninguno de ellos para evitar la tentación de agregar más a medida que cambian los requisitos.

Answer 8

Si te gusta la conformidad a Sonar rule squid:S1066 debe colapsar si las declaraciones para evitar la advertencia, ya que afirma:

plegable "if" deberían fusionarse