Как избавиться от дублирования подобных функций?

Question

[D]

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

getArchivedColleaguesList(): void {
    this.sharedColleaguesService.archivedColleaguesList
      .pipe(takeUntil(this.ngUnsubscribe))
      .pipe(filter(archivedColleaguesList => archivedColleaguesList !== null && archivedColleaguesList.length > 0))
      .subscribe((archivedColleaguesList: Colleague[]) => {
        this.archivedColleaguesList = archivedColleaguesList;
      })
  }

На данный момент у меня эта функция просто скопипащена в три разных компонента.
Можно ли как-то реализовать это по-другому, чтобы не таскать этот кусок кода в те компоненты, где он необходим?

Comments

[Ivan Brezhnev]

Как минимум должно быть так, takeUntil желательно ставить последнем в цепочке операторов

getArchivedColleaguesList(): void {
  this.sharedColleaguesService.archivedColleaguesList
    .pipe(
      filter(archivedColleaguesList => archivedColleaguesList !== null && archivedColleaguesList.length > 0),
      takeUntil(this.ngUnsubscribe)
    )
    .subscribe((archivedColleaguesList: Colleague[]) => {
      this.archivedColleaguesList = archivedColleaguesList;
    })
}

[Ivan Brezhnev]

Если интересно почему, можно подробно почитать в статье https://habr.com/ru/post/598151/

Answer 1

[Антон Швец]

function filterList() {
  return filter(list=> list!= null && list.length > 0)
}

можно использовать в пайпе.
Остальное - подписка и тейкантил - не нужно