You can also this one, you don't need to run loop in this case

    private destroy$ = new Subject();
    
    myservice.megohd().pipe(takeUntil(destroy$)).subscribe();
    
    ngOnDestroy() {
      this.destroy$.next();
      this.destroy$.complete();
    }

read here (https://www.learnrxjs.io/operators/filtering/takeuntil.html)

You also have the third option, which is a custom RxJS operator.

I had created one and found out that Netanel Basal had found it too, so I'll give his clean code.

You can install UntilDestroyed or use the code :

function isFunction(value) {
  return typeof value === 'function';
}

export const untilDestroyed = (
  componentInstance,
  destroyMethodName = 'ngOnDestroy'
) => <T>(source: Observable<T>) => {
  const originalDestroy = componentInstance[destroyMethodName];
  if (isFunction(originalDestroy) === false) {
    throw new Error(
      `${
        componentInstance.constructor.name
      } is using untilDestroyed but doesn't implement ${destroyMethodName}`
    );
  }
  if (!componentInstance['__takeUntilDestroy']) {
    componentInstance['__takeUntilDestroy'] = new Subject();

    componentInstance[destroyMethodName] = function() {
      isFunction(originalDestroy) && originalDestroy.apply(this, arguments);
      componentInstance['__takeUntilDestroy'].next(true);
      componentInstance['__takeUntilDestroy'].complete();
    };
  }
  return source.pipe(takeUntil<T>(componentInstance['__takeUntilDestroy']));
};

Then your subscriptions become

this.myService.subject.pipe(untilDestroyed(this)).subscribe(...);

Note that because of AOT compilation, you have to write a ngOnDestroy method, otherwise the operator isn't able to create it from scratch.

There's better, more succinct solution: https://www.npmjs.com/package/subsink by Ward Bell

export class SomeComponent implements OnDestroy {
  private subs = new SubSink();

  ...
  this.subs.sink = observable$.subscribe(...);
  this.subs.sink = observable$.subscribe(...);
  this.subs.sink = observable$.subscribe(...);
  ...

  // Unsubscribe when the component dies
  ngOnDestroy() {
    this.subs.unsubscribe();
  }
}

I would prefer Method 2.

On Method 1 there is no problem.

It works perfectly fine. The problem with this approach is that we’re mixing observable streams with plain old imperative logic.

Method 2 is a built in mechanism to enhance this approach.

Read here

You have good answers provided though I would try to avoid all the above methods if possible and go for async 'pipe, that is allow angular do the subscription and unsubscription.

Lets consider a number of Observables say 5 observables

observale1$: Observable<IDataDto1>;
observale2$: Observable<IDataDto2>;
observale3$: Observable<IDataDto3>;
observale4$: Observable<IDataDto4>;
observale5$: Observable<IDataDto5>;

To avoid subscribing to all this we can create one single Observable say v$

import { forkJoin } from 'rxjs';
v$ = forkJoin({
  observale1: observale1$,
  observale2: observale2$,
  observale3: observale3$,
  observale4: observale4$,
  observale5: observale5$
});

With the above we can wrap our component.html in an *ngIf and allow angular to auto subscribe and unsubscribe

<ng-container *ngIf='v$ | async as v'>
  <!-- Your html code here -->
</ng-container>

Method 2 because Subscription.unsubscribe() does more than just seemingly unsubscribing, see its source code here.

e.g.

class SomeClass implements OnInit, OnDestroy { {

  // Setup your subscriptions
  private subscriptions = new Subscription();

  // Example Observable service
  constructor(private someService: SomeService) {}

  ngOnInit(): void {
    this.subscriptions.add(this.someService.getStuff());
    // ...
    this.subscriptions.add(/* etc */);
  }

  ngOnDestroy() {
    this.subscriptions.unsubscribe();
  }

}