Угловой 5 перехват - запросы дублируются - PullRequest
Новая
       40

Угловой 5 перехват - запросы дублируются

0 голосов
/ 18 октября 2018

Я использую перехватчик с HttpInterceptor в angular 5, и у меня проблема с rxjs, когда все мои http запросы дублируются. 

import { Router } from '@angular/router';
import { Injectable, ApplicationRef } from '@angular/core';
import { HttpEvent, HttpInterceptor, HttpHandler, HttpRequest, HttpResponse } from '@angular/common/http';
import { Observable } from 'rxjs/Rx';
import 'rxjs/add/observable/throw';
import 'rxjs/add/operator/catch';
import 'rxjs/add/observable/empty';
import { NgxSpinnerService } from 'ngx-spinner';
import { ErrorHandlingService } from '../../service/error-handling.service';

@Injectable()
export class ApiRequestInterceptor implements HttpInterceptor {

  private count: number = 0;

  constructor(
    private readonly spinner: NgxSpinnerService,
    private readonly router: Router,
    private readonly errorHandling: ErrorHandlingService,
    private readonly applicationRef: ApplicationRef) { }

  intercept(req: HttpRequest<any>, next: HttpHandler): Observable<HttpEvent<any>> {
    this.count++;

    if (this.count === 1) {
      this.spinner.show();
    }

    return next.handle(req)
      .catch((err: any) => {
        this.count--;
        return Observable.throw(err);
      }).do(event => {
        if (event instanceof HttpResponse) {
          this.count--;
          if (this.count === 0) this.spinner.hide();
        }
      });
  }
}

enter image description here

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

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

Ответы [ 2 ]

0 голосов
/ 19 октября 2018

Пожалуйста, посмотрите здесь для решения с счетчиком и подсчетом запросов. 

intercept(request: HttpRequest<any>, next: HttpHandler): Observable<HttpEvent<any>> {        
            this.count++;
            if (this.count === 1) {
              this.spinner.show();
            }
            return next.handle(request)
                .pipe(
                    tap((event: HttpEvent<any>) => {
                        if (event instanceof HttpResponse) {
                            this.count--;
                            if (this.count === 0) setTimeout(this.spinner.hide());
                        }
                    }, (err: any) => {
                            this.count = 0;
                            setTimeout(this.spinner.hide());
                    })
                );
    }
0 голосов
/ 18 октября 2018

Вы звоните next.handle() дважды.Просто верните первый, не звоня subscribe: 

intercept(req: HttpRequest<any>,next: HttpHandler): Observable<HttpEvent<any>> {
    this.count++;

    if (this.count === 1) {
      this.spinner.show();
    }

    return next.handle(req)
      .catch((err: any) => {
        this.count--;
        return Observable.throw(err);
      }).do(event => {
        if (event instanceof HttpResponse) {
          this.count--;
          if (this.count === 0) setTimeout(this.spinner.hide());
        }
    });
}

Небольшой совет, обновитесь до angular6, чтобы воспользоваться новыми rxjs и тряской дерева

...