OS
<div *ngIf="serice.getUser$() | async"> и <div *ngIf="user$ | async">, который user$=this.serice.getUser$()?
И я вижу, так плохо очевидно
А вот в случае <button (click)="user$ = service.getUser()"> я ничего плохого не вижу. Делать однострочную функцию в компоненте - ну хз. Как минимум читать сложнее становится
И я вижу, так плохо очевидно
Я не вижу чем первый пример хуже второго
Кроме субъективного 'ну это очевидно'