8

El siguiente código funciona y tiene 4 condiciones pero quiero simplificarlo ya que son en realidad 150 condiciones, ¿ habrá alguna otra manera de reducir este algoritmo y que no sean 150 else if?

for (var j = 0; j < $scope.cants.length; j++) {
  if ($scope.cants[j].type === "bases") {       
    $scope.bases += $scope.cants[j].quantity;
  } else if ($scope.cants[j].type === "proteins") {     
    $scope.proteins += $scope.cants[j].quantity;
  } else if ($scope.cants[j].type === "veggies") {      
    $scope.veggies += $scope.cants[j].quantity;
  } else if ($scope.cants[j].type === "sauces") {       
    $scope.sauces += $scope.cants[j].quantity;
  } 
  $scope.componentsPlate = $scope.proteins +  $scope.veggies + $scope.sauces +     $scope.dryfruits + $scope.bases + $scope.proteinsExtra +        $scope.veggiesExtra + $scope.saucesExtra + $scope.dryfruitsExtra +  $scope.basesExtra
}
15
  • 3
    Cual es el scope de las variables que necesitas crear? Estan dentro de una funcion, las necesitas globalmente? Estas dentro de un navegador y pueden estar dentro de window? Commented el 19 ene. 2016 a las 16:53
  • 4
    Ojo es importare decir que no es buena idea hacer esto para un programa real, ya que dará problemas para leer el código y su depuración a futuro.
    – Wilfredo
    Commented el 19 ene. 2016 a las 16:58
  • 3
    Y por casulidad como piensas usar luego esas variables? Te funciona si no son variables sino propiedades dentro de una variable inicializada al comienzo de la función? Commented el 19 ene. 2016 a las 16:58
  • 2
    Puedes crear el código que has intentado para tenerlo como referencia y poder completarlo o corregirlo? Además nos podrías explicar que es lo que estás tratando de hacer? Es posible que haya alguna otra solución más fácil Commented el 19 ene. 2016 a las 17:02
  • 3
    @DanielHernández yo creo que es mejor que coloques todo el contexto de la pregunta, con algo de código para ayudarte mejor.
    – Wilfredo
    Commented el 19 ene. 2016 a las 17:02

3 respuestas 3

9

La sintaxis:

$scope.nombreDePropiedad

Es equivalente a:

$scope["nombreDePropiedad"]

Lo cual puedes aprovechar para utilizarlo de la sgte forma:

for (var j = 0; j < $scope.cants.length; j++) {
    $scope[$scope.cants[j].type] += $scope.cants[j].quantity;
}

De esta manera no necesitas hacer ningún if o switch ya que el miembro correcto del objeto $scope será seleccionado a partir del string que le pases entre corchetes.

1

Lo que pretendes hacer es totalmente contrario a lo que se recomienda hacer.

El objetivo de almacenar datos en variables con un nombre fijo en un namespace es referirse a ellas cuando sea preciso conociendo el nombre de la variable.

Cuando el nombre asignado al dato no se conoce, se emplean los diccionarios (arrays) en que se pasa el nombre asociado al tipo de esto que se pretende usar, en caso de que éste se encuentre almacenado en una variable y no se conozca su nombre a priori.

Lo que tú pretendes hacer no es solo poco recomendable, además de inmantenible al escalar el código, sino que va contra los principios de la programación — lo que debes hacer es replantearte cómo debes estructurar tus datos, para mejorar la legibilidad y calidad de tu código.

Si nos comentas por qué pretendes usar variables a toda costa, quizá podamos ayudarte, pero sin más aclaraciones, estas son mis recomendaciones.

Edición posterior

Visto tu problema, la solución planteada en la otra respuesta es la mejor.

0

Quizas usted pueda barajar la posibilidada de usar un switch, dado que usted menciona que son unos 150 casos, creo que el uso del switch mejor el rendimiento con respecto a 150 casos, si fueran unos poco quizas esto no se notaria pero para 150 me plantearia el uso del switch.

Por otro lado si usted decide usar if else, intentaria situar los casos mas comunes al principio del if.

P.D: Usted dice reducir el algoritmo, no se si tambien incluye el hacer cambios para la eficiencia del mismo

3
  • El problema no es de sintaxis, sino de implementación Commented el 19 ene. 2016 a las 17:38
  • @JuanRocamonde de acuerdo borro la respuesta. Saludos Commented el 19 ene. 2016 a las 17:40
  • No es necesario, simplemente aclaro para los lectores. Puede servir de ayuda. Commented el 19 ene. 2016 a las 17:42

Tu Respuesta

By clicking “Publica tu respuesta”, you agree to our terms of service and acknowledge you have read our privacy policy.

¿No es la respuesta que buscas? Examina otras preguntas con la etiqueta o formula tu propia pregunta.