Tengo un pequeño dilema con este código.
Se trata de verificar un valor dentro del segundo bucle for
. En cuanto ese valor se encuentre quiero salir de todo, retornando un dato.
Opción A
Si lo escribo así, teniendo una variable $mOption
que sea devuelta al final funciona bien, pero no está optimizado, porque el bucle padre sigue corriendo, puesto que el break
está dentro del bucle hijo:
function getOption ($arraySuperior,$mValue){
/*Variable de respuesta seteada a NULL por defecto*/
$mOption=NULL;
foreach ($arraySuperior as $key=>$subArray){
foreach ($subArray as $k=>$v){
echo "$key\n";
if ($v==$mValue){
/*
Sólo en este caso la variable cambiará de valor
siendo éste el de la clave siguiente a donde se encuentre a $mValue
*/
echo "Encontrado!\n";
$mOption=$subArray[$k+1];
/* Encontrado el valor, no seguimos iterando sobre el array*/
break;
}
}
}
return $mOption;
}
Prueba:
$mOption=getOption($arraySuperior,"site_name");
var_dump($mOption);
Salida:
0
0
Encontrado!
1
1
1
2
2
2
string(8) "FullAuth"
Como se puede ver, el dato fue encontrado en el índice 0
, pero sigue recorriendo el arreglo, dado que el break
está dentro del for
hijo. Por lo tanto, el código no está optimizado. Debería salir y no seguir imprimiendo los índices 1,2.
Opción B
Si escribo la función así, parece que está optimizado:
function getOption ($arraySuperior,$mValue){
/*Variable de respuesta seteada a NULL por defecto*/
//$mOption=NULL;
foreach ($arraySuperior as $key=>$subArray){
foreach ($subArray as $k=>$v){
echo "$key\n";
if ($v==$mValue){
/*
Sólo en este caso la variable cambiará de valor
siendo éste el de la clave siguiente a donde se encuentre a $mValue
*/
echo "Encontrado!\n";
return $subArray[$k+1];
}
}
}
//return $mOption;
}
Prueba:
Es exactamente la misma de antes:
$mOption=getOption($arraySuperior,"site_name");
var_dump($mOption);
Salida:
0
0
Encontrado!
string(8) "FullAuth"
Lo que me incomoda de esta segunda opción (que parece estar optimizada), es que no hay un control sobre lo que se retorna en la función Dicho de otro modo, sólo se retorna un valor cuando se encuentra algo, desde esta fila: return $subArray[$k+1];
. Pero en caso de no encontrar nada, ningún return
ocurre.
Ahora si pruebo con un valor que no exista, ocurre esto:
$mOption=getOption($arraySuperior,"fake");
var_dump($mOption);
Salida:
0
0
0
1
1
1
2
2
2
NULL
Se puede ver que al final el resultado es NULL
. Sé que PHP es algo permisivo y asume (según mi parecer), que si encuentra nada, el valor de la variable será NULL
. Pero no quiero acogerme a esa permisividad de PHP y quiero saber alguna forma de optimizar el código, de forma que pueda tener un control de lo que se retorne, como ocurre en la opción A, pero sin que tenga que seguir leyendo el array si se encuentra el dato.
return NULL
al final del bucle padre. Claro, tienes razón.