Проблема с try catch в async await - PullRequest
6 голосов
/ 04 июня 2019

Я борюсь с асинхронным ожиданием, попробуйте перехватить блоки в течение нескольких дней.

async function executeJob(job) {

  // necessary variable declaration code here

  try {
    do {
      let procedureRequests = await ProcedureRequestModel.find(filter, options);

      // doing process here...

    } while (fetchedCount < totalCount);

    providerNPIIds = [...providerNPIIds];

    // Fetch provider details
    let providerDetails = await esHelper.getProvidersById(providerNPIIds, true);

    try {

      let updateProviderCount = await UserProvider.updateAll(
          {
            userId: userId
          },
          {
            providers: providerNPIIds,
            countByType: providerCountType
          });

      if(updateProviderCount) {
        try {
          let destroyJobId = await  app.models.Job.destroyById(job.idd);
        } catch (e) {
          var err = new QueueError();
          console.log(err instanceof QueueError);
          throw new QueueError();
        }

      }
    } catch (e) {
      logger.error('Failed to update UserProviders  & Count: %O', err);
      throw e;
    }

    executeNextJob();
  } catch (e) {
    if(e instanceof QueueError) {
      console.log('Exiting from process');
      process.exit(1);
    } else {
      console.log('Working Code');
      buryFailedJobAndExecuteNext(job);
    }

  }
}

Является ли моя попытка уловить эту асинхронную функцию?

Так я создал пользовательский класс ошибок и экспортировал его глобально.

// error.js file

class QueueError extends Error {

}

global.QueueError = QueueError;

Требование:

Преднамеренно изменил job.id на job.idd в

let destroyJobId = await  app.models.Job.destroyById(job.idd); 

чтобы я мог отловить ошибку. Если есть ошибка, тогда бросьте недавно созданный пользовательский класс Error Но бросание QueueError приведет к регистрации

logger.error('Failed to update UserProviders  & Count: %O', err); 

тоже, хотя нет необходимости отлавливать там ошибки, так как блок try работает Если я выбрасываю QueueError, я хочу только перехватить ошибку только в последнем блоке перехвата.

Ниже приведена версия обратного вызова, я конвертирую ее в асинхронное ожидание.

 Updating providersNPIId & category count
     UserProvider.updateAll({userId: userId},
       {
         providers: providerNPIIds,
         countByType: providerCountType,
       }, function(err, data) {
         if (err) {
           logger.error('Failed to update UserProviders  & Count: %O', err);
           // throw new QueueError();
         }
         // Remove countProvider Job
         app.models.Job.destroyById(job.id, function(err) {
           if (err) {
             logger.error('Failed to remove countProvider job: %O', err);

           }
         });
       });

Ответы [ 2 ]

2 голосов
/ 04 июня 2019

Вы можете реорганизовать свой код в меньшие функции, которые возвращают обещание, для которого вы можете локально обернуть try-catch и обработать его.

async function executeJob(job) {

  // necessary variable declaration code here
  try {

    await doProcedure();

    providerNPIIds = [...providerNPIIds];

    // Fetch provider details
    let providerDetails = await esHelper.getProvidersById(providerNPIIds, true);
    const updateProviderCount = await getProviderCount(userId, providerNPIIds, providerCountType);

    if(updateProviderCount) {
        await destroyJobById(job.idd);
    }

    executeNextJob();
  } catch (e) {
    if(e instanceof QueueError) {
      console.log('Exiting from process');
      process.exit(1);
    } else {
      console.log('Working Code');
      buryFailedJobAndExecuteNext(job);
    }
  }
}

async function doProcedure() {
    try {
        do {
          let procedureRequests = await ProcedureRequestModel.find(filter, options);

          // doing process here...

        } while (fetchedCount < totalCount);
    } catch (err) {
        throw err;
    }
}

async function getProviderCount(userId, providerNPIIds, providerCountType) {
    try {
        let updateProviderCount = await UserProvider.updateAll({ userId: userId }, { providers: providerNPIIds, countByType: providerCountType });
        return updateProviderCount;
    } catch (err) {
      logger.error('Failed to update UserProviders  & Count: %O', err);
      throw e;
    }
}

async function destroyJobById(Id) {
    try {
          let destroyJobId = await app.models.Job.destroyById(Id);
    } catch (err) {
          throw err;
    }
}
0 голосов
/ 04 июня 2019

Это в основном то же самое, что Вы имеете:

1 function myFunctionPromise() {                                                                                     
2   return new Promise((response, reject) => {                                                                       
3       setTimeout(() => reject("Reject Err"), 3000);                                                                
4   });                                                                                                              
5 } 

7 async function myFunction() {                                                                                      
8      try {                                                                                                         
9        try {                                                                                                       
10          await myFunctionPromise();                                                                                
11        } catch(e) {                                                                                                
12          console.log("Second Err");                                                                                
13          throw e;                                                                                                  
14        }
15 
16      } catch (e) {                                                                                                 
17        throw e;
18      }
19 
20      console.log("test");
21 }
22 
23 
24 myFunction()

Единственное отличие, которое я вижу, это отказ от строки обещания # 3. Так что мне интересно, если:

let destroyJobId = ожидание app.models.Job.destroyById (job.idd);

Правильно отклоняет обещание.

...