web-dev-qa-db-ja.com

HTTPエラー処理はオブザーバブルでどのように機能しますか？

私はこのようなことをしている多くのチュートリアルを見ます：

http.get("...").subscribe(
  success => console.log('hello success'),
  error => console.log('bye error')
);

タイプや何かがないため、これがどのように機能するのかわかりませんただし私はそれを自分でやろうとしましたが、リクエストが常に成功することになりますエラー。何が問題ですか？

トラブルメーカー：

this.memberService.create(this.currentMember)
      .subscribe(
        success => {
          let mem: Member = success.json() as Member;
          if (this.selectedOrganization) {
            this.addMemberToOrganization(mem);
          } else if (this.selectedServiceProvider) {
            this.addMemberToServiceProvider(mem);
          } else {
            this.toastsService.error("lbl_users_service_provider_and_organization_undefined");
          }
        },
        error => console.log(error)
      );

memberServiceのCreate-Method：

  create(member: Member): Observable<any> {
    return this.http
      .post(this.RESOURCE_BASE_URL, member)
      .map(response => {
        if (response.status === 200) this.toastsSerivce.success(this.translateService.instant('lbl_users_member_created'));
        return response;
      })
      .catch(error => this.toastsSerivce.error(this.translateService.instant('lbl_users_member_create_failed')));
  }

私もエラーをキャッチしますが、subscribe部分は気にしないようです。エラーがある場合はjsonがないため、success.json()で失敗します。ただし、エラーがある場合は、successの代わりにerror =>...を呼び出すようにします。どんなアドバイスも大歓迎です。

angularangular-httpangular-observable
21
codepleb

問題は、throwingではなく、Observable.throw(errMsg)のエラーではないことだと思います。

したがって、次のように使用できます。

.catch((error:any) => Observable.throw(error.json().error || 'Server error'));

あなたの例では：

create(member: Member): Observable<any> {
    return this.http
      .post(this.RESOURCE_BASE_URL, member)
      .map(response => {
        if (response.status === 200) this.toastsSerivce.success(this.translateService.instant('lbl_users_member_created'));
        return response;
      })
      .catch((error:any) => Observable.throw(this.toastsSerivce.error(this.translateService.instant('lbl_users_member_create_failed'))));
  }

しかし、Angularが提案するようなエラーハンドラを使用できます here ：

private handleError (error: Response | any) {
    // In a real world app, you might use a remote logging infrastructure
    let errMsg: string;
    if (error instanceof Response) {
      const body = error.json() || '';
      const err = body.error || JSON.stringify(body);
      errMsg = `${error.status} - ${error.statusText || ''} ${err}`;
    } else {
      errMsg = error.message ? error.message : error.toString();
    }
    console.error(errMsg);
    return Observable.throw(errMsg);
  }

したがって、メソッドは次のようになります。

create(member: Member): Observable<any> {
    return this.http
      .post(this.RESOURCE_BASE_URL, member)
      .map(response => {
        if (response.status === 200) this.toastsSerivce.success(this.translateService.instant('lbl_users_member_created'));
        return response;
      })
      .catch(this.handleError);
  }

実際には、サービス内で作成できる他のメソッドに対して、よりクリーンで再利用可能です。

Angularの開発者が使用するような応答ハンドラthis.extractDataも使用することをお勧めします。

明らかに、エラーハンドルメソッドの内部に、独自のカスタムロジックを配置できますが、エラーの表示または処理方法によって異なります。

注：あなたのコードもここに投稿したコードもテストしませんでした。しかし、私はコンセプトを示し/表現したかった。 success everytimeに入らないようにするには、エラーをスローする必要があります。それをどのように扱うかは、あなたとあなたのアプリに依存します。

15
SrAxi

わたしにはできる：

this.http.post('http://example.com/path/', {sampleData: 'd'}).subscribe(
res => {alert('ok!');},
err => alert(err.error)
)
0
yaya