¿Cómo puedo reescribir el módulo de promesa de Axios con solo sintaxis async/await? Por favor, soy nuevo en Axios y lo prometo, sintaxis asíncrona => ¿qué debo poner dentro de .push() ?
const axios = require("axios"); const _ = require("lodash"); const queue = require("queue"); // To control the request rate const GetDataQueue_B = queue({ autostart: true, concur: 1 }); // someone's codes const getDATA = () => { return new Promise((resolve, reject) => { // To make sure only one request call GetDataQueue_B.push( async () => { try{ const res = await axios.get(`${whichAPI}/api`); resolve( _.map( _.get(res, "data.infor"), obj => ({ quotes:obj.quotes, author:boj.author, }) ) ); }catch(e) {reject(e);console.log(e);} } ); });a continuación está mi intento: => ¿qué debo poner dentro de .push() ?
const getDATA_1 = async () => { try{ const res = await axios.get(`${whichAPI}/api`); const data = _.map( _.get(res, "data.infor"), obj => ({ quotes:obj.quotes, author:boj.author, }) ) GetDataQueue_B.push(); // => what should i put inside the .push() method? return data; }catch(e){console.log(e);} };queue no es la mejor opción aquí para el código limpio porque, si bien permite empujar funciones que devuelven una promesa, en sí misma no devuelve una promesa, por lo que siempre se verá obligado a usar una new Promise en algún momento y resolve en su devolución de llamada exitosa o al final de su propio código. Supongo que esa es la razón principal por la que no puedes mejorar este código.
Recomiendo usar p-limit en su lugar. Funciona así:
const pLimit = require("p-limit"); const myQueue = pLimit(5); // Allow only 5 things running in parallel const promise = myQueue(async () => { /* stuff */ }); // promise will resolve once the passed function ran, and will return its // result.Entonces, el código se ve así:
const axios = require("axios"); const _ = require("lodash"); const pLimit = require("p-limit"); // To control the request rate const GetDataQueue_B = pLimit(1); // someone's codes const getDATA = () => GetDataQueue_B(async () => { const res = await axios.get(`${whichAPI}/api`); return _.map( _.get(res, "data.infor"), obj => ({ quotes: obj.quotes, author: boj.author, }) ) ); }); Eliminé el try / catch porque en la nueva versión realmente no haría nada; un error aquí rechazaría la promesa resultante de todos modos.
Nota al margen: el uso lodash no es necesario aquí. Siempre es bueno mantener las dependencias al mínimo y usar la funcionalidad integrada en su lugar. Podríamos reescribir el código sin lodash de la siguiente manera:
const axios = require("axios"); const pLimit = require("p-limit"); // To control the request rate const GetDataQueue_B = pLimit(1); // someone's codes const getDATA = () => GetDataQueue_B(async () => { const res = await axios.get(`${whichAPI}/api`); return res.data?.infor?.map(({ quotes, author }) => ({ quotes, author })) ?? []; }); Si estaba seguro de que la matriz res.data.infor existe y tampoco le importan las propiedades adicionales que no sean las quotes y el author en el resultado, esto podría simplificarse aún más:
const getDATA = () => GetDataQueue_B(async () => { const res = await axios.get(`${whichAPI}/api`); return res.data.infor; });