2

Estoy iniciándome con Javascript y aún no manejo bien el tema asíncrono, promesas...etc

La cuestión es, que dentro de un callback necesito acceder a los datos de un array (recorrerlo mediante un for) y esto me lanza un error que no sé como arreglar.

Mi código:

var objetoQueGuardo = {'event': 'evento1', 'label': 'valor'};
var dataLayer = [];
dataLayer.push(objetoQueGuardo);

factura.createFactura(id, false, token)
                .then(function successCallback(response) {

            for (var i = 0; dataLayer.length; i++) {
                dataLayer[i].label;
            }

}, function errorCallback(response) {
                    console.warn(response);
});

He acortado un poco el código, para que ejemplo sea más simple. El contenido de dataLayer es un array de objetos y me quiero quedar con el atributo label de cada uno.

El error que me muestra:

 TypeError: dataLayer[i] is undefined
 Traza de la pila:
    successCallback@http://api.local/components/api/assets/js/factura.js:150:7
    h/<@http://api.local/components/api/assets/bower_components/angular/angular.min.js:135:278
    $digest@http://api.local/components/api/assets/bower_components/angular/angular.min.js:146:238
    $apply@http://api.local/components/api/assets/bower_components/angular/angular.min.js:149:464

 Possibly unhandled rejection: {}
3
  • 1
    ¿Qué se supone que es dataLayer[i].label? Si fuera una función, para invocarla harías dataLayer[i].label();. Si fuera un atributo/variable, lo usarías para algo alert(dataLayer[i].label);. Y, aunque se agradece el esfuerzo por hacer el código conciso, deberías mostrar dónde y cuando se asignan los valores a dataLayer[i] porque parece muy relevante para la pregunta.
    – SJuan76
    el 31 ago. 2017 a las 11:51
  • No es una función, es un atributo del objeto que guardo en el array. Ya lo he indicado en el texto antes de indicar el error, gracias
    – Norak
    el 31 ago. 2017 a las 11:53
  • No te vale hacer un forEach() ?? developer.mozilla.org/es/docs/Web/JavaScript/Referencia/… el 31 ago. 2017 a las 11:54

1 respuesta 1

5

Tu problema es la condición del bucle for:

for (var i = 0; dataLayer.length; i++) {
    dataLayer[i].label; // En este ejemplo esta sentencia realmente no hace nada, pero sirve para detectar el error
}

es equivalente a escribir

for (var i = 0; !!(dataLayer.length); i++) { // !!1 === true
    dataLayer[i].label;
}

puesto que dataLayer.length es 1, siempre se calcula true. Lo que necesitas es comparar la longitud con tu índice:

for (var i = 0; i < dataLayer.length; i++) {
    dataLayer[i].label;
}

Por cierto, lo de dar nombre a las funciones de callback es una buena práctica porque ayuda a depurar errores, como has visto aparece en el "stacktrace" del mensaje de error.

1
  • Tienes toda la razón, pensé que era error con la promesa y tenía mal escrita la condición del bucle for... Jo mil gracias!! a veces una se ciega y no ve el error
    – Norak
    el 31 ago. 2017 a las 14:07

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.