我看到很多教程都是这样做的:
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
。任何建议都非常感谢。
答案 0 :(得分:14)
我认为问题在于 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
。你如何处理它取决于你和你的应用程序。
答案 1 :(得分:1)
它对我有用:
this.http.post('http://example.com/path/', {sampleData: 'd'}).subscribe(
res => {alert('ok!');},
err => alert(err.error)
)