2

Tengo un input de tipo texto ahí se introduce un id para verificar si este exististe o no.

list.forEach(function(a) {
    if (b == a["id"]) {
        g["setstatus"](true, a["id"], f);
        return c(true)
        alert("Correcto!");
    }
    g["setstatus"](false, null, f);
    return c(false)
});

Los datos que recibo dentro del foreach es de tipo JSON

[
  {"id":2,"email":"[email protected]","name":"User example 3","c":false},
  {"id":2,"email":"[email protected]","name":"User example 2","c":false},
  {"id":1,"email":"[email protected]","name":"User Example 1","c":false}
]

El problema es al momento de verificar b == a["id"], en este caso b es la variable que recibe el valor del usuario.

Cuando pongo 3 o 2 en el input esta no reconoce, pero cuando pongo 1 recién reconoce, se que es por el foreach, También realicé de esta manera:

a.forEach(function(a) {
    if (b == a["id"]) {
        g["setstatus"](true, a["id"], f);
        alert("Correcto!");
        return c(true)
    }

});
g["setstatus"](false, null, f);
return c(false)

Pero lastimosamente tampoco me funcionó.

11
  • No comprendo bien qué es lo que quieres realizar con tu programa?
    – Yikarus
    Commented el 22 may. 2017 a las 21:55
  • ¿Cuál es el valor de b?
    – Rene Limon
    Commented el 22 may. 2017 a las 21:58
  • Al momento de comprar b == a["id"] esta se tiene que activar "g["setstatus"](true, a["id"], f); return c(true)", el problema es con el foreach ya que esta recorriendo todos los registros y solo me reconoce el ID = 1 y no los demas 2 o 3. Commented el 22 may. 2017 a las 21:58
  • El valor de B llega de un formulario INPUT. Commented el 22 may. 2017 a las 21:59
  • @JhosselinGiménez de la sentencia dentro del foreach me parece correcto que solo haga el g["setstatus"](true, a["id"], f); return c(true) en la primera iteración si b = 1. Si b = 2 debería hacer el setStatus en la segunda iteración, etc.
    – learnercys
    Commented el 22 may. 2017 a las 22:11

1 respuesta 1

3

La razón por la cuál tu sentencia está fallando para valores dónde b!=1 es porque tu condición no hace ningún break al foreach y por lo tanto siempre evaluará los otros valores(que estarían dando falso)

Ejemplo: b=3

# primer ciclo, a = {"id":3,"email":"[email protected]","name":"User example 3","c":false}
# se activa el botón porque la sentencia b = a["id"], se cumple

# segundo ciclo, a = {"id":2,"email":"[email protected]","name":"User example 2","c":false}
# se desactiva el botón porque la sentencia deja de cumplirse.

La razón por la que con b=1 siempre queda activado es debido a que es el último item en tu lista y por lo tanto no existe un siguiente que desactive el botón.

Una posible solución sería dejar de usar el foreach y en su lugar hacer:

var exist = list.some(function(a) {
  return b == a["id"]; // or b == a.id;
});

if(exist) {
  g["setstatus"](true, a["id"], f);
  alert("Correcto!");
}

Puedes ver más sobre el uso de some aquí

3
  • Gracias, me funcionó!!! Commented el 22 may. 2017 a las 23:25
  • solo una cosa que creo me hizo falta en la condición, en caso que el if(exist) de false, debería existir un else con g["setstatus"](false, null, f); o algo parecido
    – learnercys
    Commented el 22 may. 2017 a las 23:28
  • Si, ya lo agregué, muchas gracias Learnercys! Commented el 22 may. 2017 a las 23:29

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.