Несколько Асинхронных / Ожидать Блокировка Try-Catch

Question

Несколько Асинхронных / Ожидать Блокировка Try-Catch

1

Я здесь, потому что у меня проблемы с отладкой моего приложения. Это очень раздражает, не вижу, почему мое приложение падает. Я использовал обещания (с помощью блоков /catch), но я получаю необходимость использования async/wait.

У меня есть метод, в котором я много раз жду. Проблема здесь в том, что если мое приложение падает, потому что по какой-либо причине я никогда не знаю, в чем проблема. Я описал блок следующим образом:

         static async processCSGOGroupsAndUsers (groupName)  {

    try{
        const csgoApiData = await csgoApi(groupName);

        const parsedData = await xmltojson(csgoApiData);

        const id = parsedData.memberList.groupID64;
        //const members = await retrieveMembers(groupName, parsedData.memberList.memberCount);
        const totalUsers = await UsersService.processCSGOUsers(id, parsedData);

        const csgoGroup = {
            name: parsedData.memberList.groupDetails.groupName,
            siteUrl: parsedData.memberList.groupDetails.groupURL,
            id,
            totalUsers
        };

        await GroupsDao.save(csgoGroup);

    }catch (err){
        return err;
    }

}


  static async processCSGOUsers (groupId, parsedData) {

    try{

        let steamIdsArr = [];

        const usersSteamIdsObj = parsedData.memberList.members.steamID64;

        Object.keys(usersSteamIdsObj).forEach(key => {
            //if (steamIdsArr.length < 2)  // TODO csGOBackPackAPI don't let me do more than 50 request per hour
                steamIdsArr.push({
                    steam_group_id_64: groupId,
                    steam_id_64: usersSteamIdsObj[key]
                });
        });

        //const filteredUsers =  await UserService.filterUsersByInventoryValue(steamIdsArr);
        UsersDao.saveUsers(steamIdsArr);

    }   catch(err){

        console.log(err);

        return err;
    }
}

static processCSGOGroups(req, res){
    GroupService
        .processCSGOGroupsAndUsers(req.body.group)
        .then( () => res.status(200).end())
        .catch( error => res.status(400).send(error));

}

Есть ли лучший подход, чем мой?

Nicolas Taboada 31 июль 2017, в 05:16

Источник

2

« Я понимаю необходимость использования async / await. » - что? Там нет необходимости вообще. Если у вас есть рабочий код, который использует методы обещания, зачем его менять?
Bergi 31 июль 2017, в 03:05
0

Вы определенно не хотите return err , просто игнорируя ее и рассматривая ее как успешное значение результата. Если вы не пытаетесь обработать какие-либо ошибки, почему вы вообще используете try / catch ?
Bergi 31 июль 2017, в 03:07
0

Я начал использовать async / await, потому что нужно работать с переменными области видимости высокого уровня. Да, я хочу обрабатывать ошибки, как вы видите в коде, я делаю внешние вызовы API, операции с базой данных
Nicolas Taboada 31 июль 2017, в 03:18
2

Ну так как ты хочешь с ними справиться? Код, который вы показали, ничего не делает (кроме одного журнала), когда что-то выходит из строя.
Bergi 31 июль 2017, в 04:06
0

Я инкапсулирую свою бизнес-логику в сервисе. Эта услуга вызывается контроллером высокого уровня. Таким образом, было бы идеально, если бы я мог выдать (как Java) ошибку контроллеру (чтобы я мог ответить с кодом состояния 400 + ошибка).
Nicolas Taboada 31 июль 2017, в 04:30
0

Как я уже сказал, для этого вы не должны catch и return его. throw это вместо этого, или просто не поймайте это во-первых, чтобы исключение просто пузырилось.
Bergi 31 июль 2017, в 04:41
1

изменить блок catch с } catch (err) { return err; } to } catch (err) { console.log(err) } . или throw new Error(err) . Ключевое слово return используется внутри функций для возврата значения. Вы не можете использовать return таким образом. Чтобы увидеть, что происходит, когда запускается этот код ошибки, откройте новый файл и вставьте в него return 'ok i see now' а затем запустите его.
agm1984 31 июль 2017, в 06:40

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

Теги:

javascript

node.js

async-await

asynchronous

1 ответ

Ещё вопросы

« Я понимаю необходимость использования async / await. » - что? Там нет необходимости вообще. Если у вас есть рабочий код, который использует методы обещания, зачем его менять?
Вы определенно не хотите return err , просто игнорируя ее и рассматривая ее как успешное значение результата. Если вы не пытаетесь обработать какие-либо ошибки, почему вы вообще используете try / catch ?
Я начал использовать async / await, потому что нужно работать с переменными области видимости высокого уровня. Да, я хочу обрабатывать ошибки, как вы видите в коде, я делаю внешние вызовы API, операции с базой данных
Ну так как ты хочешь с ними справиться? Код, который вы показали, ничего не делает (кроме одного журнала), когда что-то выходит из строя.
Я инкапсулирую свою бизнес-логику в сервисе. Эта услуга вызывается контроллером высокого уровня. Таким образом, было бы идеально, если бы я мог выдать (как Java) ошибку контроллеру (чтобы я мог ответить с кодом состояния 400 + ошибка).
Как я уже сказал, для этого вы не должны catch и return его. throw это вместо этого, или просто не поймайте это во-первых, чтобы исключение просто пузырилось.
изменить блок catch с } catch (err) { return err; } to } catch (err) { console.log(err) } . или throw new Error(err) . Ключевое слово return используется внутри функций для возврата значения. Вы не можете использовать return таким образом. Чтобы увидеть, что происходит, когда запускается этот код ошибки, откройте новый файл и вставьте в него return 'ok i see now' а затем запустите его.

Mike · Answer 1 · 2018-08-22T05-09-00.000Z

Я создал пакет NPM, чтобы помочь в подобных ситуациях. https://www.npmjs.com/package/@simmo/task

Ниже показано использование примера. Идея заключается в том, что она должна помочь удалить замену try/catch с помощью условной логики - надеюсь, что поможет сделать все более читаемым! :)

import task from '@simmo/task'

const { error, data } = await task(fetch('/some-api'))

if (error) {
  // Failure
  console.error(error)
} else {
  // Success
  console.log(data)
}