如何将两个呼叫的响应合并为一个响应？

Question

我得到一个空响应 ( {} )，而我预期的响应格式为：

{
    locationResponse: "location foo",
    forecastResponse: "forecast bar"
}

在我的索引文件中，我有：

const {getCity} = require('./routes/city');
const {getForecasts} = require('./routes/forecast');

app.get('/forecasts', function (req, res) {
    var location = getCity(req, res);
    var forecast = getForecasts(req, res);

    //these are logged as undefined
    console.log("Inside index.js");
    console.log(location);
    console.log(forecast);
    res.send({locationResponse: location, forecastResponse: forecast});
});

在预测文件中，我有以下内容，城市文件中有一个类似的：

module.exports = {
    getForecasts: (req, res) => {
       var result = //mySQL DB calls and processing
       console.log("Inside getForecasts");
       console.log(result); //actual result printed
       return "Forecast";
    }

更新：所以我在每次调用的返回语句之前添加了一些日志，并发现日志按以下顺序打印，这意味着它没有按预期工作，因为我没有考虑到它们是异步调用的事实。

Inside index.js
undefined
undefined
Inside getForecasts
{result}

Answer 1

这里的问题是，在你的./routes/forecast/ getForecasts 方法中，你告诉响应发送，数据“预测”。 每个请求只应使用 res.send 一次，因为这将解析响应并返回给客户端。

相反，您的 getForecasts 方法应该只返回您需要的任何数据，并且您的索引文件应该处理响应。 如果您也需要 getForecasts 来处理响应，也许是因为您将请求直接发送到不需要位置数据的预测端点，那么您可以重构您的代码，以便索引和预测都调用以获取您的数据需要。 例如：

/* index.js */
const {getCity} = require('./data/city');
const {getForecasts} = require('./data/forecast');

app.get('/forecasts', function (req, res) {
    var location = getCity();
    var forecast = getForecasts();
    res.send({locationResponse: location, forecastResponse: forecast});
});

/* data/forecast.js */
module.exports = {
    getForecasts: () => {
        return "Forecast";
    }
};

/* data/city.js */
module.exports = {
    getCity: () => {
        return "City";
    }
};

然后你还可以有：

/* routes/forecast.js */
const {getForecasts} = require('../data/forecast');
module.exports = {
    getForecasts: (req, res) => {
        res.send(getForecasts());
    }
};

上面的内容可能过于复杂，但我假设如果您使用routes目录，您可能希望将路由处理程序存储在那里。 希望这可以帮助。

Answer 2

似乎您的 getCity() 和 getForecasts() 函数都是异步的。 这些异步函数返回一个承诺而不是实际的响应。

所以你可以使用简单的asysn/await或者 JS 中的Promise.all来解决这个问题。

选项 1：在将消息记录到控制台之前，使用await作为要解决的承诺：

app.get('/forecasts', async function (req, res) {
    var location = await getCity(req, res);
    var forecast = await getForecasts(req, res);

    //these are logged as undefined
    console.log("Inside index.js");
    console.log(location);
    console.log(forecast);
    res.send({locationResponse: location, forecastResponse: forecast});
});

选项 2：使用Promise.all()等待所有Promise.all()完成。

app.get('/forecasts', function (req, res) {
    var list = await Promise.all([getCity(req, res), getForecasts(req, res)]);

    //these are logged as undefined
    console.log("Inside index.js");
    console.log(list[0]);
    console.log(list[1]);
    res.send({locationResponse: list[0], forecastResponse: list[1]});
});

Answer 3

您可以使用async/await语法。

app.get('/forecasts', async function (req, res) {
    var location = await getCity(req, res);
    var forecast = await getForecasts(req, res);

    //these are logged as undefined
    console.log("Inside index.js");
    console.log(location);
    console.log(forecast);
    res.send({locationResponse: location, forecastResponse: forecast});
});