Обработка исключений в возвращенном обещании

Question

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

Проблема в том, что я не могу вспомнить какие-либо исключения, которые могут возникнуть всердце возвращенного обещания.

Код разработан так, что и reject, и resolve возвращают действительные данные.Итак, если вы измените значение const CONDITION на 0.4, мы получим отказ.Если значение const CONDITION останется на 0.6, мы получим разрешение.Пока это работает.

Однако в тех случаях, когда у нас возникают структурные сбои, например, в приведенном ниже примере, когда мы пытаемся передать неправильное имя переменной в отклонение:

let reasoxxxx = '__FAILED__';
reject({error: reason, data: output});

IЯ не могу вызвать throw, чтобы вызвать ошибку.По этой причине я получаю обычное уродливое сообщение и не могу вспомнить правильное исключение:

Exchange request was rejected due to error(s).
(node:224) UnhandledPromiseRejectionWarning: ReferenceError: reason is not defined
(node:224) UnhandledPromiseRejectionWarning: Unhandled promise rejection. This error originated either by throwing inside of an async function without a catch block or by rejecting a promise which was not handled with .catch(). (rejection id: 1)
(node:224) [DEP0018] DeprecationWarning: Unhandled promise rejections are deprecated. In the future, promise rejections that are not handled will terminate the Node.js process with a non-zero exit code.

Есть идеи?Отрезанный код должен работать.

function fakeFetch() {
  // Promisify the request.
  return new Promise((resolve, reject) => {
    // Emulate an asynchroneous fetch.
    setTimeout(() => {
      let result = 0.4; // Change to 0.4 to trigger a failed fetch.
      if (result < 0.5) {;
        reject('__FAIL__');
      } else {
        resolve({name: 'apple', price: '1234.12', time: 1549926859970});
      }
    }, 2000);
  });
}

async function sendExchangeRequest(id, pair, symbols, callback)
{
  let err, result
  await fakeFetch().then((output) => { result = output }).catch((error) => {err = error})
  if(err){
    result = 'None'
  }else{
    err = 'None'
  }
  callback(err, result)
}

async function fetchExchangeData(id, pair, symbols) {
    // Promisify the request.
    try {
        return new Promise((resolve, reject) => {
            try {
                // Send the request.
                sendExchangeRequest(id, pair, symbols, ((err, output) => {
                    try{
                        if(err){
                            // Soft Failure
                            console.log('Exchange request was rejected due to error(s).');
                            reject({error: err, data: output});
                        }else{
                            // Success
                            console.log('Exchange request was successful.');
                            resolve({error: err, data: output});
                        }
                    } catch(error) {
                        throw error;
                    }
                }));
            } catch(error) {
                console.log('---\n', error, '\n---');
                throw error;
            }
        });
    } catch(error) {
        // Bubble up the error?
        console.log('+++\n', error, '\n+++');
        throw error;
    }
}

(async () => {
  await fetchExchangeData('myid', 'MYPAIR', 'mySymbol')
  .then((result) => console.log(result))
  .catch((failure) => console.log(failure))
})();

--- РЕДАКТИРОВАТЬ (01) ---

Я обновил свой пример snipped, чтобы включить поддельный вызов API.Надеюсь, это прояснит мой вопрос.

Answer 1

Я думаю, вы не понимаете концепцию асинхронных функций.Ваши первые try/catch в fetchExchangeData по сути ничего не делают, и в этом нет ничего асинхронного.Try / catch внутри исходного обещания также практически бесполезен, так как я уверен, что большинство / все проблемы с sendExchangeRequest, скорее всего, будут решаться с помощью функции обработчика ошибок (err).Что касается блоков try / catch, вы не должны выбрасывать ошибки в обещании, вы должны вместо этого отклонить ошибку (что приведет к ее появлению).Попробуйте / поймайте, только если вы находитесь в асинхронной функции и НЕ используете обратные вызовы.

function fetchExchangeData(id, pair, symbols) {
    // Promisify the request.
    return new Promise((resolve, reject) => {
        try {
            // Send the request.
            sendExchangeRequest(id, pair, symbols, ((err, output) => {
                try{
                    if(err){
                        // Soft Failure
                        console.log('Exchange request was rejected due to error(s).');
                        let reason = '__FAILED__';
                        reject({error: reason, data: output});
                    }else{
                        // Success
                        console.log('Exchange request was successful.');
                        resolve({error: '__NONE__', data: output});
                    }
                } catch(error) {
                    reject(error);
                }
            }));
        } catch(error) {
            console.log('---\n', error, '\n---');
            reject(error);
        }
    });
}