3

Tengo una api básica con NodeJs y Express, lo que estoy intentado realizar es hacer múltiples inserciones dentro de un ciclo for, recorriendo un arreglo de clientes, por medio de una petición http utilizando la librería 'axios'. Este es mi código:

function agregarClientes(){
    var array = [{nombre:"Cliente1", numero:123},{nombre:"Cliente2", numero:123}]
    array.forEach(cliente => {
        axios({
            method: 'post',
            url: 'http://test.com', 
            data: cliente,
        }).then(response => {

        }).catch(e => {

        })
    })
}

Necesito que no se realice la siguiente inserción hasta que la anterior haya finalizado.

5
  • 2
    ¿Que se realice o que no se realice? el 3 jul. 2019 a las 9:18
  • 2
    No deberías ejecutar llamadas xhr (ajax) de manera síncrona, si no de manera consecutiva asíncronamente. ¿Cuál es el motivo por el que necesitas que se haga consecutivamente? ¿El usuario pulsa varias veces o porque tu aplicación realiza dos altas?
    – franmost
    el 3 jul. 2019 a las 9:20
  • 1
    then(response => {, Ejecuta aquí la siguiente llamada y así ...
    – EduBw
    el 3 jul. 2019 a las 9:48
  • 1
    Que no se realice la siguiente inserción hasta haber concluido la anterior @PabloLozano el 3 jul. 2019 a las 10:11
  • 1
    En tu caso yo implementaría la lógica de la inserción de clientes del lado servidor. Enviar una petición http por cada elemento de tu array no es para nada eficiente. En cambio si envías una sola petición enviando tu array de datos y ejecutas la inserción en serie en el lado servidor seguro es más eficiente. Saludos el 4 jul. 2019 a las 10:00

1 respuesta 1

2

Deberías usar Async/Await para poder manipular las Promises de manera sincrónica:

const agregarClientes = () => {
  const array = [
    { nombre: 'Cliente1', numero: 123 },
    { nombre: 'Cliente2', numero: 123 }
  ];
  array.forEach(async cliente => {
    try {
      await axios({
        method: 'post',
        url: 'http://test.com',
        data: cliente
      })
    } catch (e) {
      // catch error
    }
    // success
  });
}
3
  • 1
    En el último comentario, el OP indica que requiere realizar las operaciones de tal manera que cada nueva inserción (o solicitud POST) no se realice hasta que la anterior haya finalizado. Tu respuesta sugiere usar Promise.all(), la cual ejecutará las llamadas en paralelo y no en serie como pide el OP. Revisa un poco tu respuesta. Saludos el 3 jul. 2019 a las 16:45
  • @MauricioContreras no había leído el comentario, ahora modifico la respuesta.
    – Yakume
    el 3 jul. 2019 a las 16:52
  • Muchas gracias, buena aportación. el 23 jul. 2019 a las 16:05

Tu Respuesta

Al pulsar en “Publica tu respuesta”, muestras tu consentimiento a nuestros términos de servicio, política de privacidad y política de cookies

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