Angular 7 Guard - Лучшая практика - PullRequest
Новая
       14

Angular 7 Guard - Лучшая практика

0 голосов
/ 21 июня 2019

Мне нужна помощь, чтобы улучшить мой код на Angular 7 Guard. Я думаю, что не рекомендуется сначала подписываться на текущего пользователя, чтобы использовать его после этого на switchMap. Заранее спасибо за помощь. 

import { Injectable } from '@angular/core';
import { CanActivate } from '@angular/router';
import { User } from '@app/core/models/user/user';
import { select, Store } from '@ngrx/store';
import * as fromRoot from '@redux/app.reducers';
import * as invoiceActions from '@redux/payment/subscription/invoice/invoice.actions';
import * as fromSubscription from '@redux/payment/subscription/subscription.reducer';
import * as fromUser from '@redux/user/user.reducers';
import { Observable, of } from 'rxjs';
import { filter, switchMap, take, tap } from 'rxjs/operators';

@Injectable()
export class LoadSubscriptionInvoicesGuard implements CanActivate {
  constructor(private store: Store<fromRoot.AppState>) {
  }

  getFromAPI(user: User): Observable<any> {
    return this.store.pipe(
      select(fromSubscription.selectInvoiceLoaded),
      tap((loaded: boolean) => {
        if (!loaded) {
          this.store.dispatch(new invoiceActions.LoadCollection(user));
        }
      }),
      filter((loaded: boolean) => loaded),
      take(1));
  }

  canActivate(): Observable<boolean> {
    let user : User;
    this.store.pipe(select(fromUser.selectCurrentUser),
    take(1))
    .subscribe((_user: User) => user = _user);

    return this.getFromAPI(user).pipe(
      switchMap(() => of(true)));
  }
}

1 Ответ

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

Это неплохая практика, но ваш код не самый лучший для этого. 

  canActivate(): Observable<boolean> {
    return this.store.pipe(
      select(fromUser.selectCurrentUser),
      switchMap(user => this.getFromAPI(user)),
      map(value => !!value),
      take(1),
    );
  }

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

...