Обещания внутри foreach в угловых 4 / машинопись - PullRequest
Новая
       30

Обещания внутри foreach в угловых 4 / машинопись

0 голосов
/ 23 апреля 2019

Как лучше всего вызывать обещание внутри foreach?

У меня есть массив, и мне нужно отправить каждый элемент в RestFull API.Но в то же время я должен проверить, является ли ответ верным (у меня есть счетчик errorSendPayment).После всех запросов мне интересно отправить булево или Promise, если у меня есть одна (или более) ошибка.

Как я могу улучшить свой код? 

public sendPaymentValues(sale: any, session: string): Promise<boolean> {
    const headers = this.getHeader();
    // counter
    let errorSendPayment = 0;

    sale.data.list.forEach(payment=> {
      this.http.post(`${api}payment/`, payment, headers)
      .toPromise()
      .then((result: any) => {
        if (!result.sucess) {
          errorSendPayment++;
        }
      })
      .catch((response: HttpErrorResponse) => {
        this.httpErrorResponseHandler(response);
      });
    });

    const promise: Promise<boolean> = new Promise((resolve, reject) => {
      if (errorSendPayment <= 0) {
        resolve();
      } else {
        reject();
      }
    });
    return promise;
}

1 Ответ

1 голос
/ 23 апреля 2019

  1. Используйте zip оператор Observable для объединения вызовов HTTP.

  2. И проверьте, есть ли в результатах ошибка.

  3. Тогда дайте обещание, позвонив toPromise.

  4. Обновление Если вы используете версию выше RxJS 5.5, импортируйте оператор. 


import { zip } from 'rxjs';

public sendPaymentValues(sale: any, session: string): Promise<boolean> {
  const headers = this.getHeader();
  const obs$ = sale.data.list.map(payment => this.http.post(`${api}payment/`, payment, headers))

  return zip(...obs$).map(arr => arr.some(res => !res.success)).toPromise()
}
...