Esperaba una serie de mensajes que dependen de los códigos de respuesta de los servicios, pero en realidad estoy obteniendo TypeError: result.push is not a function del código a continuación:
import axios from "axios"; const services = ["https://google.com", "https://facebook.com", "https://youtube.com"]; const results = await services.reduce(async (result, service) => { const response = await axios.get(service); result.push(response.status === 200 ? `${service} is up.` : `${service} seems down.`); return result; }, []); console.log(results); Intenté establecer el valor inicial de reduce() en Promise.resolve([]) pero no hace ningún cambio.
Sin embargo, cuando eliminé async & await stuff, el error ya no aparece, pero las comprobaciones de estado funcionan mal ahora.
import axios from "axios"; const services = ["https://google.com", "https://facebook.com", "https://youtube.com"]; const results = services.reduce((result, service) => { const response = axios.get(service); result.push(response.status === 200 ? `${service} is up.` : `${service} seems down.`); return result; }, []); console.log(results);¡NO use funciones que devuelvan promesas en el método .reduce() !
¿Por qué estás usando el método .reduce() cuando lo que quieres que te devuelva es una matriz de la misma longitud? Use .map() en su lugar.
Además, use try...catch en caso de que ocurra algún error y manéjelo apropiadamente. Si una solicitud falla por completo, todo el programa fallará con su código actual.
Hay varias maneras de hacer esto. La primera forma es obtener primero todos los datos y luego mapearlos:
import axios from 'axios'; const services = ['https://google.com', 'https://facebook.com', 'https://youtube.com']; // Create an array of promises which will fetch the services // We do this PRIOR to manipulating the data const promises = services.map((service) => (async () => { try { const res = await axios.get(service); return { status: res.status, service, }; } catch (error) { return { status: null, service, }; } })() ); // Wait for all of them to finish const serviceData = await Promise.all(promises); const results = serviceData.map(({ status, service }) => (status && status === 200 ? `${service} is up` : `${service} is down`)); console.log(results);O bien, puede hacerlo todo en un solo ciclo:
import axios from 'axios'; const services = ['https://google.com', 'https://facebook.com', 'https://youtube.com']; const results = []; for (const service of services) { try { const res = await axios.get(service); results.push(`${service} is ${res?.status === 200 ? 'up' : 'down'}`); } catch (error) { results.push(`${service} is down`); } } console.log(results); También puede usar for await en combinación con la primera solución si se siente elegante y le gusta la optimización.
import axios from 'axios'; const services = ['https://google.com', 'https://facebook.com', 'https://youtube.com']; const results = []; const promises = services.map((service) => (async () => { try { const res = await axios.get(service); return { status: res.status, service, }; } catch (error) { return { status: null, service, }; } })() ); for await (const { status, service } of promises) { results.push(`${service} is ${status === 200 ? 'up' : 'down'}`); } console.log(results);El tiempo de ejecución de todas estas soluciones es: