Optimizar este codigo

[juanla]

Bueno, tengo una funcion que me cree para probar una cosilla pero una vez listo me gustaría optimizarla de forma que no tenga que ejecutar 2 veces cada una de las funciones de dentro, os la dejo aquí:

Código PHP:

function devolver($url){
   if( funcion1($url)!=""){
       return funcion1($url);
       break;
   }elseif(funcion2($url)!=""){
       return funcion2($url);
       break;
   }elseif(funcion3($url)!=""){
       return funcion3($url);
       break;
   }elseif(funcion4($url)!=""){
       return funcion4($url);
       break;
   }else{ 
       return "";
   }
    
}
 

Me surgio necesidad de llamar a esa funcion y vaya ejecutando funciones hasta que una devuelva algo distinto de nulo, tal y como esta la aplicación funciona perfectamente, pero como podréis ver, no me gustaría repetir la función ahi.. estas cosas son las que pasan programando para probar, luego vas agrandando y agrandando y te quedas así :)

[kiko's]

Código PHP:

      function devolver($url){ 
       if( ($funcion1 = funcion1($url)) != ""){ 
           return $funcion1; 
           break; 
       }elseif( ($funcion2 = funcion2($url)) !=""){ 
           return $funcion2; 
           break; 
       }elseif( ($funcion3 = funcion3($url)) !=""){ 
           return return $funcion3; 
           break; 
       }else{ 
           return ""; 
       }      
    }  
   

¿Esto?

[maycolalvarez]

utiliza recursión.

[juanla]

Iniciado por kiko's

Código PHP:

      function devolver($url){ 
       if( ($funcion1 = funcion1($url)) != ""){ 
           return $funcion1; 
           break; 
       }elseif( ($funcion2 = funcion2($url)) !=""){ 
           return $funcion2; 
           break; 
       }elseif( ($funcion3 = funcion3($url)) !=""){ 
           return return $funcion3; 
           break; 
       }else{ 
           return ""; 
       }      
    }  
   

¿Esto?

Bueno eso lo había pensado, pero creo que queda bastante feo no? Pienso que tiene que haber una forma más sencilla de lograrlo.. aunque de momento la que va ganando es esa, para mi aplicación probablemente en algunos casos me ahorro casi la mitad de tiempo. :)

[GrupoC]

Código PHP:

function devolver($url){ 
       if(($funcion1 = funcion1($url))!="") return $funcion1;
       elseif(($funcion2 = funcion2($url))!="") return $funcion2;
       elseif(($funcion3 = funcion3($url))!="") return $funcion3;
       else return "";
}
 

No sirve para nada el break.

[mogurbon]

se podrian ver las funciones?? es decir el codigo de funcion1 funcion 2 funcion n?

[juanla]

Las funciones van bien por separado, su funcion es leer si una cadena existe en la url que se le ha pasado, en ese caso la devuelve, sino hace un return ""

Gracias a todos por responder, me quedaré pues con esa version de 6 lineas que no repite las funciones :D

Un saludo!

[kiko's]

Hola

Yo creo que no he entendido bien que querías/quieres hacer. La versión de GrupoC es exactamente igual que la mía, no ha optimizado nada, salvo que por optimizar te refirieras a tener un código más compacto, porque es lo único que hemos hecho. Sin el contexto del programa es difícil optimizar de verdad. :)

Saludos.

[juanla]

Si, es cierto es el mismo código, por eso dije que me quedaba con la versión de 6 lineas, que ya hace lo mismo y ocupa algo menos (al margen que sea la misma), luego os he dado reputación a los 2 porque me habeis respondido cosas utiles :)

Lo que yo quería optimizar es hacer sólo 1 llamada a la función, no 2 y lo he conseguido con tu ayuda, y con la ayuda de GrupoC lo he conseguido tener incluso en menos lineas.

De veras muchas gracias a todos los que han posteado aquí, espero que nadie se haya molestado.

[truman_truman]

Que alguien me corrija si esto es incorrecto

Código PHP:

function devolver($url){
$res = ( funcion1($url)!="") ? funcion1($url) : '';
$res = ( funcion2($url)!="") ? funcion2($url) : '';
$res = ( funcion3($url)!="") ? funcion3($url) : '';
$res = ( funcion4($url)!="") ? funcion4($url) : '';
return $res;
}