evitar if anidados

[cucuru]

Hola a todos, no se como evitar hacer una serie de if anidados por un switch o similar (lo de los if anidados me parece una solución muy poco elegante y quiero evitarlo)

Se trata de ejecutar 5 funciones independientes que devuelven booleanos, si la primera es "true" se comprueba la segunda, si da "true" la tercera...

si alguna devuelve "false" se sale, vamos, es un switch... pero como son boolean, ¿cómo se podría hacer?

Gracias!!!

[Xerelo]

Si quieres que todas se ejecuten y luego comprobar si han sido correctas, por cada true suma 1 a una variable, si llega a 5 todo OK. Podrías hacer algo así.

i = i + (funcion1)?1:0

Si quieres que se ejecuten si la anterior se ha ejecutado correctamente, haces la llamada a la siguiente dentro de la primera función si el resultado de la primera es correcto.

Cada función devuelve false si no cumple, y el valor de la función que llama si cumple.

Algo parecido es lo que se usa en los interceptores de Struts2, por ejemplo.

[cucuru]

hola Xerelo, muchas gracias, la primera opción no es lo que quiero, mi programa hará cosas distintas para cada false y otra cosa diferente si todas son ciertas.

La segunda opción, al fin y al cabo son if anidados, ¿Es esto a lo que te refieres?

Código:

public boolean funcion1{

      if(condicion){
            boolean b = función2
      }else {
            return false
       }

}

public boolean funcion2{

      if(condicion){
            boolean b = función3
      }else {
            return false
       }
}

Saludos

[Xerelo]

Creo que sí, yo la haría así

Código Javascript:

public boolean funcion1{
 
boolean b = false;
      if(condicion){
            b = funcion2
 }
return b;
}
 
public boolean funcion2{
 
boolean b = false;
      if(condicion){
            b = funcion3
 }
return b;
}
 

Piensa que lo de if anidados se refiere más a la forma del código, si haces un if dentro de otro y así sucesivamente, es difícil de entender de un vistazo y las llaves son engorrosas si quieres cambiar algo.

Así en tu main, sólo vas a tener una llamada a una función, en vez de un tochazo de ifs. Mucho más claro, y limpio.

Si tuvieses que indicar cual es la que ha fallado, sería distinto.

Con la misma estructura, podrías hacer que si es incorrecta devuelva su número en negativo. Si llega al final tendrás un 5, y si falla un -X.

[cucuru]

hola! si que tengo que indicar el fallo, creo que lo voy a hacer con estructura if-else if - else, ¿hay algo más elegante?

Gracias!

[nup_]

Hola:

Por qué no pruebas con lo obvio:

if (funcion1() && funcion2() && funcion3() && funcion4() && funcion5())
// algo
else
// otro algo

Al final hace lo q deseas.

saludos;

Nup_


PD: Upps... no habia visto la parte en q tienes q indicar cual falló. Si es así esa forma no te va a servir.

[White_King]

jaja, simon, son puros ands... XD

[cucuru]

Gracias! pero claro tengo que saber cual falló! al final he optado por el if-else if- else