0

he traido la informacion de una API cuyo json me trae un array con objetos de jugadores y cada jugador tiene una altura, entonces estoy tratando de sumar la altura de cada jugador para que solo me aparezcan los jugadores cuya suma sea igual a lo que digite el usuario como entrada. Por ejemplo si el usuario digita 150, me tiene que salir todos los jugadores cuya suma de 150. Creo tener el codigo bien pero el operador +=, no entiendo por que me presenta un error y me dice: que se esperaba ",". Este es el código compañeros.

private getAllPlayers(){
    return this.http.get<Jugadores[]>('https://mach-eight.uc.r.appspot.com')
    .pipe(
      map(item => item.values)
    )
    .subscribe( (data: any) => {
      this.datos= data;

      //En este array quiero almacenar los objetos cuya altura sumen 140
      let newJugadores:Jugadores[]=[];

      //Filtramos y almacenamos los objetos cuya altura son menores a 70 (item.h_in es la altura)
      const Jugadores= this.datos.filter(item=> parseInt(item.h_in) <= 100);

      //Recorremos posicion por posicion los objetos de ese array
      for(let i=0; i < Jugadores.length; i++){

        //Trato de ir sumando la altura de cada objeto, pero el operador += me da error
        let suma += parseInt(Jugadores[i].h_in);
        //Si la suma igual a 140
        if(suma === 140){

          //Pujamos esos objetos al array
          newJugadores.push(Jugadores[i]);
        }
      }
      console.log(newJugadores);
    });
  } 

Está es la captura del error: introducir la descripción de la imagen aquí

Quiero aclarar que la propiedad de la altura del jugador osea "h_in" viene en string y por eso lo estoy parseando como entero para poder hacer la suma.

Si alguno me pudiera ayudar, se los agradecería de ante mano compañeros, no se si mi código esta bien o en que pudiera estar fallando.

Edit: Noto que se estan agragando todos los jugadores cuyo filtro, filtraba que tuvieran una altura menor a 100, más no los jugadores cuya suma de las alturas sea 140. Observa que esos 435 serían los jugadores con una altura menores a 100 y no solo los jugadores cuya suma debe ser 140: introducir la descripción de la imagen aquí

Al console.log(newJugadores); le agregue el texto de concatenacion: console.log("Objeto jugadores: "+ newJugadores);

2
  • ¿Por que declaras suma en cada ciclo? Suma debería estar inicializada a cero fuera del ciclo. Por otro lado, no entiendo lo que quieres hacer bien ¿Todos los jugadores cuya suma sea 150? O sea, ¿empezar a sumar gente random ahí hasta llegar a 150?
    – Jaime
    Commented el 16 abr. 2022 a las 6:48
  • Con dos personas que sumes ya llegas a 150 teniendo en cuenta esos números. Explica bien lo que quieres porque no hace sentido. Ahora mismo solo pondrás una sola persona en el arreglo de newJugadores si llegas ("por casualidad divina") a sumar 140, pero bien puedes nunca llegar a 140 y saltar a 180 por ejemplo con dos personas que midan 90. Por otro lado, los comentarios y el código andan desfasados totalmente, filtras por 100 y dices en comentarios que por 70. Dices que 150 y tienes 140.
    – Jaime
    Commented el 16 abr. 2022 a las 6:58

1 respuesta 1

2

Esto además debe contribuir a depurar tu código: En el parseo a int que intentas agregar, veo dos veces el mismo valor:

let suma += parseInt(Jugadores[i].h_in + Jugadores[i].h_in);

Revisa, eso sería una concatenación de la misma string, si esa construcción estuviera completa.

Debe ser algo similar a::

let suma += parseInt(Jugadores[i].h_in);

Finalmente, debes cambiar esto:

let suma = 0; // antes del for

y la instrucción queda:

suma += parseInt(Jugadores[i].h_in);

Ya que: let declara variables y opcionalmente permite asignarles un valor El operador += no es aceptable en una declaración let. Solo puedes usar =

18
  • Es cierto brother, habia una concatenacion en el parseo pero ya lo reduje como lo propusiste y me sigue dando el mismo error en esa linea de codigo. En cuanto al item, me tira error si lo agrego, creo que no es necesario porque ese item solo pertenece al filter. Pero volviendo a al tema del operador no entiendo el error si se supone que estoy aculando enteros al hacer ese casting
    – Hugo
    Commented el 16 abr. 2022 a las 6:39
  • Funciono!, inicializar el let antes del for me quito el error DEVX75, muchas gracias de verdad. Ahora esta pasando que el push no me esta añadiendo los objetos de los jugadores cuya altura sumada da 140. El console.log(newJugadores) no me esta imprimiendo nada del array como si el push no le estuviera añadiendo nada. Sabes que podría ser?
    – Hugo
    Commented el 16 abr. 2022 a las 7:03
  • 1
    Yo mejoraría dos cosas, cambiar ese nombre del const y subsiguientes. También esto: if(suma === 140) debe ser simplemente if(suma == 140) ya que === es otra cosa, implica que el tipo y el valor son iguales e idénticos
    – DEVX75
    Commented el 16 abr. 2022 a las 7:07
  • 1
    No, no es así, tienes una suma CONTINUA sobre todos los jugadores, por eso si los n primeros no te dan el valor buscado de manera exacta, jamás se agregarán al resultado. Además debes ser específico cuantos jugadores son validos que puedan sumarse, pueden ser 3, 4, 5? -> Además tu código está diseñado para guardar solo el jugador con el que completas la suma exacta, y no todos los jugadores que participan de esa suma
    – DEVX75
    Commented el 16 abr. 2022 a las 7:37
  • 2
    @Hugo yo te aconsejo pensar un poquito qué es lo que quieres. La consigna "los jugadores que sumen 140" me parece vaga y sin sentido. Si dijeras algo como "los pares de jugadores que suman 140" podrías hacer algo cómo esto playcode.io/886090. Nota que el resultado es un array de pares donde cada par suma 140. Piensa el problema de manera lógica, porque creo que ni tu mismo sabes lo que quieres. El filtrado si lo estás haciendo correcto.
    – Jaime
    Commented el 16 abr. 2022 a las 8:02

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.