Как отправить ответ в середине нескольких асинхронных вызовов в базу данных в NodeJS?

Question

Как отправить ответ в середине нескольких асинхронных вызовов в базу данных в NodeJS?

1

Необходимо прекратить функцию POST и отправить Error: something качестве ответа, без прерывания программы.

Образец for-loop:

for (let i = 0; i < req.body.listoftouristsbyid.length; i++) {
  client.query("SELECT tourists.id FROM tourists WHERE tourists.id = " + req.body.listoftouristsbyid[i], function(err, result) {
    done();
    if (result.rows.length === 0) {
      console.log("Tourist " + req.body.listoftouristsbyid[i] + " you want to add does not exist");
      res.status(500);
      return res.render("Bad listoftoutistbyid request, student with id " + i + " does not exist");
    }
  })
}

Что я должен писать вместо return res.render, поэтому POST не будет работать, функция будет завершена с кодом ошибки в ответ, но без сбоя программы, чтобы позже я мог отправить больше запросов?

λjk.jk 27 сен. 2018, в 20:18

Источник

1

Вместо того, чтобы делать много запросов к БД, я бы просто сделал один запрос, объединив их с OR ...
Jonas Wilms 27 сен. 2018, в 17:52
2

И я надеюсь, что вы делаете проверку входных данных.
Jonas Wilms 27 сен. 2018, в 17:53
0

Вы можете использовать оператор try ... catch .
rv7 27 сен. 2018, в 17:54
0

@rahul Нет, он не может
Jonas Wilms 27 сен. 2018, в 17:55
0

@Jonas Wilms "И я надеюсь, что вы делаете проверку входных данных" Что это значит?
λjk.jk 27 сен. 2018, в 17:58
1

Это означает, что любой посетитель вашего сайта может получить удовольствие от вашей базы данных xkcd.com/327
Jonas Wilms 27 сен. 2018, в 18:00
0

Что вы подразумеваете под "без сбоя программы"? Вы устанавливаете себе код ошибки 500. Может быть, вы хотите установить 404 вместо этого?
TGrif 27 сен. 2018, в 18:19
0

Ошибка TGrif [ERR_HTTP_HEADERS_SENT]: невозможно установить заголовки после их отправки клиенту. Это то, что я получил в терминале. Единственное, что я получаю в ответе Почтальона, это пользовательский «Не удалось получить ответ, произошла ошибка в ...», но мое сообщение об ошибке полностью игнорируется. Команда, которую я сейчас пробую, это res.status (404) .send ({status: 404, message: 'internal error', type: 'internal'}); я не знаю, почему это не работает как следует
λjk.jk 27 сен. 2018, в 18:36
0

Ошибка очень ясна, Cannot set headers after they are sent to the client . Вы можете отправить ответ только один раз. Вторая итерация вашего цикла for пытается отправить ответ, вот где он падает.
Dhirendra 27 сен. 2018, в 18:48
1

Ну, не делайте много запросов в цикле и попробуйте отправить ответ после каждого из них возвращается. Там может быть только один ответ.
Bergi 27 сен. 2018, в 19:41

Показать ещё 8 комментариев

Теги:

javascript

ecmascript-6

node.js

express

ecmascript-2017

2 ответа

0

Попробуйте использовать async.map для решения вашей проблемы. for цикла не будет работать с асинхронными функциями.

async.map(
    req.body.listoftouristsbyid, 
    function(touristId, callback) {
        client.query("<query>", function(err, result) {
            if (result.rows.length === 0) {
                callback(new Error("not found"));
            }
            callback(null, result);
        });
    },
    function(err, result) {
        // send response here.
    } 
);

Dhirendra 27 сен. 2018, в 17:42

Ещё вопросы

Вместо того, чтобы делать много запросов к БД, я бы просто сделал один запрос, объединив их с OR ...
И я надеюсь, что вы делаете проверку входных данных.
Вы можете использовать оператор try ... catch .
@Jonas Wilms "И я надеюсь, что вы делаете проверку входных данных" Что это значит?
Это означает, что любой посетитель вашего сайта может получить удовольствие от вашей базы данных xkcd.com/327
Что вы подразумеваете под "без сбоя программы"? Вы устанавливаете себе код ошибки 500. Может быть, вы хотите установить 404 вместо этого?
Ошибка TGrif [ERR_HTTP_HEADERS_SENT]: невозможно установить заголовки после их отправки клиенту. Это то, что я получил в терминале. Единственное, что я получаю в ответе Почтальона, это пользовательский «Не удалось получить ответ, произошла ошибка в ...», но мое сообщение об ошибке полностью игнорируется. Команда, которую я сейчас пробую, это res.status (404) .send ({status: 404, message: 'internal error', type: 'internal'}); я не знаю, почему это не работает как следует
Ошибка очень ясна, Cannot set headers after they are sent to the client . Вы можете отправить ответ только один раз. Вторая итерация вашего цикла for пытается отправить ответ, вот где он падает.
Ну, не делайте много запросов в цикле и попробуйте отправить ответ после каждого из них возвращается. Там может быть только один ответ.

NAVIN · Accepted Answer · 2018-09-27T17-44-00.000Z

Вы можете использовать async-await для этого, так как он очень часто используется среди асинхронных обработчиков вызовов, таких как Promise и Generators.

Здесь пример кода:

app.post('/APIendpoint', async (req, res) => {

    let listoftouristsbyid      = req.body.listoftouristsbyid;

    let dbResult = [];

    // Can put this whole for loop inside try too. Then success response will also be inside try after for-loop
    for (let i = 0; i < listoftouristsbyid.length; i++) {

        try {
            let result = await callDB( "SELECT tourists.id FROM tourists WHERE tourists.id = " + listoftouristsbyid[i] );
            dbResult.push(result);
        }
        catch (err) {

            console.log("Tourist " + listoftouristsbyid[i] + " you want to add does not exist");

            res.status(500);
            return res.send("Bad listoftoutistbyid request, student with id " + i + " does not exist");

        }
    }

    console.log("Successfully fetched all records ", dbResult);

    res.status(200);
    return res.send(dbResult);

});

function callDB(query) {

    return new Promise ( (resolve, reject) => {
        client.query(query, function(err, result) {

            if (result.rows.length === 0) {
                reject("not found");
            } else {
                resolve(result.rows);
            }

        })
    });
}