I have a function that subscribes to a search api. In the map function I would like to map the items to objects. I do not get an error but the response is always empty.
this is my code:
return this.http.get(searchURL)
.map((res: Response) => res.json())
.map(json => json.items.forEach(item => {
new SearchResult(
item.id,
item.title,
item.price
);
}) || []);
You need to change forEach with map and return from it:
return this.http.get(searchURL)
.map((res: Response) => res.json())
.map(json => json.items.map(item => {
return new SearchResult(
item.id,
item.title,
item.price
);
}))
.catch((err: Response) => {
// handle error
})
forEach doesn't return anything, while map produces new array with items which you return from callback.
Also, note that || [] check is useless. If json.items is an array then map will always produce another array. If it's not an array then it will throw error, which you need to handle with .catch block attached after your .map (Obeservable map I mean).
You need to map over the returned list and transform it instead of forEach.
Also, if the list of items is empty map will return an empty list, you don't need to do the || [] anymore.
return this.http.get(searchURL)
.map((res: Response) => res.json())
.map(json => json.items.map(item => {
return new SearchResult(
item.id,
item.title,
item.price
);
}));
