25

我看到很多教程都在做这样的事情：

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部分似乎并不在意。它在 处失败success.json()，因为如果有错误，则没有json。但是如果有错误，我希望它调用error =>...而不是success. 任何建议都将受到高度赞赏。

4

2 回答 2

16

我认为问题在于您不是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在此处提出的那样：

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每次都进入。你如何处理它取决于你和你的应用程序。

于 2017-06-06T09:36:31.513 回答
4

这个对我有用：

this.http.post('http://example.com/path/', {sampleData: 'd'}).subscribe(
  res => {alert('ok!');},
  err => {alert(err.error)}
)
于 2019-09-08T09:17:07.883 回答