Продолжить при ошибке в RxJs, передаваемых с помощью mergeMap - PullRequest
Новая
       46

Продолжить при ошибке в RxJs, передаваемых с помощью mergeMap

4 голосов
/ 07 октября 2019

Я делаю несколько параллельных HTTP-запросов с каналом RxJs и оператором mergeMap.

При первом сбое запроса (предположим, что / urlnotexists выдает ошибку 404) он останавливает все остальные запросы.

Я хочу, чтобы он продолжал запрашивать все оставшиеся URL, не вызывая все оставшиеся mergeMap для этого неудавшегося запроса.

Я пытался играть с throwError и catchError из RxJs, но безуспешно.

index.js

const { from } = require('rxjs');
const { mergeMap, scan } = require('rxjs/operators');

const request = {
  get: url => {
    return new Promise((resolve, reject) => {
      setTimeout(() => {
        if (url === '/urlnotexists') { return reject(new Error(url)); }
        return resolve(url);
      }, 1000);
    });
  }
};

(async function() {
  await from([
    '/urlexists',
    '/urlnotexists',
    '/urlexists2',
    '/urlexists3',
  ])
    .pipe(
      mergeMap(async url => {
        try {
          console.log('mergeMap 1:', url);
          const val = await request.get(url);
          return val;
        } catch(err) {
          console.log('err:', err.message);
          // a throw here prevent all remaining request.get() to be tried
        }
      }),
      mergeMap(async val => {
        // should not pass here if previous request.get() failed 
        console.log('mergeMap 2:', val);
        return val;
      }),
      scan((acc, val) => {
        // should not pass here if previous request.get() failed 
        acc.push(val);
        return acc;
      }, []),
    )
    .toPromise()
    .then(merged => {
      // should have merged /urlexists, /urlexists2 and /urlexists3
      // even if /urlnotexists failed
      console.log('merged:', merged);
    })
    .catch(err => {
      console.log('catched err:', err);
    });
})();

$ node index.js
mergeMap 1: /urlexists
mergeMap 1: /urlnotexists
mergeMap 1: /urlexists2
mergeMap 1: /urlexists3
err: /urlnotexists
mergeMap 2: /urlexists
mergeMap 2: undefined <- I didn't wanted this mergeMap to have been called
mergeMap 2: /urlexists2
mergeMap 2: /urlexists3
merged: [ '/urlexists', undefined, '/urlexists2', '/urlexists3' ]

Я ожидаю сделать параллельные запросы GET и уменьшить их соответствующие значения в одном объекте в конце.

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

Любой совет?

Ответы [ 2 ]

1 голос
/ 09 октября 2019

Если вы хотите использовать RxJS, вам следует добавить обработку ошибок с помощью catchError и любые дополнительные задачи к одному запросу, прежде чем выполнять все ваши запросы одновременно с forkJoin. 

const { of, from, forkJoin } = rxjs;
const { catchError, tap } = rxjs.operators;

// your promise factory, unchanged (just shorter)
const request = {
  get: url => {
    return new Promise((resolve, reject) => setTimeout(
      () => url === '/urlnotexists' ? reject(new Error(url)) : resolve(url), 1000
    ));
  }
};

// a single rxjs request with error handling
const fetch$ = url => {
  console.log('before:', url);
  return from(request.get(url)).pipe(
    // add any additional operator that should be executed for each request here
    tap(val => console.log('after:', val)),
    catchError(error => {
      console.log('err:', error.message);
      return of(undefined);
    })
  );
};

// concurrently executed rxjs requests
forkJoin(["/urlexists", "/urlnotexists", "/urlexists2", "/urlexists3"].map(fetch$))
  .subscribe(merged => console.log("merged:", merged));
<script src="https://unpkg.com/@reactivex/rxjs@6.5.3/dist/global/rxjs.umd.js"></script>
0 голосов
/ 07 октября 2019

Если вы готовы отказаться от RXJS и просто решить с помощью async / await, это очень просто: 

const urls = ['/urlexists', '/urlnotexists', '/urlexists2', '/urlexists3']
const promises = urls.map(url => request(url)
const resolved = await Promise.allSettled(promises)

// print out errors
resolved.forEach((r, i) => {
  if (r.status === "rejected') {
    console.log(`${urls[i]} failed: ${r.reason})
  }
})

// get the success results
const merged = resolved.filter(r => r.status === "resolved").map(r => r.value)
console.log('merged', merged)

Это позволяет использовать Promise.allSettled предложенный вспомогательный метод. Если в вашей среде нет этого метода, вы можете реализовать его, как показано в этот ответ .

...