Tengo entendido que meter en mitad de una función un return no es muy
buena idea
Es una recomendación para que el código sea más legible, se la llama "punto único de salida" (single exit point). Por ejemplo, si modificas una función y haces cambio, puede ser que el cambio
Por ejemplo, ahora mismo he depurado una función de inicialización y, si el valor de la configuración era null
, hacía un return null;
en medio del código porque no había que hacer nada. Después añadí más código de inicialización que se tenía que ejecutar siempre y me dejé el return
.
También puede afectar a la facilidad de depuración.
Las alternativa más sencilla es cambiar el flujo para que la comprobación implique la ejecución, es decir de:
function hola() {
if (valor == 0) {
return false;
}
// Hacer hola();
return loquetoque;
}
pasa a :
function hola() {
if (valor != 0) {
// hacer hola();
return loquetoque;
}
return false;
}
o variedades similares (poner el return false
en un else
, poner el valor a devolver en una variable y solo hacer un return
, etc.)
En todo caso, lo primero es la claridad del código, no es una regla que deba seguirse ciegamente. En concreto, hacer una serie de return
al principio del código según comprobaciones de parámetros puede ser una idea mejor.
function hola(par1, par2, par3, par4) {
if (par1 == 0) {
return false;
}
if ((par2 == 1) && (par4 ==5)) {
return false;
}
if (par3 == 2) {
return false;
}
return loquetoque;
// hacer hola
}
antes que:
function hola(par1, par2, par3, par4) {
if ((par1 != 0) || ((par2 != 1) && (par4 != 5)) || (par3 != 1) {
// hacer hola
return loquetoque;
}
return false;
}