Skip to main content
formatear la primera frase como una cita
Origen Enlace
user7176
user7176

Tengo entendido que meter en mitad de una función un return no es muy buena idea

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;
 }

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;
 }

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;
 }
Origen Enlace
SJuan76
  • 11k
  • 5
  • 20
  • 31

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;
 }