Angularでサーバからのレスポンスを取得後その値を画面に表示

Question

Angularで以下のようなことがやりたいと思っています。

1. あるボタンをクリック後、記事表示ページ(article.componentとします）を表示
2. article.componentのngOnInitでサーバにgetで記事の情報をリクエスト。（非同期）
3. レスポンスが返ってきた後、画面に記事の内容を表示

まず、最初は何も考えず、サーバにリクエストを投げる関数を作り、その関数をngOnInitで呼んでました。
この場合は初期表示時に、まだサーバからレスポンスが返ってきてないので画面に何も表示されませんでした。

次にサーバにリクエストを投げる関数の最後で@angular/routerのnavigateByUrlを使い、同画面を再表示するようにすることで、正しく記事の内容を表示できました。

しかし、何かこのやり方はダサく感じており、Angularのお作法的にはどうするのが正解なのか教えてください。

以下ソースコードです。

import { Component, OnInit } from '@angular/core';
import { ArticleService } from './article.service';
import { Article } from 'src/app/shared/models/article';
import { Router } from '@angular/router';

@Component({
  selector: 'app-article',
  templateUrl: './article.component.html',
  styleUrls: ['./article.component.scss']
})
export class ArticleComponent implements OnInit {
  article: Article;

  constructor(private articleService: ArticleService,
    private router: Router) { }

  ngOnInit() {
    this.getArticle(1);
  }

  getArticle(articleId: number) {
    this.articleService.getArticle(articleId).subscribe(
      response => {
        this.article = response;
        this.router.navigateByUrl('/');
      }
    );
  }
}

Comments

[BlueMoon]

articleが非同期更新されるのに画面表示が更新されない(双方向バインドができていない)趣旨の質問でしょうか。そうでしたらarticle.component.htmlの内容も記載できますか？

Answer 1

記事の情報を取得してからページを表示したいとの事なのでRouterResolveを使うのが良いかと
https://angular.jp/api/router/Resolve
https://angular.io/guide/router#fetch-data-before-navigating

resolveを使えばデータが揃った時点で画面遷移させることができます

Answer 2

サーバにリクエストを投げる関数の最後で@angular/routerのnavigateByUrlを使い、同画面を再表示するようにすることで、正しく記事の内容を表示できました。

何かこのやり方はダサく感じており、Angularのお作法的にはどうするのが正解なのか教えてください

Angularのお作法的には、rxjsのObservableを使って非同期処理をします。

サーバにリクエストを投げる（サーバーからデータを取得する）関数はObservableを返却し、関数を呼び出す方はそれをsubscribeしてその中でプロパティを設定してあげます。

もちろんPromiseとかでも非同期処理できますが、AngularのHttpClient.get()がObservableを返すため、それをそのまま流してあげるのがAngular的にはベターということになります。

Comments

[kuuritar]

すいません、少し言葉足らずでした。
ソースコードを追記しましたので、こちらを見ていただけるとわかりやすいかと思います。

こうすると非同期なのでcomponent読み込み時にはまだthis.articleに値が入っておらず、記事が表示されていません。
ですので、取得後にthis.router.navigateByUrl('/');こいつで表示させているのですが、何かダサく感じている、という感じです。

[xenbeat]

Observableで処理されていたんですね。

> 非同期なのでcomponent読み込み時にはまだthis.articleに値が入っておらず、記事が表示されていません。

念のためですが、これは当たり前なのでこれが問題ではないですよね？

> 取得後にthis.router.navigateByUrl('/');こいつで表示させているのですが、何かダサく感じている

これがなくても取得後にデータ表示できると思うのですが、あえてこのダサいと感じられているものを入れている理由を知りたいです。

[kuuritar]

すいません、こちら私の記載方法が悪かったようです。

this.articleが読み込み済の場合、表示、読み込んでいない場合は表示しないようにすることで、画面に正しく表示させることができました。