1

Tengo un array tal que:

[
  {
    423435235: { data: .... }
  },
  {
    6548u9645: { data: .... }
  },
  ...
]

Los números son ids dinámicas, y tengo una función que recibe una id y una {data}, que debe de:

  • Buscar por el array una key determinada
  • Si no encuentra un elemento del array que cumple la condición anterior, pushear este elemento a un estado en React.

La función es la siguiente:

const setToState = (id, data) => {
    const exist = widgetValues.filter(element => {
      let isOn = null;
      for (let key in element){
        if(element[key] === id){
          isOn = true;
        }
      }
      return isOn ? element : null
    }) 
    if(exist.length === 0){
      setWidgetValues(oldValues => [...oldValues, {[id]: data }])
    }
  }

Pero no funciona el filter... ¿Qué estoy haciendo mal?

1
  • puedes intentar algo como let idABuscar = 423435235; const a = resultado.filter((item)=> { return item[idABuscar]; }); el 11 dic. 2019 a las 16:19

2 respuestas 2

4
const setWidgetValuesF = (id, data) => {
    const exist = widgetValues.filter(element => {
      let isOn = null;
      for (let key in element){
        if(key === id){
          isOn = true;
        }
      }
      return isOn ? element : null
    }) 
    if(exist.length === 0){
      setWidgetValues(oldValues => [...oldValues, {[id]: data} ])
    }
  }

He dado con el fallo! muchas gracias @jackNavaRow

Era un fallo humano, estaba buscando mal en el For in, necesitaba buscar por la key, no por el valor!

6
  • ¿cuál era el fallo?, ayudaría mucho expliques que modificaste
    – user128299
    el 11 dic. 2019 a las 16:31
  • He puesto la solución. En el for in, estaba mirando el valor, no la key, entonces nunca coincidía :/
    – Adrián
    el 11 dic. 2019 a las 16:35
  • 1
    por favor esa explicación si la puedes añadir a tu respuesta ayudaría mucho
    – user128299
    el 11 dic. 2019 a las 16:38
  • Modificado! muchas gracias por las sugerencias :)
    – Adrián
    el 11 dic. 2019 a las 16:39
  • Por favor cuando te sea posible considera marcar tu respuesta como aceptada, de ese modo queda indicado que ya esta solucionada
    – user128299
    el 11 dic. 2019 a las 16:40
3

El problema de usar .filter() es que este va a recorrer siempre todos los registros, siendo que con la primer coincidencia debería dejar de buscar, para solucionar eso sería mejor usar .some() que va a retornar true en la primer coincidencia. Como lo que tenemos que buscar está dentro de un objeto dentro de un array, podemos usar dos some anidados, o bien .some y .hasOwnProperty de manera de asegurarnos la mínima cantidad de iteraciones. Ejemplo:

let widgetValues = [{423435235: { data: "" }},{65489645: { data: "" }}];

const setWidgetValuesF = (id, data) => {
  if(!widgetValues.some(e=>e.hasOwnProperty(id))){
    widgetValues.push({[id]: data });
  }  
}

setWidgetValuesF("no existe", "datos");
console.log(widgetValues);
setWidgetValuesF("423435235", "datos");
console.log(widgetValues);

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.