Problema:
Quiero devolver empresas de un determinado rango (km) desde una determinada ubicación. Estas empresas están en una base de datos que actualmente contiene 2 entradas para probar. Entre otras cosas, también uso la API de matriz de distancia de Google para calcular la distancia.
Después de que no funcionó, una depuración me mostró que la función devuelve [Promise {<pending>}, Promise {<pending>}] .
Código:
const axios = require("axios"), knex = require('knex')(require('../knexfile')); const getAllByDistance = (location) => knex('companies') .select() .then(entries => entries.map(company => getDistance(location, `${company.street}, ${company.postcode} ${company.place}`) .then(distance => { knex('companies') .select() .where(parseInt(company.maximum_distance_km) >= parseInt(distance.toString().slice(0, -3))) })) ); const getDistance = async (loc1, loc2) => { const origins = encodeURI(`?origins=${loc1}`); const destinations = encodeURI(`&destinations=${loc2}`); const key = `&key=${process.env.VUE_APP_GOOGLE_MAPS_API_KEY}`; const config = { method: 'get', url: `https://maps.googleapis.com/maps/api/distancematrix/json${origins}${destinations}${key}`, headers: {} }; return await axios(config) .then((response) => { return response.data['rows'][0]['elements'][0]['distance'].value; }) .catch((err) => { console.log(err); }); }La llamada de función con depuración:
companyService .getByDistance(location) .then(companies => { console.log(companies) res.status(200); res.json(companies); }) .catch(err => { res.status(500); res.end(`Error: ${err.message}`); });Le sugiero que comience con async / await completo en lugar de una combinación de eso y then() .
Comenzando en el método getDistance :
const getDistance = async (loc1, loc2) => { const origins = encodeURI(`?origins=${loc1}`); const destinations = encodeURI(`&destinations=${loc2}`); const key = `&key=${process.env.VUE_APP_GOOGLE_MAPS_API_KEY}`; const config = { method: 'get', url: `https://maps.googleapis.com/maps/api/distancematrix/json${origins}${destinations}${key}`, headers: {} }; try{ const response = await axios(config) return response.data['rows'][0]['elements'][0] } catch(e){ console.log(e); // return 0; // What should we return if there's an error? } } Ahora, el método getAllDistances se vuelve mucho más fácil de administrar sin todo ese anidamiento (advertencia: no sé nada sobre knex, solo me baso en su código que, como comenté, parece extraño que consulte repetidamente a todas sus empresas ... pero tratando de replicar la misma funcionalidad que creo que tienes)
const getAllByDistance = async (location) => { const entries = await knex("companies").select(); const results = []; for(let i=0;i<entries.length;i++){ const company = entries[i]; const distance = await getDistance(location, `${company.street}, ${company.postcode} ${company.place}`); const result = await knex('companies') .select() .where(parseInt(company.maximum_distance_km) >= parseInt(distance.toString().slice(0, -3))); results.push(result); } return results; }Lo anterior tiene algunos inconvenientes, principalmente que recorre secuencialmente la lista original de empresas que obtienen la distancia y luego carga todas las empresas dentro de esa distancia, pero estoy seguro de que lo ayudará a comenzar con un algoritmo más eficiente.