【发布时间】:2016-12-26 06:15:02
【问题描述】:
今天我在尝试实现一个抽象的http服务时遇到了以下问题。该服务应该是扩展所有其他 http 服务的基础。
目前的实现如下,跳过其他方法进行说明:
@Injectable()
export abstract class HttpWrapper<T> {
private options: RequestOptions;
constructor(private http: Http, private endpointUrl: string) {
let headers = new Headers();
headers.append('Content-Type', 'application/json');
this.options = new RequestOptions({ headers: headers });
}
public getAll(): Observable<T[]>{
return this.http.get(this.endpointUrl, this.options)
.map(this.extractAll)
.catch(this.handleError);
}
abstract handleError(error: any):Observable<Response>;
abstract extractOne(res: Response):T;
abstract extractAll(res: Response):T[];
}
现在,如果我想使用抽象 HttpWrapper,我执行以下操作:
@Injectable()
export class BlastReportService extends HttpWrapper<Item> {
constructor(http: Http) {
super(http,'/api/items');
}
handleError(error: any):Observable<Response>{
//Handling error
return Observable.throw(error);
}
extractAll(res: Response):Item[]{
let body = res.json();
let formatedBody = body.map((item: Item) => {
item = new Item(item);
return blastReport;
});
return formatedBody || [{}];
}
}
但是这样做我得到以下编译错误:
Type 'Observable<Response>' is not assignable to type 'Observable<T[]>'.
Type 'Response' is not assignable to type 'T[]'.
Property 'find' is missing in type 'Response'.
我无法理解这一点,因为方法 extractAll 清楚地返回 Item[] 并在映射从服务器返回的结果时使用。
我决定实现这个抽象的 HttpWrapper “保持 DRY”。我不确定这是否是最好的方法。
【问题讨论】:
-
extractAll 返回 Item[],但您的类应该是 HttpWrapper
。所以它应该返回一个 BlastReport[]。还不清楚此错误消息来自何处。请发布完整和准确的错误。 -
另外,你的方法应该返回一个 Observable
,但是 handleError 返回一个 Observable 。 -
对那个 blastReport 感到抱歉,这不应该是问题所在。编辑那个。
标签: angular typescript rxjs angular2-observables