Skip to main content
se añadieron 1976 caracteres en el cuerpo
Origen Enlace
ArianJM
  • 3.4k
  • 10
  • 22

Con tu código quedaría algo de este estilo:

// Función que hace las llamadas a las API. Recibe una lista de IDs y devuelve una promesa de que hará las llamadas.
function llamadasAPI(listaIDs) {
    const rp = require('request-promise');
    let respuesta = [];
    const promesas = [];

    listaIDs.forEach((item) => {
        let uri = `url/${item.param}`;
        const promesa = rp({
            rejectUnauthorized: false,
            method: 'GET',
            uri: `${params.baseURI}${uri}`,
            json: true,
            headers: {...}
        });

        // Cada ID de la lista tiene su propia promesa de hacer una llamada a la API.
        promesa.then(res => {
            // Y cuando la promesa termine, ahí metes los datos que necesites.
            respuesta.push({
                ...
            });
        });

        // Guarda una lista de las promesas
        promesas.push(promesa);
    });

    // La función devuelve una promesa que se cumplirá una vez terminen todas las promesas del loop.
    return new Promise(resolve => {
    
        // Cuando terminen todas las promesas, se resolverá esta con el valor de respuesta.
        Promise.all(promesas).then(() => {
            resolve(respuesta);
        });
    });
}

// Llamamos la función y guardamos la promesa.
const promesaLlamadaAPI = llamadasAPI([ 1, 2, 3 ]);

promesaLlamadaAPI.then(respuesta => {
    // Lo que necesites hacer con los datos de las llamadas.
});

Las últimas líneas las puedes hacer sin promesaLlamadaAPI así:

llamadasAPI([ 1, 2, 3 ]).then(respuesta => {
    // Lo que necesites hacer con los datos de las llamadas.
});

Con tu código quedaría algo de este estilo:

// Función que hace las llamadas a las API. Recibe una lista de IDs y devuelve una promesa de que hará las llamadas.
function llamadasAPI(listaIDs) {
    const rp = require('request-promise');
    let respuesta = [];
    const promesas = [];

    listaIDs.forEach((item) => {
        let uri = `url/${item.param}`;
        const promesa = rp({
            rejectUnauthorized: false,
            method: 'GET',
            uri: `${params.baseURI}${uri}`,
            json: true,
            headers: {...}
        });

        // Cada ID de la lista tiene su propia promesa de hacer una llamada a la API.
        promesa.then(res => {
            // Y cuando la promesa termine, ahí metes los datos que necesites.
            respuesta.push({
                ...
            });
        });

        // Guarda una lista de las promesas
        promesas.push(promesa);
    });

    // La función devuelve una promesa que se cumplirá una vez terminen todas las promesas del loop.
    return new Promise(resolve => {
    
        // Cuando terminen todas las promesas, se resolverá esta con el valor de respuesta.
        Promise.all(promesas).then(() => {
            resolve(respuesta);
        });
    });
}

// Llamamos la función y guardamos la promesa.
const promesaLlamadaAPI = llamadasAPI([ 1, 2, 3 ]);

promesaLlamadaAPI.then(respuesta => {
    // Lo que necesites hacer con los datos de las llamadas.
});

Las últimas líneas las puedes hacer sin promesaLlamadaAPI así:

llamadasAPI([ 1, 2, 3 ]).then(respuesta => {
    // Lo que necesites hacer con los datos de las llamadas.
});
Origen Enlace
ArianJM
  • 3.4k
  • 10
  • 22

Y no podrías usar Promesas sin async/await? Por ejemplo algo así:

let array = [ 1, 1.5, 2 ];

const promesas = array.map((item) => {
  return new Promise(resolve => {
    setTimeout(() => {
      resolve(`esperado ${item} segundos`);
    }, item * 1000);
  });
});

// Promise.all recibe un array de Promise y ejecutará la función en then() cuando estén todas resueltas.
Promise.all(promesas).then(respuesta => {
  // respuesta es un array con los valores devueltos. Haz lo que quieras con respuesta.
  console.log(respuesta);
});