8

Estoy intentando generar una una función que esté parametrizada y devuelva un array con resultados en porcentaje siguiendo un orden. El resultado final debe ser un array que muestre el porcentaje de valores mayores a cero, iguales a cero y menores que cero.

Ejemplo:

masMenos(1,2,0,-1) 

debe dar por resultado:

0.5, 0.25, 0.25

Donde 0.5 es el porcentaje de números mayores a 0, 0.25 iguales a cero y 0.25 menores que cero.

¿Alguna sugerencia?

    var array=[1,2,0,-1];
    function masMenos(array){
      var cantidad=1;
      for(var i =0;i<array.length;i++){
        if(array[i]>0){
          cantidad=(array[i]*cantidad)/100;
        }else if(array[i]==0){
          cantidad=(array[i]*cantidad)/100;
        }else {
          cantidad=(array[i]*cantidad)/100;
        }
      }return cantidad;
    }
    console.log(masMenos(1,2,0,-1));

4
  • 1
    ¿Y cuál es el problema con el código que incluiste en la pregunta?
    – Alfabravo
    el 10 sep. 2019 a las 17:04
  • Que solo me muestra un resultado, necesito que me muestre tres. el 10 sep. 2019 a las 17:05
  • 1
    Pues tu variable cantidad es un humilde entero, así que ese es el primer cambio.
    – Alfabravo
    el 10 sep. 2019 a las 17:09
  • jajaja tienes razón jajaja @Alfabravo el 10 sep. 2019 a las 17:24

3 respuestas 3

7

Lo puedes hacer de forma simple con la función filter():

    function masMenos(array){
      may=array.filter(num => num>0);
      igu=array.filter(num => num==0);
      men=array.filter(num => num<0);
      return {
        mayores: may.length/array.length,
        iguales: igu.length/array.length,
        menores: men.length/array.length
      };
    }
    
    console.log(masMenos([1,2,-1,0,1,0,1,0]));

La función filter() filtra los valores del array según una función dada (en este caso si son positivos, negativos o ceros).

Este método si bien es simple y legible no es el mas optimo, ya que cada vez que se emplea dicha función se recorre el array una vez.

Una opción más optima y recomendable que emplee un solo bucle es la siguiente:

function masMenos(array, res=[0,0,0], n=array.length) {
    for (i in array)
      array[i]>0 ? res[0]++ : array[i]<0 ? res[2]++ : res[1]++
    return res.map(a =>a/n);
  }

  console.log(masMenos([1,2,-1,0,1,0,1,0]));

Desde luego las respuestas mas optimas pierden legibilidad.

Espero que la solución haya quedado mas clara, saludos.

8
  • 1
    Disculpa que no haya agregado mucho contenido, si gustas puedo agregar mas información. el 10 sep. 2019 a las 17:10
  • 1
    Te agradezco la respuesta, me ha servido bastante el 10 sep. 2019 a las 17:19
  • 1
    @FranqoBalsamo de nada, que tengas buen día. el 10 sep. 2019 a las 17:19
  • 2
    Bien optimizado el codigo, Excelente!! el 10 sep. 2019 a las 17:30
  • 3
    En realidad no es muy óptimo, porque se recorren tres veces el mismo array.
    – Emeeus
    el 10 sep. 2019 a las 17:48
3

Si quieres que te muestre tres resultados, necesitas declarar y retornar un array, no una variable simple. Puedes hacerlo asi:

var array=[1,2,0,-1];
    function masMenos(array){      
      var cantidad = [0,0,0]
      for(var i =0;i<array.length;i++){
        if(array[i]>0){
          cantidad[0]+=1;
        }else if(array[i]==0){
          cantidad[1]+=1;
        }else {
          cantidad[2]+=1;
        }
      }
      cantidad[0] = cantidad[0]/array.length;
      cantidad[1] = cantidad[1]/array.length;
      cantidad[2] = cantidad[2]/array.length;
      return cantidad;
    }
    console.log(masMenos(array));

2
  • 1
    Gracias por la ayuda el 10 sep. 2019 a las 17:25
  • 2
    Bien por usar solo una iteración, y no tres como la respuesta aceptada.
    – Emeeus
    el 10 sep. 2019 a las 17:47
3

Lo mas óptimo sería resolverlo en una sola iteración, y puede ser usando for, como ya han respondido, o forEach:

const arr = [1, 2, 0, -1];

let p = [0, 0, 0];
const inv = 1 / arr.length;

arr.forEach(e => e > 0 ? p[0] += inv : e < 0 ? p[2] += inv : p[1] += inv);

console.log(p);

se puede evitar dividir la cantidad por el total si uno hace la suma de las partes 1/largo, porque aplicamos la propiedad distributiva (1+2)/2 = (1/2)+(2/2) .

Usando reduce() es casi lo mismo:

const arr = [1, 2, 0, -1];
const inv = 1 / arr.length;

const res = arr.reduce((p, c) => {
  c > 0 ? p[0] += inv : c < 0 ? p[2] += inv : p[1] = inv;
  return p;
}, [0, 0, 0]);

console.log(res);

1
  • +1 por espíritu deportivo el 10 sep. 2019 a las 19:07

Tu Respuesta

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

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