1

我不完全理解我需要在哪里以及如何在 Angular 组件中声明可观察对象/主题。

目前，我开发了一个与 MovieDB API 交互的网站，并且一切正常，但同时我知道我的代码很糟糕，因为在销毁组件后没有清理订阅，但是要进行这些清理，我需要最不了解如何正确使用 RxJS。

我认为我的用法不正确，因为我在每次与页面交互时都有新订阅。另外据我了解，它们需要在构造函数中声明。

这个页面的想法是有一个输入表单，用户可以在其中输入查询并检查单选按钮以选择要搜索的内容：“电视”或“电影”。当有搜索结果时，会出现一个按钮来展开结果。

所以，这里是代码：

import {Component, OnDestroy} from '@angular/core';
import {ISearchParams} from '../../models/search-params.interface';
import {ShowService} from '../../services/show.service';
import {IResultsIds} from '../../models/results.interface';
import {distinctUntilChanged} from 'rxjs/operators';
import {Subscription} from 'rxjs';

@Component({
  selector: 'app-search',
  templateUrl: './search-tab.component.html',
  styleUrls: ['./search-tab.component.scss']
})
export class SearchTabComponent implements OnDestroy {

  searchParams!: ISearchParams;

  searchResults!: IResultsIds;

  searchSub!: Subscription;
  showMoreSub!: Subscription;

  constructor(private movieService: ShowService) {
  }

  search(searchParams: ISearchParams): void {
    this.searchParams = searchParams;

    this.searchSub = this.movieService.search(searchParams)
      .pipe(distinctUntilChanged())
      .subscribe(results => {
        this.searchResults = results;
      });
  }

  showMore(): void {
    if (!this.isFinished()) {
      this.searchParams.page++;

      this.showMoreSub = this.movieService.search(this.searchParams)
        .subscribe(results => {
          this.searchResults!.ids.push(...results.ids);
        });
    }
  }

  isFinished = () => this.searchParams.page >= this.searchResults!.total_pages;

  ngOnDestroy(): void {
    // this.searchSub.unsubscribe();
    // this.showMoreSub.unsubscribe();
  }
}

和 HTML：

<main class="main-content search-container">
  <app-search-form (searchParams)="search($event)"></app-search-form>
  <div class="results" *ngIf="searchResults">
    <app-show-description *ngFor="let id of searchResults.ids"
                          [showType]="searchParams.type"
                          [showId]="id"></app-show-description>
  </div>
  <button *ngIf="searchResults && !isFinished()"
          (click)="showMore()"
          class="load-btn more-btn">show more...</button>
</main>

如果你能帮助我并告诉我哪里出错了，我将不胜感激。在更多的时间里，一切都以这种方式工作，但我想了解 RxJS 的用法。

输入输入并得到结果

扩展按钮

UPD @H3AR7B3A7 非常感谢你，你让我的知识有点清楚了！但现在我有另一个误解：如何将 2 个数组 observables 转换为一个？我用谷歌搜索，但找不到解决问题的方法：我有加载更多电影的功能 - 它必须扩展 distinctMovies$ 数组，它看起来就像一个数字（id）数组，但我不能加入两个数组，我得到的只是 [...], [...] 但不是 [......]

我试过这个：

showMore(): void {
    this.searchParams.page++;

    this.distinctMovies$ = concat(this.distinctMovies$,
      this.movieService.search(this.searchParams)
        .pipe(pluck('ids')))
      .pipe(tap(console.log));
  }
4

2 回答 2

2

对我来说，您似乎了解基础知识。

虽然，您不需要在构造函数中声明任何内容。

另外据我了解，它们需要在构造函数中声明。

您通常只使用构造函数来注入服务，就像您已经在做的那样。

您可能希望使用 ngOnInit() 方法来声明组件的初始状态：

export class SearchTabComponent implements OnInit {
  distinctMovies$!: Observable<Movie[]>

  ngOninit(): void {
    //...
  }
}

您可以通过从不订阅组件代码来解决很多问题。这样，您也不必取消订阅 OnDestroy ......

例如（在你的 onInit 中）：

this.distinctMovies$ = this.movieService.search(searchParams).pipe(distinctUntilChanged())

在模板中只需使用异步管道：

*ngFor = "let movie of distinctMovies$ | async"

除了不必取消订阅之外，您还可以通过使用异步管道而不是订阅来使用 OnPush ChangeDetectionStrategy。

于 2021-10-28T23:47:00.320 回答
1

乍一看，这里有一些可以改进的地方。

关于退订

有几种模式被认为是“好的做法”。其中之一是：

// Define destroy subject
readonly destroy$ = new Subject<any>();

// On each subscription add
.pipe(takeUntil(this.destroy$))

// And just emit value to destroy$ subject in onDestroy hook
// And all streams that had takeUntil will be ended
ngOnDestroy(): void {
  this.destroy$.next();
  this.destroy$.complete();
}

关于搜索东西

当您处理异步请求（例如 API 调用）时，您必须考虑例如多次单击“搜索”按钮会发生什么。会有几个 API 调用，对吧？但我们不能 100% 确定服务器响应的顺序。

例如，如果您单击：

单击 1、单击 2、单击 3、单击 4... 将产生 APICall1、APICall2、APICall3、APICALL4... 按该顺序...但是来自服务器的响应可以按其他顺序（是的，这是可能的） )，同样如果您毫不拖延地点击多次，那么您的服务器此时会收到很多请求，但您可能只需要最后一个请求。

因此，一种常见的解决方案是拥有一个正在监听例如搜索更改的流：

searchTerm = new Subject();

ngOnInit() {

  this.searchTerms
    .pipe(
      distinctUntilChanged(),
      // Debounce for 250 miliseconds (avoid flooding backend server with too many requests in a short period of time)
      debounceTime(250),
      // Call api calls
      switchMap(terms => {
        return this.movieService.serach(terms);
      })),
      // Unsubsribe when onDestroy is triggered
      takeUntil(this.destroy$),
    .subscribe(results => {
         // Push/set results to this.searchResults
         if (this.addingMore) {
         //  push in this.searchResults
         } else {
            this.searchResults = results;
         }

         this.addingMore = false;
    });

}

search(searchTerms) {
  this.searchParams = searchParams;
  this.searchTerms.next(searchTerms);
}

showMore(): void {
  if (this.isFinished() return;

  this.searchParams.page++;

  // You can for example set a flag to know if its search or loadMore
  this.addingMore = true;
 
  this.searchTerms.next(searchTerms);
}
于 2021-10-28T23:41:30.410 回答